Skip to content

Enclave-tools#187

Merged
QuentinI merged 9 commits intocelo-integration-rebase-13.1from
ag/batcher-reg
Jul 17, 2025
Merged

Enclave-tools#187
QuentinI merged 9 commits intocelo-integration-rebase-13.1from
ag/batcher-reg

Conversation

@QuentinI
Copy link
Copy Markdown
Collaborator

@QuentinI QuentinI commented Jul 2, 2025

Closes #<ISSUE_NUMBER>

This PR:

  • Adds small enclave-tools utility to the op-batcher
  • It supports: building the batcher EIF image, registering the resulting PCR and launching the EIF
  • Adds an utility to the batcher image entrypoint to support passing in command-line arguments dynamically. This means that just changing the batcher arguments no longer means we need to rebuild and re-register the EIF.
  • enclave-tools run supports this new functionality: you can pass batcher arguments with --args --argname,argvalue,--argname2,argvalue...

This PR does not:

Key places to review:


@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jul 2, 2025

CLA assistant check
All committers have signed the CLA.

@QuentinI QuentinI force-pushed the ag/batcher-reg branch 4 times, most recently from 9f4fd5b to 1e1f9b3 Compare July 3, 2025 16:14
@QuentinI QuentinI marked this pull request as ready for review July 3, 2025 16:17
@QuentinI QuentinI force-pushed the ag/batcher-reg branch 2 times, most recently from 70fa625 to 2d2610f Compare July 3, 2025 16:51
@philippecamacho
Copy link
Copy Markdown
Collaborator

Could you add a section in the README_ESPRESSO.md file to explain when and how to use this tool?

@philippecamacho
Copy link
Copy Markdown
Collaborator

How can we test the tool is working (even manually)?

@QuentinI
Copy link
Copy Markdown
Collaborator Author

QuentinI commented Jul 7, 2025

Could you add a section in the README_ESPRESSO.md file to explain when and how to use this tool?

Sure, I'll add a section on how to use this tool with examples of usage

Copy link
Copy Markdown
Collaborator

@philippecamacho philippecamacho left a comment

Choose a reason for hiding this comment

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

LGTM

@QuentinI QuentinI merged commit 2d389d5 into celo-integration-rebase-13.1 Jul 17, 2025
29 checks passed
@QuentinI QuentinI deleted the ag/batcher-reg branch July 17, 2025 13:31
shenkeyao pushed a commit that referenced this pull request Jul 18, 2025
Co-authored-by: Philippe Camacho <philippe@espressosys.com>
shenkeyao pushed a commit that referenced this pull request Jul 18, 2025
Co-authored-by: Philippe Camacho <philippe@espressosys.com>
dailinsubjam pushed a commit that referenced this pull request Oct 4, 2025
Co-authored-by: Philippe Camacho <philippe@espressosys.com>
dailinsubjam pushed a commit that referenced this pull request Oct 7, 2025
Co-authored-by: Philippe Camacho <philippe@espressosys.com>
shenkeyao pushed a commit that referenced this pull request Jan 15, 2026
Co-authored-by: Philippe Camacho <philippe@espressosys.com>
shenkeyao pushed a commit that referenced this pull request Feb 2, 2026
Co-authored-by: Philippe Camacho <philippe@espressosys.com>
shenkeyao pushed a commit that referenced this pull request Feb 10, 2026
Co-authored-by: Philippe Camacho <philippe@espressosys.com>
shenkeyao pushed a commit that referenced this pull request Feb 12, 2026
Co-authored-by: Philippe Camacho <philippe@espressosys.com>
QuentinI added a commit that referenced this pull request Feb 20, 2026
Co-authored-by: Philippe Camacho <philippe@espressosys.com>
shenkeyao pushed a commit that referenced this pull request Mar 20, 2026
Co-authored-by: Philippe Camacho <philippe@espressosys.com>
QuentinI added a commit that referenced this pull request Apr 9, 2026
Co-authored-by: Philippe Camacho <philippe@espressosys.com>
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