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

feat(pd): add --ready-to-start flag to pd migrate #4499

Merged
merged 1 commit into from
May 30, 2024
Merged

Conversation

conorsch
Copy link
Contributor

@conorsch conorsch commented May 29, 2024

Describe your changes

Adds a new flag to pd migrate --ready-to-start that makes an in-place edit to local state, setting an enabled halt bit to false.

Refs #4494.

Issue ticket number and link

Checklist before requesting a review

  • If this code contains consensus-breaking changes, I have added the "consensus-breaking" label. Otherwise, I declare my belief that there are not consensus-breaking changes, for the following reason:

    This code directly munges local state, via a migration, and therefore deserves careful scrutiny.

Adds a new flag to `pd migrate --ready-to-start` that makes an in-place
edit to local state, setting an enabled halt bit to false.

Refs #4494.
@conorsch
Copy link
Contributor Author

Tested this on a local devnet, and it appears to work well: if the adjacent cometbft instance also sets --p2p.pex false, then the node remains at the pre-upgrade height, and I can query the CometBFT RPC without problems.

@conorsch conorsch marked this pull request as ready for review May 29, 2024 21:32
@conorsch conorsch requested a review from avahowell May 29, 2024 21:42
@conorsch conorsch merged commit 47dbb9a into main May 30, 2024
16 checks passed
@conorsch conorsch deleted the 4494-ready-to-start branch May 30, 2024 15:13
conorsch added a commit that referenced this pull request Jun 12, 2024
Collects some information from the cuiloa README [0], as well as some
generalized instructions captured in #4566, particularly the use of
`--ready-to-start` from #4499. Refs #4494, closes #4566.

[0] https://github.com/penumbra-zone/cuiloa/blob/dc4133f7b36706cdf5a3ee6b4e0fb2c09e5a8bb8/README.md
conorsch added a commit that referenced this pull request Jun 13, 2024
Collects some information from the cuiloa README [0], as well as some
generalized instructions captured in #4566, particularly the use of
`--ready-to-start` from #4499. Refs #4494, closes #4566.

[0] https://github.com/penumbra-zone/cuiloa/blob/dc4133f7b36706cdf5a3ee6b4e0fb2c09e5a8bb8/README.md
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.

4 participants