Skip to content

v3.0: bugfix: vote worker reached_end_of_slot set by is_complete (backport of #8474)#8481

Merged
apfitzge merged 2 commits intov3.0from
mergify/bp/v3.0/pr-8474
Oct 14, 2025
Merged

v3.0: bugfix: vote worker reached_end_of_slot set by is_complete (backport of #8474)#8481
apfitzge merged 2 commits intov3.0from
mergify/bp/v3.0/pr-8474

Conversation

@mergify
Copy link
Copy Markdown

@mergify mergify Bot commented Oct 14, 2025

Problem

#7351 comment

Summary of Changes

  • use is_complete() instead of !is_complete
  • separate into function so we can test easily

Fixes #


This is an automatic backport of pull request #8474 done by [Mergify](https://mergify.com).

reached_end_of_slot set by is_complete

(cherry picked from commit 904be9f)

# Conflicts:
#	core/src/banking_stage/vote_worker.rs
@mergify mergify Bot added the conflicts label Oct 14, 2025
@mergify mergify Bot requested a review from a team as a code owner October 14, 2025 19:25
@mergify
Copy link
Copy Markdown
Author

mergify Bot commented Oct 14, 2025

Cherry-pick of 904be9f has failed:

On branch mergify/bp/v3.0/pr-8474
Your branch is up to date with 'origin/v3.0'.

You are currently cherry-picking commit 904be9f96.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   core/src/banking_stage/vote_worker.rs

no changes added to commit (use "git add" and/or "git commit -a")

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@mergify mergify Bot added the conflicts label Oct 14, 2025
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.5%. Comparing base (60b612f) to head (80517e7).
⚠️ Report is 36 commits behind head on v3.0.

Additional details and impacted files
@@           Coverage Diff           @@
##             v3.0    #8481   +/-   ##
=======================================
  Coverage    83.5%    83.5%           
=======================================
  Files         810      810           
  Lines      365988   366008   +20     
=======================================
+ Hits       305607   305626   +19     
- Misses      60381    60382    +1     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown

@t-nelson t-nelson left a comment

Choose a reason for hiding this comment

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

can we get rid of all this fuckin' c in master?

https://github.com/anza-xyz/agave/pull/8481/files#diff-adaec8b3abe01d14df2eca34bcf3ef65f42862cf551f8250211cc9aca7cfb869R266-R276

sry gh too dumb to unfold the context. here

fn process_packets(
&mut self,
bank: &Bank,
consumed_buffered_packets_count: &mut usize,
rebuffered_packet_count: &mut usize,
banking_stage_stats: &BankingStageStats,
slot_metrics_tracker: &mut LeaderSlotMetricsTracker,
) -> bool {

@apfitzge apfitzge merged commit ea24c1f into v3.0 Oct 14, 2025
42 checks passed
@apfitzge apfitzge deleted the mergify/bp/v3.0/pr-8474 branch October 14, 2025 22:07
Japif pushed a commit to helius-labs/agave that referenced this pull request Feb 4, 2026
…kport of anza-xyz#8474) (anza-xyz#8481)

* bugfix: vote worker reached_end_of_slot set by is_complete (anza-xyz#8474)

reached_end_of_slot set by is_complete

(cherry picked from commit 904be9f)

# Conflicts:
#	core/src/banking_stage/vote_worker.rs

* resolve conflicts

---------

Co-authored-by: Andrew Fitzgerald <apfitzge@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants