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: provision wallet in PREPARE instead of USE #10461

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

frazarshad
Copy link
Contributor

@frazarshad frazarshad commented Nov 13, 2024

Description

In a3p tests we added a new gov4 wallet and we provision it after the chain upgrade has executed. We also do this in emerynet, but it has been provisioned before the chain upgrade.
Since the proposal needs to send an invite to gov4, it waits for the wallet to be provisioned, then executes the latter part of the proposal, this waiting does not happen in emerynet and that is what causes the problem.

This PR fixes this problem by moving the provisioning step from USE stage to PREPARE stage, thereby provisioning it before the execution of the proposal

Security Considerations

Scaling Considerations

Documentation Considerations

Testing Considerations

Upgrade Considerations

@frazarshad frazarshad requested a review from a team as a code owner November 13, 2024 11:27
Copy link

cloudflare-workers-and-pages bot commented Nov 13, 2024

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: c365c94
Status: ✅  Deploy successful!
Preview URL: https://f7403ee8.agoric-sdk.pages.dev
Branch Preview URL: https://fraz-fix-a3p-provision-fail.agoric-sdk.pages.dev

View logs

@frazarshad frazarshad added the force:integration Force integration tests to run on PR label Nov 13, 2024
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.

1 participant