Skip to content

feat: add salt and secret params to env.deploy (backport #21183)#21191

Closed
AztecBot wants to merge 3 commits intobackport-to-v4-stagingfrom
claudebox/backport-deploy-salt-secret-v4
Closed

feat: add salt and secret params to env.deploy (backport #21183)#21191
AztecBot wants to merge 3 commits intobackport-to-v4-stagingfrom
claudebox/backport-deploy-salt-secret-v4

Conversation

@AztecBot
Copy link
Collaborator

@AztecBot AztecBot commented Mar 5, 2026

Backport of #21183 to v4.

Cherry-pick had conflicts in oracle version files. Resolved by bumping v4 oracle version to 14, aligned with next per #21183 (comment).

ClaudeBox log: http://ci.aztec-labs.com/846172556ed88e43-2

Originally requested by @xorsal and @zkfrov.

This simply adds a new `deploy_opts` function which takes
`DeployOptions`, via which custom salt and secret can be speficied. In
the future we might be able to allow specifying more things.

The bigger change is that the deployment oraacle now supports salt,
secret (and deployer, currently hardcoded), so we should not need to
break this API in the near future.

Closes #16656

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Martin Verzilli <martin@aztec-labs.com>
@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label Mar 5, 2026
@ludamad ludamad marked this pull request as ready for review March 6, 2026 00:15
@ludamad ludamad requested a review from nventuro as a code owner March 6, 2026 00:15
@ludamad ludamad enabled auto-merge (squash) March 6, 2026 00:15
@ludamad ludamad force-pushed the backport-to-v4-staging branch from 4ff264b to 2d467c9 Compare March 12, 2026 22:29
@nventuro nventuro disabled auto-merge March 16, 2026 13:52
@nventuro
Copy link
Contributor

Closing, backporting to v4-next instead.

@nventuro nventuro closed this Mar 16, 2026
@nventuro nventuro deleted the claudebox/backport-deploy-salt-secret-v4 branch March 16, 2026 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants