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

CI: build nitro docker image #19

Merged
merged 51 commits into from
Dec 11, 2023
Merged

CI: build nitro docker image #19

merged 51 commits into from
Dec 11, 2023

Conversation

sveitser
Copy link

@sveitser sveitser commented Dec 4, 2023

Close EspressoSystems/espresso-sequencer#783

  • Add CI step to build docker images for nitro.
  • [WIP] download the wasm binary from espresso builds. We first need a tagged release to finish this. So there will be a follow up PR once this one is merged.
  • No need to review the new Dockerfile, it's the same as the existing one except for downloading the wasm machine from our github releases for the WIP step above.

- Fix github if expressions.
- Add trigger for release tag.
For this to work we first need a tagged release on github that exported
the wasm machine as release artifact.
@sveitser sveitser marked this pull request as ready for review December 11, 2023 13:27
@sveitser
Copy link
Author

There still something wrong with the docker outputs field. Will investigate.

It's not quite clear how to configure the docker build step to do the
right thing. With this change the docker image is not built on PR but
can still be built via manual triggering if that's desired.
@sveitser
Copy link
Author

I removed the PR trigger for the docker build workflow. Once the PR is merged the workflow can still be manually triggered for testing.

@sveitser sveitser requested a review from Ancient123 December 11, 2023 14:55
Copy link

@nomaxg nomaxg left a comment

Choose a reason for hiding this comment

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

LGTM

@sveitser sveitser merged commit b1a7133 into integration Dec 11, 2023
5 checks passed
@sveitser sveitser deleted the ma/nitro-docker branch December 11, 2023 15:35
zacshowa pushed a commit that referenced this pull request Nov 26, 2024
zacshowa pushed a commit that referenced this pull request Nov 26, 2024
Add support for custom token fee - ERC20 based rollups
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.

Build nitro node docker image on CI and push to registry
3 participants