Skip to content

fix: retry hosted compute updates after failed apply - #531

Merged
alongubkin merged 1 commit into
mainfrom
alon/alien-613-retry-hosted-compute-update
Aug 30, 2026
Merged

fix: retry hosted compute updates after failed apply#531
alongubkin merged 1 commit into
mainfrom
alon/alien-613-retry-hosted-compute-update

Conversation

@alongubkin

Copy link
Copy Markdown
Member

Summary

  • Resubmit the requested compute target for hosted deployments in pending or retryable lifecycle states, even when it equals the persisted desired settings.
  • Keep the equality fast path for fully running deployments.
  • Cover failed, pending, setup, provisioning, and updating lifecycle retries.

Why

Platform persists desired settings before the deployment engine applies them. After a failed apply, the desired target may already match the caller while the cloud still has the previous capacity. Re-submitting the target lets Platform’s idempotent operation queue repair the incomplete update.

@greptile-apps

greptile-apps Bot commented Aug 30, 2026

Copy link
Copy Markdown

Greptile Summary

The PR changes hosted compute updates so equal persisted targets are resubmitted while a deployment is in any supported non-running lifecycle state, allowing interrupted applies to be retried while preserving the equality fast path for running deployments.

  • Adds a lifecycle-aware predicate for deciding whether compute settings must be submitted.
  • Covers active, pending, and retryable hosted deployment states with a focused unit test.

Confidence Score: 5/5

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

The new predicate preserves changed-compute updates for running deployments and resubmits equal targets only for lifecycle states already admitted by the existing hosted-compute status guard.

Important Files Changed

Filename Overview
crates/alien-deploy-cli/src/commands/up.rs Broadens hosted compute submission only within the existing supported-status gate and adds tests for running, in-flight, and failed lifecycle states.

Reviews (1): Last reviewed commit: "fix: retry hosted compute updates after ..." | Re-trigger Greptile

@alongubkin
alongubkin merged commit edf7b32 into main Aug 30, 2026
24 checks passed
@alongubkin
alongubkin deleted the alon/alien-613-retry-hosted-compute-update branch August 30, 2026 08:55
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