Skip to content

split grpc server into static and dynamic server#317

Merged
raphael-goetz merged 5 commits into#311-new-module-endpointfrom
#299-split-server
Apr 30, 2026
Merged

split grpc server into static and dynamic server#317
raphael-goetz merged 5 commits into#311-new-module-endpointfrom
#299-split-server

Conversation

@raphael-goetz
Copy link
Copy Markdown
Member

Resolves: #299 & #310

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 30, 2026

GitLab Pipeline Action

General information

Link to pipeline: https://gitlab.com/code0-tech/development/aquila/-/pipelines/2491579692

Status: Passed
Duration: 1 minutes

Job summaries

docs:preview

Documentation preview available at https://code0-tech.gitlab.io/-/development/telescopium/-/jobs/14163781268/artifacts/out/index.html

@raphael-goetz raphael-goetz marked this pull request as ready for review April 30, 2026 13:56
Copilot AI review requested due to automatic review settings April 30, 2026 13:56
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Splits Aquila’s gRPC server startup into distinct static vs dynamic mode servers, and updates dynamic mode to initiate the Sagittarius execution (test) stream to align with the latest tucana interface.

Changes:

  • Introduces AquilaStaticServer and AquilaDynamicServer, moving mode-specific server bootstrapping into static_mode and dynamic_mode.
  • Adds initialization/reconnect loop for the Sagittarius test execution stream in dynamic mode.
  • Improves logging and token-safe observability (masked tokens), plus refactors readiness interception into a reusable interceptor helper.

Reviewed changes

Copilot reviewed 19 out of 19 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/startup/static_mode.rs Static-mode startup now loads fallback flows and starts a static-only gRPC server.
src/startup/mod.rs Startup now selects static vs dynamic mode and passes shared NATS/KV dependencies into each mode.
src/startup/dynamic_mode.rs Dynamic-mode startup now creates the dynamic gRPC server and spawns Sagittarius flow + test execution stream tasks.
src/server/static_server.rs New static gRPC server implementation (action transfer + optional health).
src/server/dynamic_server.rs New dynamic gRPC server implementation (module/runtime services + action transfer + optional health).
src/server/mod.rs Removes monolithic server; exposes dynamic_server, static_server, and shared interceptor.
src/server/interceptor.rs New shared readiness interceptor helper used by both server types.
src/server/action_transfer_service_server_impl.rs Logging improvements and token masking for action transfer stream.
src/server/module_service_server_impl.rs Adds token-masked auth rejection logs and clearer debug logging.
src/server/runtime_status_service_server_impl.rs Adds token-masked auth rejection logs and clearer debug logging.
src/server/runtime_usage_service_server_impl.rs Adds token-masked auth rejection logs and clearer debug logging.
src/sagittarius/test_execution_client_impl.rs Refactors execution client construction to accept a Channel and adds auth metadata to streaming request.
src/sagittarius/retry.rs Adds structured logging around Sagittarius dial/retry attempts.
src/sagittarius/module_service_client_impl.rs Adds structured logging (module counts, rejection reason).
src/sagittarius/runtime_status_service_client_impl.rs Adds structured logging and reduces severity for “unsuccessful” responses.
src/sagittarius/runtime_usage_client_impl.rs Adds structured logging (sample counts) and reduces severity for “unsuccessful” responses.
src/main.rs Adjusts logging defaults and adds config debug logging.
src/configuration/mod.rs Reorders module declarations (no functional impact).
src/authorization/mod.rs Adds mask_token and improves auth header parsing logs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/server/dynamic_server.rs Outdated
Comment thread src/startup/dynamic_mode.rs
Comment thread src/startup/mod.rs
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Raphael Götz <52959657+raphael-goetz@users.noreply.github.com>
@raphael-goetz raphael-goetz merged commit 850d14c into #311-new-module-endpoint Apr 30, 2026
2 checks passed
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.

Aquila needs to adjust to latest tucana interface for test executions. Split Aquila Grpc into Dynamic & Static Server

2 participants