feat(consensus): sequencer context broadcasts votes

[matan-starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[matan-starkware]

• feat(consensus): sequencer context broadcasts votes #2422 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[github-actions]

Artifacts upload workflows:

• Started at 2024-12-03T09:54:01Z

[codecov]

Codecov Report

Attention: Patch coverage is 85.71429% with 2 lines in your changes missing coverage. Please review.

Project coverage is 77.49%. Comparing base (e3165c4) to head (0aa727f).
Report is 683 commits behind head on main.

Files with missing lines	Patch %	Lines
...us_orchestrator/src/sequencer_consensus_context.rs	50.00%	0 Missing and 2 partials ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #2422       +/-   ##
===========================================
+ Coverage   40.10%   77.49%   +37.38%     
===========================================
  Files          26      387      +361     
  Lines        1895    41109    +39214     
  Branches     1895    41109    +39214     
===========================================
+ Hits          760    31856    +31096     
- Misses       1100     7001     +5901     
- Partials       35     2252     +2217     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[asmaastarkware]

Reviewed 5 of 5 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @dan-starkware and @yair-starkware)

[dan-starkware]

crates/sequencing/papyrus_consensus_orchestrator/src/sequencer_consensus_context.rs line 92 at r1 (raw file):

    pub fn new(
        batcher: Arc<dyn BatcherClient>,
        _proposal_streaming_client: BroadcastTopicClient<ProposalPart>,

How is this still the case?

Code quote:

_proposal_streaming_client

[dan-starkware]

Reviewed 5 of 5 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @matan-starkware and @yair-starkware)

[matan-starkware]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @dan-starkware and @yair-starkware)

---

crates/sequencing/papyrus_consensus_orchestrator/src/sequencer_consensus_context.rs line 92 at r1 (raw file):

Previously, dan-starkware wrote…

How is this still the case?

It's a naming issue, guy is clenaing up

[matan-starkware]

Merge activity

• Dec 4, 2:32 AM EST: A user started a stack merge that includes this pull request via Graphite.
• Dec 4, 2:32 AM EST: A user merged this pull request with Graphite.