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

fix multichain-testing flake #10915

Merged
merged 4 commits into from
Jan 30, 2025
Merged

fix multichain-testing flake #10915

merged 4 commits into from
Jan 30, 2025

Conversation

turadg
Copy link
Member

@turadg turadg commented Jan 30, 2025

Refs: #9934

Description

Patches over the sequence mismatch in Override Chain Registry by retrying.

Doesn't yet solve the wallet provisioning error but I think the above is worth landing. This does add logging to help diagnose the wallet one.

I'll clean up commits before merge.

Security Considerations

none

Scaling Considerations

none

Documentation Considerations

none

Testing Considerations

none

Upgrade Considerations

none

Copy link

cloudflare-workers-and-pages bot commented Jan 30, 2025

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2268352
Status: ✅  Deploy successful!
Preview URL: https://d32dbdec.agoric-sdk.pages.dev
Branch Preview URL: https://9934-multichain-flake.agoric-sdk.pages.dev

View logs

@turadg turadg added the force:integration Force integration tests to run on PR label Jan 30, 2025
@turadg turadg marked this pull request as ready for review January 30, 2025 03:36
@turadg turadg requested a review from a team as a code owner January 30, 2025 03:36
@turadg turadg requested review from AgoricTriage, dckc and samsiegart and removed request for AgoricTriage January 30, 2025 03:36
Copy link
Contributor

@samsiegart samsiegart left a comment

Choose a reason for hiding this comment

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

The error in CI is what I see when I run locally as well. Approving so we can get the logs in at least.

@turadg turadg force-pushed the 9934-multichain-flake branch from 7b90909 to 2268352 Compare January 30, 2025 15:06
@turadg turadg added the automerge:rebase Automatically rebase updates, then merge label Jan 30, 2025
@mergify mergify bot merged commit 7f4df76 into master Jan 30, 2025
87 checks passed
@mergify mergify bot deleted the 9934-multichain-flake branch January 30, 2025 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge:rebase Automatically rebase updates, then merge force:integration Force integration tests to run on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants