[Consensus] Standalone startup of hotstuff participant

[durkmurder]

https://github.com/dapperlabs/flow-go/issues/6420

Context

This PR implements standalone startup of hotstuff participant separating startup from compliance engine.

Important changes

• Hotstuff participant decoupled from compliance engine, hotstuff is created before compliance core and is injected using constructor.
• Refactored epoch factories for collection nodes to split construction of sync core and sync engine to allow injection using construction for compliance engine.

[jordanschalm]

Looks great, nice clean up 🧹

[codecov-commenter]

Codecov Report

Merging #3513 (6a6b5eb) into feature/active-pacemaker (75af49b) will decrease coverage by 3.40%.
The diff coverage is n/a.

@@                     Coverage Diff                      @@
##           feature/active-pacemaker    #3513      +/-   ##
============================================================
- Coverage                     55.79%   52.39%   -3.41%     
============================================================
  Files                           769      498     -271     
  Lines                         71228    46130   -25098     
============================================================
- Hits                          39745    24171   -15574     
+ Misses                        28274    19865    -8409     
+ Partials                       3209     2094    -1115     

Flag	Coverage Δ
unittests	52.39% <ø> (-3.41%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
admin/command_runner.go	82.12% <0.00%> (-1.12%)	⬇️
fvm/environment/contract_updater.go	66.88% <0.00%> (-0.66%)	⬇️
ledger/complete/wal/checkpointer.go	63.10% <0.00%> (-0.27%)	⬇️
engine/collection/compliance/core.go
engine/collection/compliance/engine.go
engine/collection/epochmgr/epoch_components.go
engine/consensus/compliance/core.go
engine/consensus/compliance/engine.go
engine/common/rpc/convert/validate.go
consensus/hotstuff/signature/packer.go
... and 264 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[AlexHentschel]

Great cleanup. Only stylistic comments. Thanks

[durkmurder]

bors merge

[bors]

Build succeeded:

• Integration Tests (make -C integration access-tests)
• Integration Tests (make -C integration bft-tests)
• Integration Tests (make -C integration collection-tests)
• Integration Tests (make -C integration consensus-tests)
• Integration Tests (make -C integration epochs-tests)
• Integration Tests (make -C integration execution-tests)
• Integration Tests (make -C integration ghost-tests)
• Integration Tests (make -C integration mvp-tests)
• Integration Tests (make -C integration network-tests)
• Integration Tests (make -C integration verification-tests)
• Lint (./)
• Lint (./crypto/)
• Lint (./integration/)
• Unit Tests (access)
• Unit Tests (admin)
• Unit Tests (cmd)
• Unit Tests (consensus)
• Unit Tests (engine)
• Unit Tests (fvm)
• Unit Tests (ledger)
• Unit Tests (module)
• Unit Tests (network)
• Unit Tests (others)
• Unit Tests (utils)