Skip to content

Revert "TPU: Replace governor crate with TokenBucket (#8154)"#8627

Merged
apfitzge merged 2 commits intoanza-xyz:masterfrom
apfitzge:revert_8154
Oct 22, 2025
Merged

Revert "TPU: Replace governor crate with TokenBucket (#8154)"#8627
apfitzge merged 2 commits intoanza-xyz:masterfrom
apfitzge:revert_8154

Conversation

@apfitzge
Copy link
Copy Markdown

This reverts commit 589d58b.

Problem

  • This commit is making CI fail on test_run_test_load_program_accounts_partition_root taking an extraordinarily long period of time

Summary of Changes

  • revert the problem commit

Fixes #

Copy link
Copy Markdown

@steviez steviez left a comment

Choose a reason for hiding this comment

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

:shipit:

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 89.88764% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.2%. Comparing base (dd2a230) to head (6e03864).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##           master    #8627     +/-   ##
=========================================
- Coverage    83.2%    83.2%   -0.1%     
=========================================
  Files         851      851             
  Lines      370529   370581     +52     
=========================================
+ Hits       308485   308517     +32     
- Misses      62044    62064     +20     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@apfitzge apfitzge added this pull request to the merge queue Oct 22, 2025
@apfitzge
Copy link
Copy Markdown
Author

Some additional context on failures here: https://discord.com/channels/428295358100013066/560503042458517505/1430675288638885988

@apfitzge
Copy link
Copy Markdown
Author

@alexpyattaev for awareness on this revert

Merged via the queue into anza-xyz:master with commit a06cca6 Oct 22, 2025
54 checks passed
@apfitzge apfitzge deleted the revert_8154 branch October 22, 2025 23:07
rustopian pushed a commit to rustopian/agave that referenced this pull request Nov 20, 2025
…anza-xyz#8627)

* Revert "TPU: Replace governor crate with TokenBucket (anza-xyz#8154)"

This reverts commit 589d58b.

* lock update for unclean revert
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.

3 participants