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

Build bridge which gets RUN Proto economic notification, exposes via Prom end points #4587

Closed
Tartuffo opened this issue Feb 17, 2022 · 3 comments
Assignees
Labels
Inter-protocol Overarching Inter Protocol telemetry vaults_triage DO NOT USE

Comments

@Tartuffo
Copy link
Contributor

Per our Observability meeting on Feb 17, here are areas of RUN Protocol to instrument:

  1. Contracts can log data using notifiers
    a) There’s concern that allowing contracts to log using the same mechanism would allow them to spam the system.
    a) AMM
    b) All currencies and ratios
    c) rade volume/velocity
  2. Vault Manager
    a) Number of vaults
    b) Outstanding debt, collateral held
    c) How many are within epsilon of default? (maybe exposed by blockchain explorer?)
  3. getRUN
    a) Net amount minted
    b) Total collateral
    c) Distribution of utilization of loan amount
@Tartuffo Tartuffo added Inter-protocol Overarching Inter Protocol telemetry labels Feb 17, 2022
@Tartuffo
Copy link
Contributor Author

This can be done after primary code completion, but before going live. E.g. when the code is out there as a RC.

@Tartuffo Tartuffo added this to the Mainnet 1 milestone Mar 23, 2022
@arirubinstein arirubinstein self-assigned this Mar 29, 2022
@turadg turadg changed the title Telemetry for RUN Proto Database-recorded telemetry for RUN Proto Mar 30, 2022
@turadg
Copy link
Member

turadg commented Mar 30, 2022

This ticket has been scoped for econ operations. If there's an security/stability operations, please make a new ticket or point to a better ticket for the econ aspects.

In Run Proto meeting today @btulloh identified these requirements that the Econ committee needs on launch:

  • Total RUN minted ever
  • Total RUN burned ever
  • Per debt limit (contracts authorized to mint RUN), the limit and RUN outstanding to debt limits
  • Counts in reserve of PCV tokens

@Tartuffo Tartuffo changed the title Database-recorded telemetry for RUN Proto Instrument RUN Protocol to use new on-chain notifications for telemetry data May 3, 2022
@Tartuffo Tartuffo changed the title Instrument RUN Protocol to use new on-chain notifications for telemetry data Build bridge which gets RUN Proto economic notification, exposes via Prom end points May 3, 2022
@Tartuffo Tartuffo removed this from the Mainnet 1 RC0 milestone Aug 10, 2022
@ivanlei ivanlei added the vaults_triage DO NOT USE label Jan 17, 2023
@ivanlei
Copy link
Contributor

ivanlei commented Jan 23, 2023

This has been "lost to the sands of time" and is quite out of date. Closing this in favor of the slightly newer #5854

@ivanlei ivanlei closed this as not planned Won't fix, can't repro, duplicate, stale Jan 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Inter-protocol Overarching Inter Protocol telemetry vaults_triage DO NOT USE
Projects
None yet
Development

No branches or pull requests

4 participants