Add PSM upgrade for upgrade-19 #10730
Conversation
Deploying agoric-sdk with
|
| Latest commit: |
03c583b
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://d4613d43.agoric-sdk.pages.dev |
| Branch Preview URL: | https://iomekam-psm-upgrade.agoric-sdk.pages.dev |
|
The Upgrade Considerations section is a copy of the one from #10541. Please consider the differences and re-write. |
|
I see that you included the |
| @@ -0,0 +1,80 @@ | |||
| /* eslint-env node */ | |||
| /** | |||
| * @file The goal of this file is to make sure v36-reserve upgraded. | |||
| * 1. Simulate trade of IST and USDC | ||
| * 2. Upgrade all PSMs | ||
| * 3. Verify metrics are the same after the upgrade | ||
| * 4. Verity trading is still possible after the upgrade |
There was a problem hiding this comment.
typo
| * 4. Verity trading is still possible after the upgrade | |
| * 4. Verify trading is still possible after the upgrade |
| }; | ||
| }); | ||
|
|
||
| test.serial('similate trade of IST and USDC', async t => { |
There was a problem hiding this comment.
typo
| test.serial('similate trade of IST and USDC', async t => { | |
| test.serial('simulate trade of IST and USDC', async t => { |
| console.log('[CONTRACT_KITS]', contractKits); | ||
| console.log('[ISSUER]', usdcIssuer); |
There was a problem hiding this comment.
Why print the square brackets?
| } | ||
| } | ||
|
|
||
| console.log('Minting USDC'); |
There was a problem hiding this comment.
This will break if it doesn't find a usdcMint. We should add an assertion so it'll fail deliberately if there are testnets where it's not found?
| console.log(await E(seat).getPayouts()); | ||
| console.log('Done.'); |
There was a problem hiding this comment.
please add a comment here that the success will be checked in the test by reading from vstorage.
closes: #10403
Description
Adds the upgrade of the psm contracts to upgrade 19. To ensure that this upgrade will succeed, we've also added test coverage in A3P.
Upgrade Considerations
This PR adds the PSM contract to the list of vat upgrades. The PSM upgrade proposal will upgrade all PSM instances on the chain. We've added A3P tests to ensure that the upgrade is successful and that assets are still in the reserve after the upgrade.