Skip to content

fix(cli): complete hosted setup refresh handoff - #534

Merged
alongubkin merged 1 commit into
mainfrom
alon/alien-613-refresh-cli-setup
Aug 30, 2026
Merged

fix(cli): complete hosted setup refresh handoff#534
alongubkin merged 1 commit into
mainfrom
alon/alien-613-refresh-cli-setup

Conversation

@alongubkin

Copy link
Copy Markdown
Member

Summary

  • fall back to the current release when refreshing setup for a running deployment
  • persist the applied setup revision at the normal provisioning handoff
  • avoid marking no-work or failed setup runs as applied

Testing

  • cargo test -p alien-deploy-cli --lib setup_

@greptile-apps

greptile-apps Bot commented Aug 30, 2026

Copy link
Copy Markdown

Greptile Summary

The PR completes hosted setup-refresh handoff behavior by using the current release when no desired release exists and recording the setup package revision once its setup-owned resources have been applied.

  • Prefers the desired release while falling back to the deployment's current release for setup refreshes.
  • Records the direct setup revision on successful completion or manager handoff.
  • Excludes no-work and failed loop outcomes from revision persistence.
  • Adds focused regression tests for release selection and applied-revision classification.

Confidence Score: 5/5

The PR appears safe to merge with no actionable correctness or security issues identified.

The current-release fallback feeds the existing InitialSetup refresh path, and revision persistence occurs only after setup-owned resources have completed or synchronized; no-work and failure outcomes remain excluded.

Important Files Changed

Filename Overview
crates/alien-deploy-cli/src/commands/up.rs Adds current-release fallback and accurately persists the setup revision only after successful synchronization or completion of the direct-setup handoff.

Sequence Diagram

sequenceDiagram
    participant CLI
    participant Manager
    participant Setup as Direct setup
    CLI->>Manager: Load deployment and setup revision
    CLI->>CLI: Select desired release or current release
    CLI->>Setup: Apply setup-owned Frozen resources
    Setup-->>CLI: Resources Running
    CLI->>Manager: Handoff at Provisioning
    CLI->>Manager: Persist direct setup revision
    Manager->>Manager: Continue Live-resource provisioning
Loading

Reviews (1): Last reviewed commit: "fix(cli): persist completed setup refres..." | Re-trigger Greptile

@alongubkin
alongubkin merged commit dbc28b9 into main Aug 30, 2026
23 of 24 checks passed
@alongubkin
alongubkin deleted the alon/alien-613-refresh-cli-setup branch August 30, 2026 12: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