-
Notifications
You must be signed in to change notification settings - Fork 608
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
test(e2e): upgrade e2e init image for Stride migration #4396
Conversation
@@ -1121,10 +1121,6 @@ func (s *IntegrationTestSuite) TestGeometricTWAP() { | |||
// This test is to be re-enabled for upgrade once the upgrade handler logic is added and | |||
// the balancer pool genesis is backported to v14. | |||
func (s *IntegrationTestSuite) TestStridePoolMigration() { | |||
if !s.skipUpgrade { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did we remove this logic again?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We disabled this previously because the v14 genesis was not updated. As a result, this test would fail when run with upgrade testing enabled.
Now, that we updated the v14 genesis generation via the osmolabs/osmosis-e2e-init-chain
script, this test can be enabled with the upgrade
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
* test(e2e): upgrade e2e init image for Stride migration * update tag (cherry picked from commit 00a264b)
* test(e2e): upgrade e2e init image for Stride migration * update tag (cherry picked from commit 00a264b) Co-authored-by: Roman <[email protected]>
Closes: #XXX
What is the purpose of the change
Updating previous (V14) e2e chain init Docker image to apply genesis changes from #4393 needed for #4384