Skip to content

Added Capio Discovery Service#233

Open
marcoSanti wants to merge 4 commits into
masterfrom
discovery-service
Open

Added Capio Discovery Service#233
marcoSanti wants to merge 4 commits into
masterfrom
discovery-service

Conversation

@marcoSanti
Copy link
Copy Markdown
Member

@marcoSanti marcoSanti commented Apr 26, 2026

Added CAPIO Discovery Service

This PR introduces the CAPIO Discovery Service, enabling CAPIO server instances to detect and communicate with other running server instances in the cluster.

Changes

  • Added a new filesystem-based and multicast-based discovery service (discovery_service.cpp) that allows server instances to register themselves and detect peers
  • Began adding support in the capio backend for explicit connection, by introducing the virtual method connect_to
  • Migrated management of the CAPIO canary flag into the discovery service, centralising server lifecycle logic
  • Updated CLI parsing logic to correctly handle the startup sequence with the new discovery service in place
  • Refactored the shared memory canary (shm_canary.cpp) to work alongside the new service

@marcoSanti marcoSanti force-pushed the discovery-service branch 2 times, most recently from 42369a1 to be32b74 Compare April 26, 2026 11:37
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 26, 2026

Codecov Report

❌ Patch coverage is 9.16667% with 109 lines in your changes missing coverage. Please review.
✅ Project coverage is 61.65%. Comparing base (6b14036) to head (0f64ed8).

Files with missing lines Patch % Lines
capio/server/src/discovery_service.cpp 4.25% 83 Missing and 7 partials ⚠️
capio/server/src/shm_canary.cpp 25.00% 10 Missing and 2 partials ⚠️
capio/server/include/utils/signals.hpp 20.00% 2 Missing and 2 partials ⚠️
capio/server/include/remote/discovery.hpp 0.00% 1 Missing ⚠️
capio/server/src/mpi_backend.cpp 0.00% 1 Missing ⚠️
capio/server/src/none_backend.cpp 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #233      +/-   ##
==========================================
- Coverage   63.43%   61.65%   -1.78%     
==========================================
  Files          79       82       +3     
  Lines        3268     3362      +94     
  Branches     2008     2063      +55     
==========================================
  Hits         2073     2073              
- Misses        397     1025     +628     
+ Partials      798      264     -534     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@marcoSanti marcoSanti force-pushed the discovery-service branch 2 times, most recently from 51f838e to 510dc64 Compare April 27, 2026 09:43
This commit adds the CAPIO discovery service which allows a CAPIO
server instance to detect other running server instances.
It also moves the CAPIO canary flag to be managed by the deiscovery
service, and changes the CLI parsing logic accordingly to manage
correctly the statup process.
@marcoSanti marcoSanti force-pushed the discovery-service branch from 510dc64 to d041e04 Compare May 18, 2026 13:04
@marcoSanti marcoSanti force-pushed the discovery-service branch from d041e04 to 0f64ed8 Compare May 18, 2026 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant