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: implement docker e2e ci job #222

Merged
merged 17 commits into from
Jan 18, 2024
Merged

add: implement docker e2e ci job #222

merged 17 commits into from
Jan 18, 2024

Conversation

marcello33
Copy link
Contributor

@marcello33 marcello33 commented Jan 18, 2024

Description

Implement the docker e2e ci job the same way it is done in bor and heimdall, to make sure the docker setup always work as expected. This also ensures matic-cli changes do not affect bor and heimdall CIs.
The CI now tests:

  • express-cli in a remote environment on AWS
  • matic-cli in a dockerized environment locally

The PR also merges a dependabot PR (see #219)

Changes

  • Bugfix (non-breaking change that solves an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (change that is not backwards-compatible and/or changes current functionality)
  • New test case for remote devnet

Checklist

  • I have added at least 2 reviewer or the whole pos-v1 team
  • I have added sufficient documentation in code
  • I will be resolving comments - if any - by pushing each fix in a separate commit and linking the commit hash in the comment reply

Testing

  • I have tested this code manually on local environment
  • I have tested this code manually on remote devnet using express-cli
  • I have tested this code manually on mumbai

@marcello33 marcello33 changed the title add: implement docker e2e ci / parallelize CIs add: implement docker e2e ci job Jan 18, 2024
@marcello33 marcello33 requested a review from a team January 18, 2024 09:04
Copy link
Member

@Raneet10 Raneet10 left a comment

Choose a reason for hiding this comment

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

Minor comments, else LGTM

@marcello33
Copy link
Contributor Author

Minor comments, else LGTM

Thanks for the review @Raneet10
Applied some changes. Will wait for the CI to complete.

@marcello33 marcello33 requested a review from a team January 18, 2024 09:52
@marcello33
Copy link
Contributor Author

marcello33 commented Jan 18, 2024

@Raneet10 had to push this due to latest changes.
Let's see if this makes the e2e docker tests pass

@marcello33 marcello33 force-pushed the mardizzone/docker-ci branch from a1cca76 to 8ca74f2 Compare January 18, 2024 12:22
@marcello33 marcello33 merged commit f971c88 into master Jan 18, 2024
6 checks passed
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.

4 participants