Skip to content

feat: add tool to get initialization counter and update that in contr…#245

Merged
wjmelements merged 2 commits intoFilOzone:mainfrom
DarkLord017:initialization
Jan 30, 2026
Merged

feat: add tool to get initialization counter and update that in contr…#245
wjmelements merged 2 commits intoFilOzone:mainfrom
DarkLord017:initialization

Conversation

@DarkLord017
Copy link
Contributor

Fixes #244

introduces a new upgrade pattern for the PDPVerifier and SimplePDPService contracts by adding a reinitializer version to be set via the constructor.

All deployment scripts and tests have been updated to pass the required constructor argument.

@DarkLord017
Copy link
Contributor Author

Please review @wjmelements

Also i noticed that some bash scripts in tools like deploy-transfer-ownership-upgrade-calibnet.sh and upgrade-contract.sh do not cast send announced plan upgrade txn before updating the implimentation

@rjan90 rjan90 moved this to 🔎 Awaiting review in PDP Jan 29, 2026
@rjan90 rjan90 added this to the M4.5: GA Fast Follows milestone Jan 29, 2026
@wjmelements
Copy link
Contributor

Will need a follow-on PR to filecoin-service to pull this submodule and update the scripts there that deploy the PDPVerifier implementation (such as tools/deploy-all-warm-storage.sh).

@wjmelements wjmelements merged commit 3b7e172 into FilOzone:main Jan 30, 2026
3 checks passed
@github-project-automation github-project-automation bot moved this from 🔎 Awaiting review to 🎉 Done in PDP Jan 30, 2026
@DarkLord017
Copy link
Contributor Author

Will need a follow-on PR to filecoin-service to pull this submodule and update the scripts there that deploy the PDPVerifier implementation (such as tools/deploy-all-warm-storage.sh).

Okay will do this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

immutable reinitializer parameter

3 participants