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

Add 1.8.0 permissioned to permissionless runbook. #388

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

ajsutton
Copy link
Contributor

@ajsutton ajsutton commented Dec 5, 2024

Description

Adds a runbook to upgrade from 1.8.0 permissioned configuration to permissionless configuration.

@ajsutton ajsutton requested review from a team as code owners December 5, 2024 04:35
@ajsutton ajsutton force-pushed the aj/1.8.0-permissionless-runbook branch from ae829a9 to 10f9958 Compare December 5, 2024 04:37
@ajsutton ajsutton force-pushed the aj/1.8.0-permissionless-runbook branch from 10f9958 to 2f99450 Compare December 5, 2024 04:37
runbooks/1.8.0-permissioned-to-permissionless.md Outdated Show resolved Hide resolved
runbooks/1.8.0-permissioned-to-permissionless.md Outdated Show resolved Hide resolved
runbooks/1.8.0-permissioned-to-permissionless.md Outdated Show resolved Hide resolved
runbooks/1.8.0-permissioned-to-permissionless.md Outdated Show resolved Hide resolved
runbooks/1.8.0-permissioned-to-permissionless.md Outdated Show resolved Hide resolved
Inphi
Inphi previously approved these changes Dec 9, 2024
Copy link
Contributor

@Inphi Inphi 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. One step I'm not sure if it should be in this runbook is whether SystemConfig upgrade for Holocene. The Deploy Contracts with Holocene Deployer Tool mentions that the SystemConfig shouldn't be upgrade before Holocene activation, which is correct. Should the chain operator follow up on this after Holocene activates?

@ajsutton
Copy link
Contributor Author

Good call out - I've added a step to the end to run that transaction if upgrading from 1.6.0.

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