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(fast-usdc): update, test chain policies vs. spreadsheet #11055

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

Conversation

dckc
Copy link
Member

@dckc dckc commented Feb 26, 2025

Description

Data from a requirements-gathering spreadsheet are turned into a test that shows that the MAINNET chain policies needed adjustment.

  • chore(fast-usdc): update, test chain policies vs. spreadsheet
  • feat(fast-usdc): update FeedPolicy along with Noble ICA
    • refactor(fast-usdc): export feedPolicies.MAINNET in vat code
    • docs(fast-usdc): FeedPolicy is no longer JSON serializable

Security Considerations

The values in a ChainPolicy express a balance of risk tolerance vs. speed etc.

export interface ChainPolicy {

Scaling Considerations

none

Documentation Considerations

  • End users will likely only see a condensed version of these values ("ETA: 1 min").
  • The values impact risks taken by liquidity providers.
  • BLD stakers have an interested in the balance between speed and risk tolerance.
  • for node operators: docs(fast-usdc): DON'T PANIC re "CORE_EVAL failed"

Testing Considerations

  • test(boot): snapshot GTM Fast USDC feed policy
  • test(a3p): fastUsdc.feedPolicy updated for GTM
    • test(a3p): gtm-fast-usdc

Upgrade Considerations

should be deployed as part of the fix for #11013

Copy link

cloudflare-workers-and-pages bot commented Feb 26, 2025

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: 4cab2bf
Status: ✅  Deploy successful!
Preview URL: https://a0c22c5a.agoric-sdk.pages.dev
Branch Preview URL: https://dc-fu-policy-check.agoric-sdk.pages.dev

View logs

@dckc dckc force-pushed the dc-fu-policy-check branch from ab8149d to 4cab2bf Compare February 28, 2025 18:06
@dckc dckc marked this pull request as ready for review February 28, 2025 18:12
@dckc dckc requested a review from a team as a code owner February 28, 2025 18:12
@dckc dckc requested a review from AgoricTriage February 28, 2025 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants