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

gen-upgrade-proposal.sh variable name spelling is inconsistent with forum posts #9270

Closed
gibson042 opened this issue Apr 20, 2024 · 0 comments · Fixed by #9271
Closed

gen-upgrade-proposal.sh variable name spelling is inconsistent with forum posts #9270

gibson042 opened this issue Apr 20, 2024 · 0 comments · Fixed by #9271
Labels
deployment Chain deployment mechanism (e.g. testnet) enhancement New feature or request

Comments

@gibson042
Copy link
Member

What is the Problem Being Solved?

Compare scripts/gen-upgrade-proposal.sh to e.g. [Proposal #71] agoric-upgrade-14

 COMMIT_ID=$(git rev-parse HEAD)
-ZIPURL=https://github.com/Agoric/agoric-sdk/archive/$COMMIT_ID.zip
 
-echo "Verifying archive is at $ZIPURL..." 1>&2
-zipfile=$(mktemp)
-trap 'rm -f "$zipfile"' EXIT
-curl -L "$ZIPURL" -o "$zipfile"
+ZIP_URL="https://github.com/Agoric/agoric-sdk/archive/${COMMIT_ID}.zip"
 
-echo "Generating SHA-256 checksum..." 1>&2
-checksum=sha256:$(shasum -a 256 "$zipfile" | cut -d' ' -f1)
+CHECKSUM=sha256:$(curl -L "$ZIP_URL" -o- | shasum -a 256 | cut -d' ' -f1)
 
-info="{\"binaries\":{\"any\":\"$ZIPURL//agoric-sdk-$COMMIT_ID?checksum=$checksum\"},\"source\":\"$ZIPURL?checksum=$checksum\"}"
+UPGRADE_INFO="{\"binaries\":{\"any\":\"$ZIP_URL//agoric-sdk-${COMMIT_ID}?checksum=$CHECKSUM\"},\"source\":\"$ZIP_URL?checksum=$CHECKSUM\"}"
+top-node-on-err = {{ .Streaming.ABCI.StopNodeOnErr }}

Description of the Design

Since we're demonstrably comfortable with sharing the forum post representation, adopt its patterns into gen-upgrade-proposal.sh.

Security Considerations

None.

Scaling Considerations

n/a

Test Plan

This code is well-exercised.

Upgrade Considerations

Functionality must not change.

@gibson042 gibson042 added enhancement New feature or request deployment Chain deployment mechanism (e.g. testnet) labels Apr 20, 2024
@mergify mergify bot closed this as completed in #9271 May 1, 2024
mergify bot added a commit that referenced this issue May 1, 2024
…lease posts (#9271)

Fixes #9270

## Description

Adopt variable names and interpolation patterns as seen in community
forum posts such as cf.
https://community.agoric.com/t/proposal-71-agoric-upgrade-14-interchain-stack-smart-wallet-upgrades/633#creating-an-emerynet-upgrade-proposal-for-rc1-3
. Also provides slightly more guidance regarding expected transaction
options and where to look for past proposals.

### Security Considerations

n/a

### Scaling Considerations

n/a

### Documentation Considerations

We might want to direct operators to this script, rather than
duplicating its contents.

### Testing Considerations

This script is well-exercised.

### Upgrade Considerations

Script functionality must not change.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployment Chain deployment mechanism (e.g. testnet) enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant