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

agd tx submit-proposal upgrade-info must pass new CLI validation code #9048

Closed
aj-agoric opened this issue Mar 7, 2024 · 3 comments
Closed
Assignees
Labels
enhancement New feature or request

Comments

@aj-agoric
Copy link

aj-agoric commented Mar 7, 2024

What is the Problem Being Solved?

OpCo need to support the new validation done in the CLI of software-upgrade gov proposals.

While an issue exists upstream with the parsing of the upgrade-info, it doesn't trigger for upgrade info values that contain valid binaries urls.

However the new cosmos CLI validation verifies the URLs point to valid artifacts.

Description of the Design

Change the upgrade-info generation instructions (and script) to

  • modify upgrade-info.binaries.any to include a //agoric-sdk-$COMMIT_ID suffix between the .zip and the ?checksum. This is a trick for go-getter to unpack the top level folder of the GitHub archive.
  • add a upgrade-info.source to be the previous value of the upgrade-info.binaries.any (without the suffix above), to support 3rd party downloaders which do not understand the go-getter trick.

Security Considerations

None

Scaling Considerations

None

Test Plan

Upgrade Considerations

ability to upgrade to new software

@gibson042
Copy link
Member

PR: agoric-labs/cosmos-sdk#405

@mhofman mhofman changed the title Support Cosmos Change - x/upgrade ParseInfo consider some values as URLs Handle Cosmos Change software-upgrade binaries validation Mar 8, 2024
@mhofman
Copy link
Member

mhofman commented Mar 18, 2024

The forum posts were updated to reflect the above, and for the upgrade-14 branch, we did change the gen script as well (f091b6d). A similar change for the master branch should be implemented, maybe while doing #8784

@gibson042 gibson042 changed the title Handle Cosmos Change software-upgrade binaries validation agd tx submit-proposal upgrade-info must pass new CLI validation code Mar 19, 2024
@gibson042
Copy link
Member

This was fixed for upgrade-14 by 32bff4b#diff-212b769d9a514d2e7813277e574999c609023d1d27de8d7b11244b879a81db3e as part of #9041, and is being tracked for later releases by #9105.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants