Skip to content

Deployment: BalancerContractRegistryInitializer V2#286

Merged
jubeira merged 5 commits intomasterfrom
deplyment/balancer-registry-initializer-v2
Jul 11, 2025
Merged

Deployment: BalancerContractRegistryInitializer V2#286
jubeira merged 5 commits intomasterfrom
deplyment/balancer-registry-initializer-v2

Conversation

@jubeira
Copy link
Contributor

@jubeira jubeira commented Jul 11, 2025

Deployment: BalancerContractRegistryInitializer V2

Deploying only to the missing networks where the initializer was not deployed yet.
These networks need the registry to deploy LBP v2, so for the sake of simplicity I've removed LBPs from the input to prevent circular dependencies. It can be added later on; the bulk of it is in there.

Deployment tasks checklist:

  • Contract(s) deployed to all target networks / testnets
  • TX IDs are properly generated
  • N/A Action IDs are generated after deployment (if applicable)
  • If a governance proposal is required, Action IDs have been sent to the Maxis
  • Outputs for all target networks are generated, and linked in the task readme
  • Addresses files are up to date with the new deployment
  • Contracts are verified in every network
  • Deployments CHANGELOG is updated
  • There are no code changes of any kind

Issue Resolution

N/A

@jubeira jubeira requested a review from EndymionJkb July 11, 2025 20:59
@EndymionJkb EndymionJkb changed the title Deplyment: BalancerContractRegistryInitializer V2 Deployment: BalancerContractRegistryInitializer V2 Jul 11, 2025
EndymionJkb
EndymionJkb previously approved these changes Jul 11, 2025
Copy link
Collaborator

@EndymionJkb EndymionJkb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. We'll have to tell governance to add LBPs manually to every new chain, then. (And update to v2 on any networks still pointing to v1; they should already have permissions.)

Not in the diff, but should also remove from the fork test, so that it passes:

info = await registry.getBalancerContractInfo(lbpFactory.address);
_validateInfo(info);

@jubeira jubeira requested a review from EndymionJkb July 11, 2025 22:47
Copy link
Collaborator

@EndymionJkb EndymionJkb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@jubeira jubeira merged commit 48cb2fc into master Jul 11, 2025
46 checks passed
@jubeira jubeira deleted the deplyment/balancer-registry-initializer-v2 branch July 11, 2025 23:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants