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

ci: solo pd node reset on point release #4459

Closed
conorsch opened this issue May 24, 2024 · 2 comments · Fixed by #4551
Closed

ci: solo pd node reset on point release #4459

conorsch opened this issue May 24, 2024 · 2 comments · Fixed by #4551
Labels
A-CI/CD Relates to continuous integration & deployment of Penumbra _P-low Priority: low

Comments

@conorsch
Copy link
Contributor

Notes for later: the point-release CI job doesn't handle the solo pd nodes (#3336) gracefully: https://solo-pd.testnet.plinfra.net/ was reset and now must sync from scratch. Let's circle back and make that nicer.

Originally posted by @conorsch in #4448 (comment)

@conorsch conorsch added the A-CI/CD Relates to continuous integration & deployment of Penumbra label May 24, 2024
@github-actions github-actions bot added the needs-refinement unclear, incomplete, or stub issue that needs work label May 24, 2024
@conorsch conorsch added _P-low Priority: low and removed needs-refinement unclear, incomplete, or stub issue that needs work labels May 24, 2024
@conorsch
Copy link
Contributor Author

conorsch commented Jun 3, 2024

I manually deployed v0.77.1 via a script, to avoid this reset: for now, let's just snip out the erroneously resetting workflow mention, because it's more work to resync from scratch than to upgrade the standalone node manually.

conorsch added a commit that referenced this issue Jun 4, 2024
The solo node, run outside the cluster, was being fully reset on
point releases, due to the aggressive reset logic that was written
during the days of frequent chain resets. We're performing
point-releases more regularly now, so let's make sure to avoid resetting
the solo node in those cases. Still unsupported is chain-migration logic
specifically for the solo node, but that's fine: we're handling chain
upgrades out of band for now anyway.

Closes #4459.
conorsch added a commit that referenced this issue Jun 4, 2024
The solo node, run outside the cluster, was being fully reset on
point releases, due to the aggressive reset logic that was written
during the days of frequent chain resets. We're performing
point-releases more regularly now, so let's make sure to avoid resetting
the solo node in those cases. Still unsupported is chain-migration logic
specifically for the solo node, but that's fine: we're handling chain
upgrades out of band for now anyway.

Closes #4459.
conorsch added a commit that referenced this issue Jun 4, 2024
The solo node, run outside the cluster, was being fully reset on
point releases, due to the aggressive reset logic that was written
during the days of frequent chain resets. We're performing
point-releases more regularly now, so let's make sure to avoid resetting
the solo node in those cases. Still unsupported is chain-migration logic
specifically for the solo node, but that's fine: we're handling chain
upgrades out of band for now anyway.

Closes #4459.

(cherry picked from commit 0b02ec3)
conorsch added a commit that referenced this issue Jun 4, 2024
The solo node, run outside the cluster, was being fully reset on
point releases, due to the aggressive reset logic that was written
during the days of frequent chain resets. We're performing
point-releases more regularly now, so let's make sure to avoid resetting
the solo node in those cases. Still unsupported is chain-migration logic
specifically for the solo node, but that's fine: we're handling chain
upgrades out of band for now anyway.

Closes #4459.

(cherry picked from commit 0b02ec3)
@conorsch
Copy link
Contributor Author

conorsch commented Jun 4, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-CI/CD Relates to continuous integration & deployment of Penumbra _P-low Priority: low
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant