chore(starknet_sequencer_node): set monitoring as active component

[Itay-Tsabary-Starkware]

Stack:

• chore(starknet_sequencer_node): cleanup explicit default config fns #2605
• chore(starknet_sequencer_node): set mempool p2p runner as active component #2601
• chore(starknet_sequencer_node): add validation function to active com… #2600
• chore(starknet_sequencer_node): set consensus manager as active component #2599
• chore(starknet_sequencer_node): set http server as active component #2578
• chore(starknet_sequencer_node): add remote client to active component config #2577
• chore(starknet_sequencer_node): set monitoring as active component #2576 ⬅

⚠️ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

[reviewable-StarkWare]

This change is 

[github-actions]

Artifacts upload workflows:

• Started at 2024-12-09T13:06:05Z
• Started at 2024-12-09T13:06:05Z
• Started at 2024-12-09T13:06:05Z

[codecov]

Codecov Report

Attention: Patch coverage is 70.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 38.08%. Comparing base (e3165c4) to head (8342984).
Report is 796 commits behind head on main.

Files with missing lines	Patch %	Lines
...tes/starknet_integration_tests/src/config_utils.rs	0.00%	2 Missing ⚠️
crates/starknet_sequencer_node/src/components.rs	80.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2576      +/-   ##
==========================================
- Coverage   40.10%   38.08%   -2.02%     
==========================================
  Files          26      278     +252     
  Lines        1895    32141   +30246     
  Branches     1895    32141   +30246     
==========================================
+ Hits          760    12242   +11482     
- Misses       1100    18868   +17768     
- Partials       35     1031     +996     

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

[nadin-Starkware]

Reviewed 4 of 5 files at r1, 1 of 1 files at r3, all commit messages.
Reviewable status: 5 of 6 files reviewed, all discussions resolved

[nadin-Starkware]

Reviewed 1 of 2 files at r2.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @Itay-Tsabary-Starkware)

[nadin-Starkware]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @Itay-Tsabary-Starkware)

---

crates/starknet_sequencer_node/src/config/component_config.rs line 33 at r3 (raw file):

    // Reactive component configs.
    pub monitoring_endpoint: ActiveComponentExecutionConfig,

What do you think about using two fields here: active_components, which would be a struct containing all active components, and reactive_components, another struct? This would be similar to the servers struct, which has three fields: local, remote, and wrapper.

Code quote:

    // Reactive component configs.
    #[validate]
    pub batcher: ReactiveComponentExecutionConfig,
    #[validate]
    pub consensus_manager: ReactiveComponentExecutionConfig,
    #[validate]
    pub gateway: ReactiveComponentExecutionConfig,
    #[validate]
    pub http_server: ReactiveComponentExecutionConfig,
    #[validate]
    pub mempool: ReactiveComponentExecutionConfig,
    #[validate]
    pub mempool_p2p: ReactiveComponentExecutionConfig,
    #[validate]
    pub state_sync: ReactiveComponentExecutionConfig,

    // Reactive component configs.
    pub monitoring_endpoint: ActiveComponentExecutionConfig,

[Itay-Tsabary-Starkware]

Reviewed 4 of 5 files at r1, 1 of 2 files at r2, 1 of 1 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @Itay-Tsabary-Starkware)

[Itay-Tsabary-Starkware]

✓ Commit merged in pull request #2600