Skip to content

Conversation

@PierreJeanjacquot
Copy link
Member

@PierreJeanjacquot PierreJeanjacquot commented Oct 14, 2025

Datasets use standard encryption that makes them agnostic to the TEE framework as soon as their secret is present in the SMS.
Creating datasetorders without a specified TEE framework let the dataset owner to allow the dataset to be consumed in any TEE framework.

Current limitation: when datasetorder does not specify the TEE framework to use, signDatasetorder and publishDatasetorder perform a preflight to check the dataset secret is in the default SMS (scone). We may want to check whether the secret is present in all SMS or completely skip this check.

@github-actions
Copy link

github-actions bot commented Oct 14, 2025

🐳 Hadolint Dockerfile Lint Results

Click to expand detailed results
Dockerfile:17 DL3016 warning: Pin versions in npm. Instead of `npm install <package>` use `npm install <package>@<version>`

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
44.4% Coverage on New Code (required ≥ 80%)
12.2% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Copy link
Contributor

@abbesBenayache abbesBenayache left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@PierreJeanjacquot PierreJeanjacquot force-pushed the feat/dataset-tee-agnostic-tag branch from d04903f to 2cb5b5b Compare October 27, 2025 09:45
@PierreJeanjacquot PierreJeanjacquot merged commit a5c3e86 into master Oct 27, 2025
7 checks passed
@PierreJeanjacquot PierreJeanjacquot deleted the feat/dataset-tee-agnostic-tag branch October 27, 2025 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants