refactor(node): move building the p2p sync server to its own function

[matan-starkware]

This change is 

[matan-starkware]

• refactor(node): remove test code from production code #876 : 2 dependent PRs (#877 , #880 )
• refactor(node): move main to its own file and keep run infrastructure reusable #874
• refactor(node): split run from main, as it will be used by many binaries #873
• refactor(node): introduce PapyrusTaskHandles to hold task overrides #872
• refactor(node): refactor how monitoring server is spawned #871
• refactor(node): spawned tasks should all return anyhow result #870
• refactor(node): miscellaneous clippy suggestions #867
• refactor(node): create papyrus resources for generating top level tasks #865
• refactor(node): network channel creation is done by users #863
• refactor(node): move building the p2p sync server to its own function #862 👈
• refactor(node): move building the sync client to its own function #861
• refactor(node): convert storage metrics handle to the desired task type #860
• main

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

Join @matan-starkware and the rest of your teammates on Graphite

[codecov]

Codecov Report

Attention: Patch coverage is 66.66667% with 4 lines in your changes missing coverage. Please review.

Project coverage is 74.50%. Comparing base (158f318) to head (9d6e8be).
Report is 4 commits behind head on main.

Files with missing lines	Patch %	Lines
crates/papyrus_node/src/main.rs	66.66%	3 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #862      +/-   ##
==========================================
+ Coverage   74.42%   74.50%   +0.07%     
==========================================
  Files         366      366              
  Lines       38550    38559       +9     
  Branches    38550    38559       +9     
==========================================
+ Hits        28691    28728      +37     
+ Misses       7534     7505      -29     
- Partials     2325     2326       +1     

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

[dan-starkware]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @matan-starkware)

[matan-starkware]

Merge activity

• Sep 18, 10:24 AM EDT: @matan-starkware started a stack merge that includes this pull request via Graphite.
• Sep 18, 10:25 AM EDT: @matan-starkware merged this pull request with Graphite.