aligned_layer icon indicating copy to clipboard operation
aligned_layer copied to clipboard

chore: add multisig support to upgrade ServiceManager

Open JuArce opened this issue 1 year ago • 1 comments

This PR

  • Slightly modifies the upgrade script to only generate the calldata to send to the multisig wallet
  • Writes a guide on how to upgrade the contracts using the multisig

To test

[!Note] Steps on how to deploy and upgrade are available here docs/3_guides/setup_holesky.md Steps on how to use the multisig are available here docs/multisig.md

  • Deploy Aligned contracts con sepolia
    • For this, don't forget to change the contracts/scripts/.env variables:
RPC_URL=<a sepolia one>
EXISTING_DEPLOYMENT_INFO_PATH=./script/output/sepolia/eigenlayer_deployment_output.json
DEPLOY_CONFIG_PATH=./script/deploy/config/sepolia/aligned.sepolia.config.json
OUTPUT_PATH=./script/output/sepolia/alignedlayer_deployment_output.json
MULTISIG=true
  • Test the upgrade works only with the multisig, following the guide

JuArce avatar Jun 27 '24 22:06 JuArce

Changes to gas cost

Generated at commit: 9de2e892b0b879b86a6e6299a8bd717e5e2d04c8, compared to commit: 192a535aa39bbad8cada32b9460bf59e33081685

🧾 Summary (10% most significant diffs)

Contract Method Avg (+/-) %
AlignedLayerServiceManager batchesState
receive
+44 ❌
+2,138 ❌
+6.46%
+4.76%

Full diff report 👇
Contract Deployment Cost (+/-) Method Min (+/-) % Avg (+/-) % Median (+/-) % Max (+/-) % # Calls (+/-)
AlignedLayerServiceManager 5,335,578 (+564,365) batchesState
createNewTask
disableVerifier
enableVerifier
isVerifierDisabled
receive
setDisabledVerifiers
725 (+44)
56,085 (+2,150)
23,911 (+22)
23,803 (-67)
583 (+44)
23,318 (+2,149)
23,801 (+44)
+6.46%
+3.99%
+0.09%
-0.28%
+8.16%
+10.15%
+0.19%
725 (+44)
76,161 (+2,195)
35,504 (+22)
24,447 (-67)
1,249 (+44)
47,015 (+2,138)
34,808 (+44)
+6.46%
+2.97%
+0.06%
-0.27%
+3.65%
+4.76%
+0.13%
725 (+44)
76,249 (+2,198)
35,504 (+22)
24,447 (-67)
583 (+44)
47,202 (+2,138)
34,808 (+44)
+6.46%
+2.97%
+0.06%
-0.27%
+8.16%
+4.74%
+0.13%
725 (+44)
77,095 (+2,225)
47,097 (+22)
25,092 (-67)
2,583 (+44)
47,202 (+2,138)
45,815 (+44)
+6.46%
+2.97%
+0.05%
-0.27%
+1.73%
+4.74%
+0.10%
256 (0)
256 (0)
2 (0)
2 (0)
3 (0)
256 (0)
2 (0)
StrategyManagerMock 1,270,846 (-12)
IndexRegistry 1,086,925 (-12)
BLSApkRegistryHarness 1,837,601 (-12)

github-actions[bot] avatar Oct 07 '24 15:10 github-actions[bot]