Skip to content

fix: harden iac-code Web ECS deployment flow - #228

Merged
guima-why merged 5 commits into
mainfrom
codex/iac-code-web-install-latest
Aug 5, 2026
Merged

fix: harden iac-code Web ECS deployment flow#228
guima-why merged 5 commits into
mainfrom
codex/iac-code-web-install-latest

Conversation

@guima-why

Copy link
Copy Markdown
Collaborator

What changed

  • Install the latest iac-code release in the ECS bootstrap instead of pinning a template parameter.
  • Seed Alibaba Cloud OAuth credentials with the ROS stack region and use deepseek-v4-flash-0731 as the default model.
  • Keep the iac-code Web deployment candidate on the intended single-ECS + EIP topology with port 8766.
  • Preserve structured ROS stack results through A2A events and Web output derivation while retaining compatibility with diagnostic text appended to tool output.
  • Cover stack ownership when a stack is observed before the terminal deployment result.

Why

ROS deployment diagnostics can make the display content no longer valid JSON, which prevented stack outputs from reaching the Web output panel. The deployment template also pinned an older package version and did not align its cloud region and model defaults with the deployed stack.

Impact

Pipeline deployments use the latest published iac-code package, start with the correct Alibaba Cloud region and model defaults, and reliably surface ROS stack results in the Web UI.

Validation

  • make lint
  • 371 passed targeted tests
  • make test: 13545 passed, 2 skipped
  • Independent Codex and Claude Code compatibility reviews: PASS

@guima-why
guima-why marked this pull request as ready for review August 5, 2026 13:54
@guima-why
guima-why merged commit b362161 into main Aug 5, 2026
13 checks passed
@guima-why
guima-why deleted the codex/iac-code-web-install-latest branch August 5, 2026 13:54
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