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

Add tests to docker build #339

Merged
merged 5 commits into from
Oct 7, 2024
Merged

Add tests to docker build #339

merged 5 commits into from
Oct 7, 2024

Conversation

ignasirv
Copy link
Contributor

@ignasirv ignasirv commented Sep 25, 2024

  • Added test to docker build network to check build correctness
  • New gha only to develop to test docker build gha without pushing the image

@cla-bot cla-bot bot added the cla-signed label Sep 25, 2024
@ignasirv ignasirv force-pushed the feature/fix-docker branch 5 times, most recently from 3d3739f to b9967a9 Compare September 25, 2024 11:31
docker/scripts/v2/deploy-docker.sh Outdated Show resolved Hide resolved
docker/DockerTests.test.ts Outdated Show resolved Hide resolved
- Move docker tests logic to a separate script
- Add initializer checks at docker tests
- Refactor yml gha
with:
platforms: linux/amd64,linux/arm64
push: true
tags: hermeznetwork/geth-zkevm-contracts:1.5-integration
Copy link
Collaborator

Choose a reason for hiding this comment

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

jsut double checking the tag, is this correct?¿

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

No one is using the image created by the gha. Therefore, we can delete this entire gha

Copy link

sonarcloud bot commented Oct 2, 2024

Copy link
Contributor

@krlosMata krlosMata left a comment

Choose a reason for hiding this comment

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

😸

@invocamanman invocamanman merged commit 273cb68 into develop Oct 7, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants