feat: add safe daemon rollout controls - #480
Open
alongubkin wants to merge 1 commit into
Open
Conversation
Member
Author
|
Related ALIEN-565 PRs:
Suggested review and merge order: Horizon, Alien, then Platform. |
Greptile SummaryThe PR adds deployment-level CLI controls for guarded or parallel daemon updates and extends provider daemon heartbeats with optional rollout progress.
Confidence Score: 5/5The PR appears safe to merge with no concrete blocking or independently actionable non-blocking issue identified. The CLI request shapes match the nullable Platform API contract, explicit validation limits guarded mode to the supported value, and the heartbeat additions remain backward compatible through optional serde fields.
|
| Filename | Overview |
|---|---|
| crates/alien-cli/src/commands/deployments.rs | Adds parsing, validation, deployment resolution, request serialization, output handling, and tests for guarded and parallel daemon-update modes. |
| crates/alien-cli/src/commands/deploy.rs | Updates deployment-request construction to match the regenerated Platform Rust API input. |
| crates/alien-core/src/heartbeat.rs | Adds backward-compatible optional rollout data and typed rollout states to provider daemon heartbeats. |
| crates/alien-core/src/bin/schema_exporter.rs | Registers the new daemon rollout types for exported OpenAPI and schema artifacts. |
| client-sdks/platform/openapi.json | Updates the Platform API contract with the nullable daemon-update policy and rollout-related API shapes. |
Sequence Diagram
sequenceDiagram
participant U as Operator
participant C as Alien CLI
participant P as Platform API
participant D as Deployment
participant H as Daemon heartbeat producer
U->>C: daemon-updates DEPLOYMENT --max-unavailable 1 or --parallel
C->>P: Resolve deployment
C->>P: PATCH daemon update settings
P->>D: Persist rollout policy
H-->>P: Heartbeat with optional rollout progress
P-->>U: Updated setting and observable rollout state
Reviews (1): Last reviewed commit: "feat: add safe daemon rollout controls" | Re-trigger Greptile
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
CLI
Validation
Part of ALIEN-565.