Skip to content

chore: dynamic ServerState with vec#1997

Merged
Mirko-von-Leipzig merged 5 commits intonextfrom
santiagopittella-dynamic-server-state
Apr 27, 2026
Merged

chore: dynamic ServerState with vec#1997
Mirko-von-Leipzig merged 5 commits intonextfrom
santiagopittella-dynamic-server-state

Conversation

@SantiagoPittella
Copy link
Copy Markdown
Collaborator

@SantiagoPittella SantiagoPittella commented Apr 23, 2026

closes #1909

Collapses the per-service fields on ServerState into a single Vec<watch::Receiver<ServiceStatus>>, so get_status() becomes a trivial map().collect() and adding a new service is a one-line append in start.rs.
The prover pair is handled by a small combiner task that watches the status and test channels and publishes a single merged ServiceStatus.

This changes go in line with the next change: adding a Service trait to standardize the interface #1932

@SantiagoPittella SantiagoPittella added the no changelog This PR does not require an entry in the `CHANGELOG.md` file label Apr 23, 2026
Comment thread bin/network-monitor/src/commands/start.rs Outdated
Comment thread bin/network-monitor/src/commands/start.rs Outdated
SantiagoPittella and others added 2 commits April 27, 2026 12:54
Co-authored-by: Mirko <48352201+Mirko-von-Leipzig@users.noreply.github.com>
Comment thread bin/network-monitor/src/commands/start.rs Outdated
SantiagoPittella and others added 2 commits April 27, 2026 13:17
Co-authored-by: Mirko <48352201+Mirko-von-Leipzig@users.noreply.github.com>
@Mirko-von-Leipzig Mirko-von-Leipzig merged commit 0670698 into next Apr 27, 2026
18 checks passed
@Mirko-von-Leipzig Mirko-von-Leipzig deleted the santiagopittella-dynamic-server-state branch April 27, 2026 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog This PR does not require an entry in the `CHANGELOG.md` file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Monitor: Dynamic ServerState with Vec<Receiver>

2 participants