diff --git a/contrib/images/Makefile b/contrib/images/Makefile index c0afef21b..beef16916 100644 --- a/contrib/images/Makefile +++ b/contrib/images/Makefile @@ -1,6 +1,6 @@ RELAYER_TAG := $(shell grep '^ENV RELAYER_TAG' cosmos-relayer/Dockerfile | cut -f3 -d\ ) BABYLON_FULL_PATH := $(shell git rev-parse --show-toplevel) -BABYLON_VERSION_BEFORE_UPGRADE ?= 5f8af8ced17d24f3f0c6172293cd37fb3d055807 +BABYLON_VERSION_BEFORE_UPGRADE ?= v0.9.0 all: babylond cosmos-relayer diff --git a/contrib/images/babylond/Dockerfile b/contrib/images/babylond/Dockerfile index 50f7857b8..344877e0b 100644 --- a/contrib/images/babylond/Dockerfile +++ b/contrib/images/babylond/Dockerfile @@ -20,7 +20,7 @@ RUN go mod download COPY ./ /go/src/github.com/babylonlabs-io/babylon/ # If version is set, then checkout this version RUN if [ -n "${VERSION}" ]; then \ - git fetch origin ${VERSION}; \ + git fetch --all; \ git checkout -f ${VERSION}; \ fi diff --git a/test/e2e/configurer/factory.go b/test/e2e/configurer/factory.go index 985216cdd..df41dc9a2 100644 --- a/test/e2e/configurer/factory.go +++ b/test/e2e/configurer/factory.go @@ -114,7 +114,8 @@ var ( // TODO currently only one configuration is available. Consider testing upgrades // when necessary func NewBTCTimestampingConfigurer(t *testing.T, isDebugLogEnabled bool) (Configurer, error) { - containerManager, err := containers.NewManager(isDebugLogEnabled, false, false) + identifier := t.Name() + containerManager, err := containers.NewManager(identifier, isDebugLogEnabled, false, false) if err != nil { return nil, err } @@ -130,7 +131,8 @@ func NewBTCTimestampingConfigurer(t *testing.T, isDebugLogEnabled bool) (Configu } func NewIBCTransferConfigurer(t *testing.T, isDebugLogEnabled bool) (Configurer, error) { - containerManager, err := containers.NewManager(isDebugLogEnabled, false, false) + identifier := t.Name() + containerManager, err := containers.NewManager(identifier, isDebugLogEnabled, false, false) if err != nil { return nil, err } @@ -147,7 +149,8 @@ func NewIBCTransferConfigurer(t *testing.T, isDebugLogEnabled bool) (Configurer, // NewBTCTimestampingPhase2Configurer returns a new Configurer for BTC timestamping service (phase 2). func NewBTCTimestampingPhase2Configurer(t *testing.T, isDebugLogEnabled bool) (Configurer, error) { - containerManager, err := containers.NewManager(isDebugLogEnabled, false, false) + identifier := t.Name() + containerManager, err := containers.NewManager(identifier, isDebugLogEnabled, false, false) if err != nil { return nil, err } @@ -164,7 +167,8 @@ func NewBTCTimestampingPhase2Configurer(t *testing.T, isDebugLogEnabled bool) (C // NewBTCTimestampingPhase2RlyConfigurer returns a new Configurer for BTC timestamping service (phase 2), using the Go relayer (rly). func NewBTCTimestampingPhase2RlyConfigurer(t *testing.T, isDebugLogEnabled bool) (Configurer, error) { - containerManager, err := containers.NewManager(isDebugLogEnabled, true, false) + identifier := t.Name() + containerManager, err := containers.NewManager(identifier, isDebugLogEnabled, true, false) if err != nil { return nil, err } @@ -181,7 +185,8 @@ func NewBTCTimestampingPhase2RlyConfigurer(t *testing.T, isDebugLogEnabled bool) // NewBTCStakingConfigurer returns a new Configurer for BTC staking service func NewBTCStakingConfigurer(t *testing.T, isDebugLogEnabled bool) (Configurer, error) { - containerManager, err := containers.NewManager(isDebugLogEnabled, false, false) + identifier := t.Name() + containerManager, err := containers.NewManager(identifier, isDebugLogEnabled, false, false) if err != nil { return nil, err } @@ -198,7 +203,8 @@ func NewBTCStakingConfigurer(t *testing.T, isDebugLogEnabled bool) (Configurer, // NewSoftwareUpgradeConfigurer returns a new Configurer for Software Upgrade testing func NewSoftwareUpgradeConfigurer(t *testing.T, isDebugLogEnabled bool, upgradePath string) (Configurer, error) { - containerManager, err := containers.NewManager(isDebugLogEnabled, false, true) + identifier := t.Name() + containerManager, err := containers.NewManager(identifier, isDebugLogEnabled, false, true) if err != nil { return nil, err } diff --git a/test/e2e/containers/containers.go b/test/e2e/containers/containers.go index 2de3bc413..88f55566d 100644 --- a/test/e2e/containers/containers.go +++ b/test/e2e/containers/containers.go @@ -36,15 +36,17 @@ type Manager struct { network *dockertest.Network resources map[string]*dockertest.Resource isDebugLogEnabled bool + identifier string } // NewManager creates a new Manager instance and initializes // all Docker specific utilities. Returns an error if initialization fails. -func NewManager(isDebugLogEnabled bool, isCosmosRelayer, isUpgrade bool) (docker *Manager, err error) { +func NewManager(identifier string, isDebugLogEnabled bool, isCosmosRelayer, isUpgrade bool) (docker *Manager, err error) { docker = &Manager{ ImageConfig: NewImageConfig(isCosmosRelayer, isUpgrade), resources: make(map[string]*dockertest.Resource), isDebugLogEnabled: isDebugLogEnabled, + identifier: identifier, } docker.pool, err = dockertest.NewPool("") if err != nil { @@ -380,3 +382,8 @@ func (m *Manager) RunChainInitResource(chainId string, chainVotingPeriod, chainE } return initResource, nil } + +// NetworkName returns the network name concatenated with the identifier name +func (m *Manager) NetworkName() string { + return fmt.Sprintf("bbn-testnet-%s", m.identifier) +}