Skip to content

Conversation

jbern0rd
Copy link
Contributor

@jbern0rd jbern0rd commented Oct 3, 2025

No description provided.

@jbern0rd jbern0rd self-assigned this Oct 3, 2025
@jbern0rd jbern0rd marked this pull request as ready for review October 6, 2025 06:56
@jbern0rd jbern0rd requested a review from Copilot October 6, 2025 09:14
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements bulk processing functionality to support multiple datasets in TEE sessions. The feature enables the handling of multiple datasets through IPFS-stored dataset orders, validating them on-chain and injecting appropriate environment variables into TEE enclaves.

Key changes include:

  • Added IPFS client for reading bulk dataset orders from IPFS
  • Implemented bulk dataset order validation with signature and volume checks
  • Extended TEE session creation to handle bulk processing environment variables

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/main/java/com/iexec/sms/tee/bulk/IpfsClient.java New IPFS client interface for reading bulk CIDs and dataset orders
src/main/java/com/iexec/sms/tee/session/base/SecretSessionBaseService.java Core bulk processing logic with dataset order validation and environment variable injection
src/main/resources/application.yml Added IPFS gateway configuration
src/test/java/com/iexec/sms/tee/session/TeeSessionTestUtils.java Refactored test utilities to support bulk processing test scenarios
src/test/java/com/iexec/sms/tee/session/base/SecretSessionBaseServiceTests.java Comprehensive test coverage for bulk processing functionality
gradle.properties Updated dependency versions to support new features

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

sonarqubecloud bot commented Oct 7, 2025

@jbern0rd jbern0rd merged commit e2a758d into main Oct 7, 2025
3 checks passed
@jbern0rd jbern0rd deleted the feature/bulk-processing branch October 7, 2025 13:33
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