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 that Zoe support seats and offers through upgrade. #8502

Closed
Chris-Hibbert opened this issue Nov 6, 2023 · 3 comments · Fixed by Agoric/agoric-3-proposals#18
Closed
Assignees
Labels
enhancement New feature or request Zoe package: Zoe

Comments

@Chris-Hibbert
Copy link
Contributor

What is the Problem Being Solved?

#8453 is ABD. The main thing that's missing at this point is a test that demonstrates that it works in the upgrade environment.

Description of the Design

A test that starts from (a copy of) mainNet. It should ensure there's a contract with outstanding offers or seats. I think vaults would work, or it could use coveredCall, atomicSwap, or a purpose-built contract in order to have additional instrumentation. After upgrading, the outstanding offers & seats should still be alive, the contract should still be responsive, and the offers should be exercisable.

Security Considerations

We're concerned about offer-safety.

Scaling Considerations

Not a big issue here.

Test Plan

See above.

Upgrade Considerations

Ditto.

@turadg
Copy link
Member

turadg commented Nov 9, 2023

Agoric/agoric-3-proposals#18 was just progress. (I neglected to remove the "closes:" before merging)

Still to do,

  1. Incorporate more of @dckc 's feedback from zoe1 proposal agoric-3-proposals#18
  2. add a test of restarting vat-admin

The SDK PR tests head against head. It shows that the bug with promise watchers is gone but can't test the "repair" because it doesn't have the earlier state to repair.

So this test needs to:

  1. Make an open offer (e.g. bid)
  2. Core-eval to restart vat-admin vat
  3. Make sure the offer stays open (demonstrating repair)

@turadg turadg reopened this Nov 9, 2023
@ivanlei ivanlei assigned Chris-Hibbert and unassigned turadg Dec 4, 2023
@Chris-Hibbert
Copy link
Contributor Author

Blocked on [A3P#74])Agoric/agoric-3-proposals#74).

@Chris-Hibbert
Copy link
Contributor Author

This is sufficiently tested in A3P.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Zoe package: Zoe
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants