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

[LoadTesting] Test tokenomics actors balances. #921

Draft
wants to merge 78 commits into
base: main
Choose a base branch
from

Conversation

red-0ne
Copy link
Contributor

@red-0ne red-0ne commented Nov 13, 2024

Summary

This PR tests the tokenomics correct tokens distribution by:

  • Observing claim settlements.
  • Maintain a tokenomics actors balances derived from claim settlement.
  • Check that onchain balances are the same as the locally maintained ones.

Issue

Type of change

Select one or more from the following:

Testing

  • Documentation: make docusaurus_start; only needed if you make doc changes
  • Unit Tests: make go_develop_and_test
  • LocalNet E2E Tests: make test_e2e
  • DevNet E2E Tests: Add the devnet-test-e2e label to the PR.

Sanity Checklist

  • I have tested my changes using the available tooling
  • I have commented my code
  • I have performed a self-review of my own code; both comments & source code
  • I create and reference any new tickets, if applicable
  • I have left TODOs throughout the codebase, if applicable

Olshansk and others added 30 commits August 15, 2024 21:39
@red-0ne red-0ne self-assigned this Nov 13, 2024
@red-0ne red-0ne changed the title [LoadTesting [LoadTesting] Test tokenomics actors balances. Nov 13, 2024
@red-0ne red-0ne added tokenomics Token Economics - what else do you need? loadtest Work related to load testing labels Nov 13, 2024
@red-0ne red-0ne added this to the Shannon Beta TestNet Launch milestone Nov 13, 2024
@red-0ne red-0ne changed the base branch from feat/no-overservicing to main November 13, 2024 23:07
Copy link

gitguardian bot commented Nov 13, 2024

⚠️ GitGuardian has uncovered 2 secrets following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secrets in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
12819930 Triggered Generic Password ba13177 localnet/kubernetes/values-pocketdex-postgres.yaml View secret
12819930 Triggered Generic Password 5ad3b28 localnet/kubernetes/observability-prometheus-stack.yaml View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secrets safely. Learn here the best practices.
  3. Revoke and rotate these secrets.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

// account for GMI
mintUPOKT := s.getGlobalInflationMintedUPOKT(settledUPOKT)

// TODO_IN_THIS_PR: Should check with multiple supplier owners

Choose a reason for hiding this comment

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

[linter-name (fail-on-found)] reported by reviewdog 🐶
// TODO_IN_THIS_PR: Should check with multiple supplier owners

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
loadtest Work related to load testing tokenomics Token Economics - what else do you need?
Projects
Status: 🏗 In progress
Development

Successfully merging this pull request may close these issues.

2 participants