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

Standardize on "migrate" terminology for performing upgrades #3578

Closed
conorsch opened this issue Jan 4, 2024 · 1 comment · Fixed by #3631
Closed

Standardize on "migrate" terminology for performing upgrades #3578

conorsch opened this issue Jan 4, 2024 · 1 comment · Fixed by #3631
Assignees
Labels
A-upgrades Area: Relates to chain upgrades

Comments

@conorsch
Copy link
Contributor

conorsch commented Jan 4, 2024

In the context of performing chain upgrades (#1804), we use the word "upgrade" a lot, so much that it's become overloaded and therefore ambiguous. To reduce confusion, let's use "upgrade" to refer to:

  1. the set of operations required to move a node from one version of the chain to another, e.g. penumbra-testnet-rhea to penumbra-testnet-titan chain id;
  2. the act of updating a specific software component from one version to another, e.g. pd from 0.63.3 to 0.64.0.

However, we currently also use the term "upgrade" in the subcommand pd upgrade, which migrates local pd state to a format compatible with the future chain. Instead, let's call that action pd migrate and update documentation accordingly. Doing so will allow us to differentiate in documentation between the act of "upgrading pd" and "migrating state", both of which are necessary subtasks to be performed by operators within the greater whole of "performing a chain upgrade".

@erwanor erwanor moved this to Testnet 65: Deimos in Testnets Jan 17, 2024
@erwanor
Copy link
Member

erwanor commented Jan 17, 2024

SGTM, if we do this we should do it ahead of the 66 upgrade.

@github-project-automation github-project-automation bot moved this to 🗄️ Backlog in Penumbra Jan 19, 2024
@conorsch conorsch moved this from 🗄️ Backlog to In progress in Penumbra Jan 19, 2024
conorsch added a commit that referenced this issue Jan 19, 2024
conorsch added a commit that referenced this issue Jan 19, 2024
conorsch added a commit that referenced this issue Jan 19, 2024
@github-project-automation github-project-automation bot moved this from In progress to Done in Penumbra Jan 19, 2024
@github-project-automation github-project-automation bot moved this from Testnet 65: Deimos to Testnet 63: Rhea (Web Wallet) in Testnets Jan 19, 2024
@erwanor erwanor moved this from Testnet 63: Rhea (Web Wallet) to Testnet 65: Deimos in Testnets Jan 28, 2024
@cratelyn cratelyn added the A-upgrades Area: Relates to chain upgrades label Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-upgrades Area: Relates to chain upgrades
Projects
Archived in project
Status: Testnet 65: Deimos
Development

Successfully merging a pull request may close this issue.

3 participants