Skip to content

Commit

Permalink
feat(cosmos): Next Upgrade (patch) is agoric-upgrade-18a (#10946)
Browse files Browse the repository at this point in the history
<!-- < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < ☺
v                               ✰  Thanks for creating a PR! ✰
☺ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >  -->

<!-- Integration testing generally doesn't run until a PR is labeled for
merge, but can be opted into for every push by adding label
'force:integration', and can be customized to use non-default external
targets by including lines here that **start** with leading-`#`
directives:
* (https://github.com/Agoric/documentation) #documentation-branch:
$BRANCH_NAME
* (https://github.com/endojs/endo) #endo-branch: $BRANCH_NAME
* (https://github.com/Agoric/dapp-offer-up) #getting-started-branch:
$BRANCH_NAME
* (https://github.com/Agoric/testnet-load-generator) #loadgen-branch:
$BRANCH_NAME

These directives should be removed before adding a merge label, so final
integration tests run against default targets.
-->

<!-- Most PRs should close a specific issue. All PRs should at least
reference one or more issues. Edit and/or delete the following lines as
appropriate (note: you don't need both `refs` and `closes` for the same
one): -->

## Description
<!-- Add a description of the changes that this PR introduces and the
files that are the most critical to review. -->
Uses `use-upgrade-18` a3p image as base for `agoricSyntheticChain`.
Removes a:upgrade-18 proposal as a result.
Renames handlers in the cosmos app to match 18a and removes all
upgrade-18 core proposals (and dependent code).
Removes all variants (not needed) and cherry-picks the startship helm
chart URL fix to make CI pass.

### Security Considerations
<!-- Does this change introduce new assumptions or dependencies that, if
violated, could introduce security vulnerabilities? How does this PR
change the boundaries between mutually-suspicious components? What new
authorities are introduced by this change, perhaps by new API calls? -->
None

### Scaling Considerations
<!-- Does this change require or encourage significant increase in
consumption of CPU cycles, RAM, on-chain storage, message exchanges, or
other scarce resources? If so, can that be prevented or mitigated? -->
None

### Documentation Considerations
<!-- Give our docs folks some hints about what needs to be described to
downstream users. Backwards compatibility: what happens to existing data
or deployments when this code is shipped? Do we need to instruct users
to do something to upgrade their saved data? If there is no upgrade path
possible, how bad will that be for users? -->
With release notes

### Testing Considerations
<!-- Every PR should of course come with tests of its own functionality.
What additional tests are still needed beyond those unit tests? How does
this affect CI, other test automation, or the testnet? -->
Nothing new.

### Upgrade Considerations
<!-- What aspects of this PR are relevant to upgrading live production
systems, and how should they be addressed? What steps should be followed
to verify that its changes have been included in a release
(ollinet/emerynet/mainnet/etc.) and work successfully there? If the
process is elaborate, consider adding a script to scripts/verification/.
-->
Patch Chain Software Upgrade
  • Loading branch information
mujahidkay authored Feb 6, 2025
2 parents bd8fd54 + 3c9b280 commit eaeaf5a
Show file tree
Hide file tree
Showing 34 changed files with 55 additions and 7,291 deletions.
1 change: 1 addition & 0 deletions .github/workflows/multichain-e2e-template.yml
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,7 @@ jobs:
version: 0.2.20
timeout: 30m
namespace: ${{ env.NAMESPACE }}
repo: https://hyperweb-io.github.io/starship

- name: Fund Provision Pool
run: |
Expand Down
2 changes: 1 addition & 1 deletion a3p-integration/package.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"private": true,
"agoricSyntheticChain": {
"fromTag": "use-upgrade-17"
"fromTag": "use-upgrade-18"
},
"scripts": {
"build": "yarn run build:sdk && yarn run build:submissions && yarn run build:synthetic-chain",
Expand Down
1 change: 0 additions & 1 deletion a3p-integration/proposals/a:upgrade-18/.gitignore

This file was deleted.

This file was deleted.

This file was deleted.

19 changes: 0 additions & 19 deletions a3p-integration/proposals/a:upgrade-18/acceptInvites.js

This file was deleted.

20 changes: 0 additions & 20 deletions a3p-integration/proposals/a:upgrade-18/addGov4.js

This file was deleted.

207 changes: 0 additions & 207 deletions a3p-integration/proposals/a:upgrade-18/agd-tools.js

This file was deleted.

Loading

0 comments on commit eaeaf5a

Please sign in to comment.