-
Notifications
You must be signed in to change notification settings - Fork 214
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
Conversation
There was a problem hiding this 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.
Correct, and I put it in a
If the mainfork upgrade allows including proposals (via
Okay, I'll add more. I intended the |
Deploying agoric-sdk with Cloudflare Pages
|
…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.
### 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) ```
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.