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

test: add proposal w/300 E(chainTimerService).getTimerBrand() calls #10696

Merged
merged 2 commits into from
Dec 16, 2024

Conversation

Chris-Hibbert
Copy link
Contributor

Description

@warner asked for a proposal that would ensure there were several more (300 was settled on) arbitrary calls in order to provoke bring-out-your-dead.

Security Considerations

test environments only: aimed at mainFork.

Scaling Considerations

No implications

Documentation Considerations

Unnecessary

Testing Considerations

For test environments.

Upgrade Considerations

None.

@Chris-Hibbert Chris-Hibbert added force:integration Force integration tests to run on PR next-release about next agoric-sdk or endo release labels Dec 16, 2024
@Chris-Hibbert Chris-Hibbert self-assigned this Dec 16, 2024
@Chris-Hibbert Chris-Hibbert requested a review from a team as a code owner December 16, 2024 16:51
@Chris-Hibbert Chris-Hibbert removed the force:integration Force integration tests to run on PR label Dec 16, 2024
Copy link
Member

@warner warner left a comment

Choose a reason for hiding this comment

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

I'm no expert on our proposal environment, but that looks good to me.

I'm guessing that this is safe to merge onto master because it lacks some glue that would include it in any particular upgrade proposal? And that whoever does the mainfork upgrade step (@toliaqat ?) will need to write that glue (and not check it in)?

It might benefit from a comment at the beginning to warn future readers against including this in a real proposal. The consequences would be minor (just some extra work, no lasting state), but still best to make it clear that this is for a specific testing exercise.

@Chris-Hibbert
Copy link
Contributor Author

I'm guessing that this is safe to merge onto master because it lacks some glue that would include it in any particular upgrade proposal?

Correct, and I put it in a testing directory to indicate it shouldn't be included in MainNet.

And that whoever does the mainfork upgrade step (@toliaqat ?) will need to write that glue (and not check it in)?

If the mainfork upgrade allows including proposals (via cosgov, e.g.), then there's no glue necessary.

It might benefit from a comment at the beginning to warn future readers against including this in a real proposal. The consequences would be minor (just some extra work, no lasting state), but still best to make it clear that this is for a specific testing exercise.

Okay, I'll add more. I intended the @file comment to imply that, but I can add more verbiage.

Copy link

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

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1184dad
Status: ✅  Deploy successful!
Preview URL: https://a66be3b1.agoric-sdk.pages.dev
Branch Preview URL: https://cth-load-boyd.agoric-sdk.pages.dev

View logs

@Chris-Hibbert Chris-Hibbert added the automerge:squash Automatically squash merge label Dec 16, 2024
@mergify mergify bot merged commit 3e27c74 into master Dec 16, 2024
89 checks passed
@mergify mergify bot deleted the cth-load-BOYD branch December 16, 2024 18:50
mujahidkay pushed a commit that referenced this pull request Dec 17, 2024
…10696)

## Description

@warner asked for a proposal that would ensure there were several more (300 was settled on) arbitrary calls in order to provoke bring-out-your-dead.

### Security Considerations

test environments only: aimed at mainFork.

### Scaling Considerations

No implications

### Documentation Considerations

Unnecessary

### Testing Considerations

For test environments.

### Upgrade Considerations

None.
mujahidkay added a commit that referenced this pull request Dec 17, 2024
### Description

Cherry-picks the following commits from master:
- #10696
(3e27c74)
- #10694
(140c1be)
- #10704
(ab3fcb5,
c025cb7)

No new upgrade name has been added.

Using following rebase todo:
```
# PR #10704 Branch build-deps-use-backport-of-cosmos-sdk-v0-46-15-10704-
label onto
pick 140c1be Save the contractKit for the auctioneer before overwriting it (#10694)
pick 3e27c74 test: add proposal w/300 E(chainTimerService).getTimerBrand() calls (#10696)
label base-build-deps-use-backport-of-cosmos-sdk-v0-46-15-10704-
pick ab3fcb5 build(deps): use backport of cosmos-sdk v0.46.15
pick c025cb7 fix(cosmic-swingset): expect chain --halt-height exit status > 1
label pr-10704--build-deps-use-backport-of-cosmos-sdk-v0-46-15-10704-
reset base-build-deps-use-backport-of-cosmos-sdk-v0-46-15-10704- # test: add proposal w/300 E(chainTimerService).getTimerBrand() calls (#10696)
merge -C 5939b2a pr-10704--build-deps-use-backport-of-cosmos-sdk-v0-46-15-10704- # build(deps): use backport of cosmos-sdk v0.46.15 (#10704)
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge:squash Automatically squash merge next-release about next agoric-sdk or endo release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants