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

8453 release2 zoe #8802

Merged
merged 4 commits into from
Feb 8, 2024
Merged

8453 release2 zoe #8802

merged 4 commits into from
Feb 8, 2024

Conversation

Chris-Hibbert
Copy link
Contributor

refs: #8453

Description

Add to the release branch the code to allow Zoe to survive upgrade by using watchedPromises. (#8453)

This is a repeat of #8792 to recreate the tmp-upgrade-15 branch without merging PRs.

These are the same commits as on master.

Security Considerations

N/A

Scaling Considerations

N/A

Documentation Considerations

N/A

Testing Considerations

See #8453

Upgrade Considerations

see above.

@Chris-Hibbert Chris-Hibbert added the Zoe package: Zoe label Jan 23, 2024
@Chris-Hibbert Chris-Hibbert self-assigned this Jan 23, 2024
@mergify mergify bot mentioned this pull request Jan 23, 2024
@mhofman mhofman changed the base branch from tmp-upgrade-15 to mhofman/fix-up-upgrade-14 January 31, 2024 22:45
Base automatically changed from mhofman/fix-up-upgrade-14 to dev-upgrade-14 February 7, 2024 03:11
use watchPromise() to wait for contract finish; This repairs new
contracts, but doesn't help existing contracts.
There was a bug in the part of the test that was attempting to run a
bunch of scenarios across the upgrade. Rather than resuming each run
after the upgrade, it was running the whole scenario from scratch
after the upgrade.
@mhofman mhofman added the force:integration Force integration tests to run on PR label Feb 8, 2024
@mhofman mhofman merged commit c2ea6db into dev-upgrade-14 Feb 8, 2024
71 checks passed
@mhofman mhofman deleted the 8453-release2-Zoe branch February 8, 2024 01:04
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 Zoe package: Zoe
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants