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

[E2E - Upgrade Testing] switch e2e test setup to creating genesis and configs via Dockertest #1332

Closed
Tracked by #1338
p0mvn opened this issue Apr 22, 2022 · 0 comments · Fixed by #1363
Closed
Tracked by #1338
Assignees
Labels

Comments

@p0mvn
Copy link
Member

p0mvn commented Apr 22, 2022

Background

This is part of #1235 epic

The preceding tasks include #1293 and #1330. This issue is blocked on these being merged.

Assuming that both are merged, we should be able to initialize genesis and config data via:

docker run -v < path >:/tmp/osmo-test osmosis-e2e-chain-init:debug --data-dir=/tmp/osmo-test
All chain data is placed at the given < path > that is mounted as a volume on the container. In addition, this PR introduces documentation about the current state of the e2e tests.

For the next step, we need to convert func (s *IntegrationTestSuite) configureChain(chainId string) in e2e_setup_test.go to initialize genesis and configs via Dockertest by using the newly created osmosis-e2e-chain-init:debug and mount its output on a volume.

Then, we should mount osmosis:debug containers that run validators on these mounted configs.

Please note that currently the chain id in the script used by the osmosis-e2e-chain-init:debug container is hardcoded. Instead, it should be passed to the container via Dockertest and read from the script as a flag.

Acceptance Criteria

  • chain initialization in e2e tests uses Dockertest and the osmosis-e2e-chain-init:debug container
  • e2e tests continue to pass
@daniel-farina daniel-farina moved this to 🔍 Needs Review in Osmosis Chain Development Apr 22, 2022
@p0mvn p0mvn added blocked T:task ⚙️ A task belongs to a story labels Apr 22, 2022
@p0mvn p0mvn moved this to 🔍 Needs Review in Osmosis Chain Development Apr 22, 2022
@p0mvn p0mvn moved this from 🔍 Needs Review to 🕒 Todo in Osmosis Chain Development Apr 22, 2022
@p0mvn p0mvn moved this from 🕒 Todo to ❌ Blocked in Osmosis Chain Development Apr 22, 2022
@czarcas7ic czarcas7ic moved this from ❌ Blocked to 🏃 In Progress in Osmosis Chain Development Apr 25, 2022
@czarcas7ic czarcas7ic self-assigned this Apr 25, 2022
Repository owner moved this from 🏃 In Progress to ✅ Done in Osmosis Chain Development Apr 28, 2022
@p0mvn p0mvn changed the title test: switch e2e test setup to creating genesis and configs via Dockertest [Upgrade Testing] switch e2e test setup to creating genesis and configs via Dockertest May 4, 2022
@p0mvn p0mvn changed the title [Upgrade Testing] switch e2e test setup to creating genesis and configs via Dockertest [E2E - Upgrade Testing] switch e2e test setup to creating genesis and configs via Dockertest May 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants