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

chore: e2e upgrade refactory #25

Merged
merged 6 commits into from
Aug 20, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
41 changes: 0 additions & 41 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -213,47 +213,6 @@ jobs:
run: |
make test-e2e-cache-btc-staking

e2e-run-upgrade-vanilla:
needs: [e2e-docker-build-babylon, e2e-docker-build-babylon-before-upgrade, e2e-docker-build-e2e-init-chain]
runs-on: ubuntu-22.04
steps:
- name: Checkout repository
uses: actions/checkout@v4
- name: Download babylon artifact
uses: actions/download-artifact@v4
with:
name: babylond-${{ github.sha }}
path: /tmp
- name: Download babylond-before-upgrade artifact
uses: actions/download-artifact@v4
with:
name: babylond-before-upgrade
path: /tmp
- name: Download init-chain artifact
uses: actions/download-artifact@v4
with:
name: init-chain
path: /tmp
- name: Docker load babylond
run: |
docker load < /tmp/docker-babylond.tar.gz

- name: Docker load babylond-before-upgrade
run: |
docker load < /tmp/docker-babylond-before-upgrade.tar.gz

- name: Docker load init chain
run: |
docker load < /tmp/docker-init-chain.tar.gz

- name: Cache Go
uses: actions/setup-go@v5
with:
go-version: 1.21
- name: Run e2e TestSoftwareUpgradeTestSuite
run: |
sudo make test-e2e-cache-upgrade-vanilla

e2e-run-upgrade-signet:
needs: [e2e-docker-build-babylon, e2e-docker-build-babylon-before-upgrade, e2e-docker-build-e2e-init-chain]
runs-on: ubuntu-22.04
Expand Down
3 changes: 0 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -274,9 +274,6 @@ test-e2e-cache-btc-timestamping-phase-2-rly:
test-e2e-cache-btc-staking:
go test -run TestBTCStakingTestSuite -mod=readonly -timeout=60m -v $(PACKAGES_E2E) --tags=e2e

test-e2e-cache-upgrade-vanilla:
go test -run TestSoftwareUpgradeTestSuite -mod=readonly -timeout=60m -v $(PACKAGES_E2E) --tags=e2e

test-e2e-cache-upgrade-signet:
go test -run TestSoftwareUpgradeSignetLaunchTestSuite -mod=readonly -timeout=60m -v $(PACKAGES_E2E) --tags=e2e

Expand Down
5 changes: 3 additions & 2 deletions app/e2e_include_upgrades.go
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,10 @@ package app

import (
"github.com/babylonlabs-io/babylon/app/upgrades/signetlaunch"
"github.com/babylonlabs-io/babylon/app/upgrades/vanilla"
)

// init is used to include signet upgrade used for e2e testing
// this file should be removed once the upgrade testing with signet ends.
func init() {
Upgrades = append(Upgrades, vanilla.Upgrade, signetlaunch.Upgrade)
Upgrades = append(Upgrades, signetlaunch.Upgrade)
}
RafilxTenfen marked this conversation as resolved.
Show resolved Hide resolved
4 changes: 0 additions & 4 deletions app/upgrades/vanilla/README.md

This file was deleted.

97 changes: 0 additions & 97 deletions app/upgrades/vanilla/upgrades.go

This file was deleted.

105 changes: 0 additions & 105 deletions app/upgrades/vanilla/upgrades_test.go

This file was deleted.

2 changes: 1 addition & 1 deletion contrib/images/Makefile
Original file line number Diff line number Diff line change
@@ -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 ?= v0.9.0
BABYLON_VERSION_BEFORE_UPGRADE ?= v0.9.1

all: babylond cosmos-relayer

Expand Down
5 changes: 2 additions & 3 deletions test/e2e/configurer/config/constants.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,11 @@ package config

const (
// PropDepositBlocks estimated number of blocks it takes to deposit for a proposal
PropDepositBlocks float32 = 10
PropDepositBlocks float32 = 6
// PropVoteBlocks number of blocks it takes to vote for a single validator to vote for a proposal
PropVoteBlocks float32 = 1.2
// PropBufferBlocks number of blocks used as a calculation buffer
PropBufferBlocks float32 = 6
PropBufferBlocks float32 = 5

// Upgrades
// ForkHeightPreUpgradeOffset how many blocks we allow for fork to run pre upgrade state creation
Expand All @@ -16,6 +16,5 @@ const (
// PropSubmitBlocks estimated number of blocks it takes to submit for a proposal
PropSubmitBlocks float32 = 1
// Upgrade prop files json
UpgradeVanillaFilePath = "/upgrades/vanilla.json"
UpgradeSignetLaunchFilePath = "/upgrades/signet-launch.json"
)
2 changes: 1 addition & 1 deletion test/e2e/configurer/factory.go
Original file line number Diff line number Diff line change
Expand Up @@ -209,7 +209,7 @@ 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, btcHeaders []*btclighttypes.BTCHeaderInfo) (Configurer, error) {
func NewSoftwareUpgradeConfigurer(t *testing.T, isDebugLogEnabled bool, upgradePath string, btcHeaders []*btclighttypes.BTCHeaderInfo) (*UpgradeConfigurer, error) {
identifier := identifierName(t)
containerManager, err := containers.NewManager(identifier, isDebugLogEnabled, false, true)
if err != nil {
Expand Down
Loading
Loading