Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

Remove wallclock throttle from BankingStage tests #16396

Merged
merged 1 commit into from
Apr 6, 2021

Conversation

carllin
Copy link
Contributor

@carllin carllin commented Apr 6, 2021

Problem

BankingStage tests potentially are flaky due to the wallclock throttle

Summary of Changes

Remove wallclock throttle from tests

Fixes #

@carllin carllin requested a review from sakridge April 6, 2021 01:01
@codecov
Copy link

codecov bot commented Apr 6, 2021

Codecov Report

Merging #16396 (ed90eee) into master (9360e57) will increase coverage by 0.0%.
The diff coverage is 100.0%.

@@           Coverage Diff           @@
##           master   #16396   +/-   ##
=======================================
  Coverage    80.1%    80.1%           
=======================================
  Files         413      413           
  Lines      110427   110432    +5     
=======================================
+ Hits        88546    88553    +7     
+ Misses      21881    21879    -2     

@carllin carllin merged commit 1219842 into solana-labs:master Apr 6, 2021
mergify bot pushed a commit that referenced this pull request Apr 6, 2021
(cherry picked from commit 1219842)

# Conflicts:
#	core/src/banking_stage.rs
mergify bot pushed a commit that referenced this pull request Apr 6, 2021
(cherry picked from commit 1219842)
carllin added a commit that referenced this pull request Apr 6, 2021
(cherry picked from commit 1219842)
carllin added a commit that referenced this pull request Apr 6, 2021
(cherry picked from commit 1219842)

# Conflicts:
#	core/src/banking_stage.rs
mergify bot added a commit that referenced this pull request Apr 7, 2021
(cherry picked from commit 1219842)

Co-authored-by: carllin <carl@solana.com>
mergify bot added a commit that referenced this pull request Apr 7, 2021
* No wallclock throttle tests (#16396)

(cherry picked from commit 1219842)

# Conflicts:
#	core/src/banking_stage.rs

* Resolve conflicts

Co-authored-by: carllin <carl@solana.com>
@brooksprumo brooksprumo mentioned this pull request Aug 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant