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

Merge jbowen93/evmos into celestiaorg/evmos #32

Merged
merged 104 commits into from
Mar 30, 2022
Merged

Conversation

jbowen93
Copy link

@jbowen93 jbowen93 commented Mar 15, 2022

This update adds docker files to get Cevmos to work with the github.com/celestiaorg/ephemeral-cluster. It includes docker manifests, associated scripts, and github actions to build both regular and debug evmos images (which wrap the binary in dlv).

With commit 958bcce this PR will now use the Celestiaorg/Ethermint v0.11.0+0.1.0 Release.

The commit history for this PR is very messy because I've been using jbowen93/evmos as my development branch for some time. We should consider squash merging this PR to not pollute the upstream repository.

The PR consists of the following high level changes:

  • Update Go modules to use celestiaorg/ethermint v0.11.0+0.1.0 instead of tharsis/ethermint
  • Add docker files that startup an evmos server and a "debug" evmos server which is wrapped by dlv
  • Create a Github Action to create and push the above debug docker image
  • Create a Github Action to create the above non-debug docker image, startup an ephemeral cluster, and run the Ethermint RPC tests against it

@jbowen93 jbowen93 mentioned this pull request Mar 23, 2022
4 tasks
@jbowen93 jbowen93 marked this pull request as ready for review March 29, 2022 01:48
Copy link
Member

@liamsi liamsi left a comment

Choose a reason for hiding this comment

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

utACK

Copy link
Member

@tzdybal tzdybal left a comment

Choose a reason for hiding this comment

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

LGTM

@jbowen93 jbowen93 merged commit 1e2e027 into celestiaorg:main Mar 30, 2022
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