Merge pull request #140 from e-money/138-upgrade-test

Fix upgrade bdd-test and docker image

networks/docker/cosmovisor/Dockerfile

@@ -4,4 +4,10 @@
 
 FROM golang:1.16-buster AS build-env
 
-RUN go install github.com/cosmos/cosmos-sdk/cosmovisor/cmd/cosmovisor@latest
+# after the SDK upgrade to 44.x+ we should change to @latest
+# Note ref https://github.com/cosmos/cosmos-sdk/tree/master/cosmovisor#cosmosvisor
+# If validators would like to enable the auto-download option, and they are
+# currently running an application using Cosmos SDK v0.42, they will need to use
+# Cosmovisor v0.1. Later versions of Cosmovisor do not support Cosmos SDK v0.42
+# or earlier if the auto-download option is enabled.
+RUN go install github.com/cosmos/cosmos-sdk/cosmovisor/cmd/[email protected]

networktest/testnet.go

@@ -277,8 +277,11 @@ func IncChain(delta int64) (int64, error) {
 
 	return WaitForHeightWithTimeout(
 		height+delta,
-		// is 10 seconds always enough?
-		time.Duration(delta)*10*time.Second,
+		// max seconds allowance for reaching the desired height
+		// increased again to accommodate an observed backup operation during the upgrade
+		// on catalina with docker
+		// `time taken to complete the backup: 442.204684ms12:00PM INF starting ABCI with Tendermint`
+		time.Duration(delta)*16*time.Second,
 	)
 }