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

chore(a3p): setup the oracle and push price for ATOM and stATOM at use phase of f:replace-price-feeds #10296

Merged
merged 3 commits into from
Oct 29, 2024

Conversation

Jorge-Lopes
Copy link
Collaborator

@Jorge-Lopes Jorge-Lopes commented Oct 18, 2024

closes: https://github.com/Agoric/BytePitchPartnerEng/issues/26
refs: https://github.com/Agoric/BytePitchPartnerEng/issues/22

Description

This PR introduces a new script, verifyPushedPrice, which is executed during the use phase of the n:upgrade-next proposal.
The purpose of this script is to set up the oracle and push an initial price for key brands such as Atom and stAtom. This functionality ensures that a price quote is available for the subsequent proposals and acceptance tests.

Security Considerations

Scaling Considerations

Documentation Considerations

Testing Considerations

The existing test file, priceFeedUpdate.test.js, has been updated to account for the new price feed calls made during the use phase, as well as take advantage of the new helper functions built.

Upgrade Considerations

@Jorge-Lopes Jorge-Lopes requested a review from a team as a code owner October 18, 2024 16:21
Copy link

cloudflare-workers-and-pages bot commented Oct 18, 2024

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5a3ad84
Status: ✅  Deploy successful!
Preview URL: https://4c31fb13.agoric-sdk.pages.dev
Branch Preview URL: https://jorge-verify-pushed-price.agoric-sdk.pages.dev

View logs

Copy link
Contributor

@Chris-Hibbert Chris-Hibbert left a comment

Choose a reason for hiding this comment

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

I think this will be unnecessary once #10317 is approved and merged.

@anilhelvaci
Copy link
Collaborator

Hey @Chris-Hibbert , I cannot see where the oracles are registered and a quote is produced durably in #10317. Could you elaborate on how this PR will be unnecessary? I can see that f:replace-price-feeds is removed. Could moving these changes to n:upgrade-next still be an option?

@Chris-Hibbert
Copy link
Contributor

I thought we had already set up a price push in the use phase of n:upgrade-next. If that's missing, then, yes certainly move this over there.

@Jorge-Lopes Jorge-Lopes force-pushed the jorge/verify-pushed-price branch from d59c30e to 573c565 Compare October 28, 2024 20:10
@Jorge-Lopes Jorge-Lopes added the automerge:rebase Automatically rebase updates, then merge label Oct 28, 2024
@Jorge-Lopes Jorge-Lopes force-pushed the jorge/verify-pushed-price branch from 573c565 to 8d45260 Compare October 29, 2024 09:07
@mergify mergify bot merged commit d7c994b into master Oct 29, 2024
80 checks passed
@mergify mergify bot deleted the jorge/verify-pushed-price branch October 29, 2024 10:30
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants