Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add general config and secrets opts to snapshot creator #2471

Merged
merged 10 commits into from
Jul 24, 2024

Conversation

manuelmauro
Copy link
Contributor

@manuelmauro manuelmauro commented Jul 24, 2024

What ❔

This PR adds --config-path and --secrets-path options to /core/bin/snapshots_creator. Allowing the binary to source configurations from files.

Why ❔

Needed in order to run the command within zk_toolbox.

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • Code has been formatted via zk fmt and zk lint.

core/bin/snapshots_creator/src/main.rs Outdated Show resolved Hide resolved
core/bin/snapshots_creator/src/main.rs Outdated Show resolved Hide resolved
Deniallugo
Deniallugo previously approved these changes Jul 24, 2024
Deniallugo
Deniallugo previously approved these changes Jul 24, 2024
@Deniallugo Deniallugo marked this pull request as ready for review July 24, 2024 13:00
@Deniallugo Deniallugo added this pull request to the merge queue Jul 24, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 24, 2024
@Deniallugo Deniallugo added this pull request to the merge queue Jul 24, 2024
@RomanBrodetski RomanBrodetski removed this pull request from the merge queue due to a manual request Jul 24, 2024
@RomanBrodetski RomanBrodetski merged commit 0f475c9 into main Jul 24, 2024
47 checks passed
@RomanBrodetski RomanBrodetski deleted the manuel-snapshot-create-file-based-config branch July 24, 2024 14:35
github-merge-queue bot pushed a commit that referenced this pull request Jul 25, 2024
🤖 I have created a release *beep* *boop*
---


##
[24.12.0](core-v24.11.0...core-v24.12.0)
(2024-07-25)


### Features

* add general config and secrets opts to snapshot creator
([#2471](#2471))
([0f475c9](0f475c9))
* Update to consensus 0.1.0-rc.4 (BFT-486)
([#2475](#2475))
([ff6b10c](ff6b10c))


### Bug Fixes

* consensus secrets generator
([#2484](#2484))
([dea6969](dea6969))


### Performance Improvements

* writing tx to bootloader memory is no longer quadratic
([#2479](#2479))
([1c443e5](1c443e5))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: zksync-era-bot <[email protected]>
Co-authored-by: Roman Brodetski <[email protected]>
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