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

[e2e] Add wallet balance logging to staking rewards test #3426

Merged
merged 1 commit into from
Sep 27, 2024

Conversation

marun
Copy link
Contributor

@marun marun commented Sep 27, 2024

Why this should be merged

Recent flakes suggest funds exhaustion but reproduction is difficult without knowing what the balances are.

How this works

  • Refactors balance retrieval from the antithesis test for reuse
  • Updates staking rewards tests to output balances at each step

How this was tested

  • CI against regression
  • Locally verified expected output

Recent flakes suggest funds exhaustion but reproduction is difficult
without knowing what the balances are.
@marun marun added the testing This primarily focuses on testing label Sep 27, 2024
@marun marun self-assigned this Sep 27, 2024
@StephenButtolph StephenButtolph added this pull request to the merge queue Sep 27, 2024
@StephenButtolph StephenButtolph added this to the v1.11.12 milestone Sep 27, 2024
Merged via the queue into master with commit 89cd240 Sep 27, 2024
22 checks passed
@StephenButtolph StephenButtolph deleted the e2e-staking-rewards-balance-logging branch September 27, 2024 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing This primarily focuses on testing
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants