Skip to content

Commit

Permalink
chore(e2e): remove initializer from staking upgrade script (#3113)
Browse files Browse the repository at this point in the history
The Staking upgrade was upgraded before its last round of changes. As
reinitialization is not needed for this upgrade, the initializer call
needs to be removed.

issue: #3031
  • Loading branch information
Zodomo authored Feb 21, 2025
1 parent a410c55 commit 534d3f5
Showing 1 changed file with 11 additions and 7 deletions.
18 changes: 11 additions & 7 deletions e2e/app/admin/upgrade.go
Original file line number Diff line number Diff line change
Expand Up @@ -233,14 +233,18 @@ func ugpradeSlashing(ctx context.Context, s shared, c chain) error {
return nil
}

var stakingABI = mustGetABI(bindings.StakingMetaData)

func upgradeStaking(ctx context.Context, s shared, c chain) error {
// Staking.sol's initializeV2() is called after upgrade
initializer, err := stakingABI.Pack("initializeV2")
if err != nil {
return errors.Wrap(err, "pack initializer")
}
// Uncomment and update following block if re-initialization is required
/*
var stakingABI = mustGetABI(bindings.StakingMetaData)
initializer, err := stakingABI.Pack("initializeV2")
if err != nil {
return errors.Wrap(err, "pack initializer")
}
*/

initializer := []byte{}

calldata, err := adminABI.Pack("upgradeStaking", s.upgrader, s.deployer, initializer)
if err != nil {
Expand Down

0 comments on commit 534d3f5

Please sign in to comment.