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

rc1 verification - reapply upgrade #10484

Draft
wants to merge 21 commits into
base: dev-upgrade-18
Choose a base branch
from

Conversation

mujahidkay
Copy link
Member

Description

Intended to remain draft. Only needed for verification purposes

@mujahidkay mujahidkay added the force:integration Force integration tests to run on PR label Nov 14, 2024
Copy link

cloudflare-workers-and-pages bot commented Nov 14, 2024

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: 006ae98
Status: ✅  Deploy successful!
Preview URL: https://120068e6.agoric-sdk.pages.dev
Branch Preview URL: https://mk-test-upgrade-reapply.agoric-sdk.pages.dev

View logs

Comment on lines +6 to +8
// This query matches both the auction and its governor, so 2*3 for the
// original, the vaults&auctions coreEval, and the priceFeed coreEval.
t.is(Object.keys(details).length, 8);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// This query matches both the auction and its governor, so 2*3 for the
// original, the vaults&auctions coreEval, and the priceFeed coreEval.
t.is(Object.keys(details).length, 8);
// This query matches both the auction and its governor, so 2*4 for the
// original, the vaults&auctions coreEval, upgrade 18 (plus the re-run on emerynet).
t.is(Object.keys(details).length, 8);

mujahidkay added a commit that referenced this pull request Nov 19, 2024
## Description

Cherry-picks the following commits from master:
- #10454
(d16781f)
- #10455
(1aec8d0)
 
Also adds a new upgrade name `agoric-upgrade-18-basic-2`. Also, moves
provisioning GOV4 from USE to PREPARE.

used `git cherry-pick` for the above commits

[Draft PR with reapply upgrade on top of
rc0](#10484)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
force:integration Force integration tests to run on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants