Skip to content

fix: allow runtime workloads to link live sandboxes - #695

Merged
alongubkin merged 1 commit into
mainfrom
alon/alien-945-live-sandbox-runtime-links
Sep 24, 2026
Merged

alongubkin merged 1 commit into
mainfrom
alon/alien-945-live-sandbox-runtime-links

Conversation

@alongubkin

Copy link
Copy Markdown
Member

Summary

  • allow runtime-provisioned Containers and Daemons to link an AWS Live Sandbox
  • preserve the setup-time binding rejection for Workers and other setup-rendered consumers
  • keep unsupported-platform lifecycle checks unchanged

Why

A Live Sandbox receives its image ARN and active version only after its runtime controller finishes. The compile-time check rejected every linked consumer on that basis, even though Container and Daemon controllers already wait for dependencies and resolve the completed binding from controller state.

Workers still render linked bindings during setup, so they continue to fail early with an explicit setup-time diagnostic.

Validation

  • cargo nextest run -p alien-preflights --no-fail-fast (339 passed)
  • cargo fmt -p alien-preflights -- --check
  • git diff --check

Linear: ALIEN-945

@greptile-apps

greptile-apps Bot commented Sep 22, 2026

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

The PR appears safe to merge because the newly exempted workload controllers already wait for linked resources and resolve completed Sandbox bindings at runtime.

Summary

This PR narrows the Live Sandbox link validation so runtime-provisioned Containers and Daemons are accepted while setup-rendered consumers remain rejected.

  • Adds explicit Container and Daemon exemptions to the setup-time binding check.
  • Retains the rejection and clearer diagnostic for Workers and other setup-rendered consumers.
  • Adds coverage for accepted Container and Daemon links and preserves coverage for rejected Worker links.

Reviews (1) · Last reviewed commit: "fix: allow runtime workloads to link liv..."

@alongubkin
alongubkin merged commit f5bd7bc into main Sep 24, 2026
22 checks passed
@alongubkin
alongubkin deleted the alon/alien-945-live-sandbox-runtime-links branch September 24, 2026 01:42
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