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

chore: uprade 18 cherrypicks round 5 #10676

Merged
merged 5 commits into from
Dec 13, 2024
Merged

Conversation

mujahidkay
Copy link
Member

@mujahidkay mujahidkay commented Dec 11, 2024

Description

Cherry-picks the following commits from master:

No new upgrade name has been added.

Done partially via git cherry-pick and via the following rebase-todo:

# PR #10680 Branch Record-instances-that-will-be-replaced-so-we-can-manage-them-10680-
label base-Record-instances-that-will-be-replaced-so-we-can-manage-them-10680-
pick c883c39bbe feat: record instances that will be replaced so we can manage them
pick 1581127d32 refactor: provideRetiredInstances
label pr-10680--Record-instances-that-will-be-replaced-so-we-can-manage-them-10680-
reset base-Record-instances-that-will-be-replaced-so-we-can-manage-them-10680-
merge -C d35659b6ab pr-10680--Record-instances-that-will-be-replaced-so-we-can-manage-them-10680- # Record instances that will be replaced so we can manage them (#10680)

mhofman and others added 2 commits December 11, 2024 21:52
refs: #10357

## Description
I did a bad copy paste in #10357, which causes the `run.id` of `timer-poll` triggers to all end up with the value `timer-poll-undefined`. Use the blockHeight instead.

### Security Considerations
None

### Scaling Considerations
None

### Documentation Considerations
None

### Testing Considerations
Not tested

### Upgrade Considerations
Would be nice to pick in u18-rc4
@mujahidkay mujahidkay requested a review from a team as a code owner December 11, 2024 16:58
@mujahidkay mujahidkay added the force:integration Force integration tests to run on PR label Dec 11, 2024
Copy link

cloudflare-workers-and-pages bot commented Dec 11, 2024

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: d632323
Status: ✅  Deploy successful!
Preview URL: https://7698a271.agoric-sdk.pages.dev
Branch Preview URL: https://mk-u18-rc4-cherrypicks.agoric-sdk.pages.dev

View logs

@mujahidkay mujahidkay self-assigned this Dec 11, 2024
@mujahidkay mujahidkay requested a review from mhofman December 11, 2024 16:59
@Chris-Hibbert
Copy link
Contributor

still waiting on #10680.

Chris-Hibbert and others added 3 commits December 13, 2024 22:13
closes: #10669

## Description

Record instances of contracts that will be replaced so we can find them later and be able to manage them.

### Security Considerations

We've lost the handle to some vats before, and it's a hassle.

### Scaling Considerations

The problem gets worse as we upgrade more vats.

### Documentation Considerations

None.

### Testing Considerations

Verified that Auctions and Committee are present. The priceFeeds vary too much across test environments to be worth checking.

### Upgrade Considerations

Yes..
@mujahidkay mujahidkay added force:integration Force integration tests to run on PR and removed force:integration Force integration tests to run on PR labels Dec 13, 2024
Copy link
Contributor

@Chris-Hibbert Chris-Hibbert left a comment

Choose a reason for hiding this comment

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

LGTM

@mujahidkay mujahidkay merged commit 6cab101 into dev-upgrade-18 Dec 13, 2024
90 checks passed
@mujahidkay mujahidkay deleted the mk/u18-rc4-cherrypicks branch December 13, 2024 18:17
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.

4 participants