From 10c1fe50982b061b59152a7fce3dcb32b20bd9e3 Mon Sep 17 00:00:00 2001 From: Roman Date: Tue, 1 Nov 2022 20:32:33 -0700 Subject: [PATCH] fix(e2e): various e2e build issues breaking debugging on linux amd64 (#3207) * fix(e2e): various e2e build issues breaking debugging on linux amd64 * fix test-e2e-debug --- .dockerignore | 1 - Makefile | 6 +++--- tests/e2e/containers/config.go | 4 ++-- tests/e2e/e2e_test.go | 4 ++-- 4 files changed, 7 insertions(+), 8 deletions(-) diff --git a/.dockerignore b/.dockerignore index a1e4b36e518..ec94acf103e 100644 --- a/.dockerignore +++ b/.dockerignore @@ -4,7 +4,6 @@ docs/ networks/ proto/ scripts/ -tests/ tools/ .github/ .git/ diff --git a/Makefile b/Makefile index 5d4a5261e1d..d9e8102fa18 100644 --- a/Makefile +++ b/Makefile @@ -272,7 +272,7 @@ test-e2e-ci: # test-e2e-debug runs a full e2e test suite but does # not attempt to delete Docker resources at the end. test-e2e-debug: e2e-setup - @VERSION=$(VERSION) OSMOSIS_E2E_UPGRADE_VERSION=$(E2E_UPGRADE_VERSION) OSMOSIS_E2E_SKIP_CLEANUP=True go test -tags e2e -mod=readonly -timeout=25m -v $(PACKAGES_E2E) -count=1 + @VERSION=$(VERSION) OSMOSIS_E2E=True OSMOSIS_E2E_DEBUG_LOG=True OSMOSIS_E2E_UPGRADE_VERSION=$(E2E_UPGRADE_VERSION) OSMOSIS_E2E_SKIP_CLEANUP=True go test -mod=readonly -timeout=25m -v $(PACKAGES_E2E) -count=1 # test-e2e-short runs the e2e test with only short tests. # Does not delete any of the containers after running. @@ -296,10 +296,10 @@ docker-build-debug: @DOCKER_BUILDKIT=1 docker tag osmosis:${COMMIT} osmosis:debug docker-build-e2e-init-chain: - @DOCKER_BUILDKIT=1 docker build -t osmosis-e2e-init-chain:debug --build-arg E2E_SCRIPT_NAME=chain -f tests/e2e/initialization/init.Dockerfile . + @DOCKER_BUILDKIT=1 docker build -t osmosis-e2e-init-chain:debug --build-arg E2E_SCRIPT_NAME=chain --platform=linux/x86_64 -f tests/e2e/initialization/init.Dockerfile . docker-build-e2e-init-node: - @DOCKER_BUILDKIT=1 docker build -t osmosis-e2e-init-node:debug --build-arg E2E_SCRIPT_NAME=node -f tests/e2e/initialization/init.Dockerfile . + @DOCKER_BUILDKIT=1 docker build -t osmosis-e2e-init-node:debug --build-arg E2E_SCRIPT_NAME=node --platform=linux/x86_64 -f tests/e2e/initialization/init.Dockerfile . e2e-setup: e2e-check-image-sha e2e-remove-resources @echo Finished e2e environment setup, ready to start the test diff --git a/tests/e2e/containers/config.go b/tests/e2e/containers/config.go index 0c223477132..03f3503b5ba 100644 --- a/tests/e2e/containers/config.go +++ b/tests/e2e/containers/config.go @@ -24,10 +24,10 @@ const ( // It should be uploaded to Docker Hub. OSMOSIS_E2E_SKIP_UPGRADE should be unset // for this functionality to be used. previousVersionOsmoRepository = "osmolabs/osmosis" - previousVersionOsmoTag = "12.1" + previousVersionOsmoTag = "12.2" // Pre-upgrade repo/tag for osmosis initialization (this should be one version below upgradeVersion) previousVersionInitRepository = "osmolabs/osmosis-e2e-init-chain" - previousVersionInitTag = "v12.1.0-e2e-v1" + previousVersionInitTag = "v12.2.0" // Hermes repo/version for relayer relayerRepository = "osmolabs/hermes" relayerTag = "0.13.0" diff --git a/tests/e2e/e2e_test.go b/tests/e2e/e2e_test.go index 84025c23276..b30584f5e54 100644 --- a/tests/e2e/e2e_test.go +++ b/tests/e2e/e2e_test.go @@ -398,7 +398,7 @@ func (s *IntegrationTestSuite) TestStateSync() { // start the state synchin node. err = stateSynchingNode.Run() - s.NoError(err) + s.Require().NoError(err) // ensure that the state synching node cathes up to the running node. s.Require().Eventually(func() bool { @@ -414,7 +414,7 @@ func (s *IntegrationTestSuite) TestStateSync() { // stop the state synching node. err = chainA.RemoveNode(stateSynchingNode.Name) - s.NoError(err) + s.Require().NoError(err) } func (s *IntegrationTestSuite) TestExpeditedProposals() {