Skip to content

Substitute image_config "." with build image during deploy#4772

Open
rustyconover wants to merge 1 commit intosuperfly:masterfrom
rustyconover:image-config-substitution
Open

Substitute image_config "." with build image during deploy#4772
rustyconover wants to merge 1 commit intosuperfly:masterfrom
rustyconover:image-config-substitution

Conversation

@rustyconover
Copy link

Summary

  • Extend updateContainerImage() to substitute image_config references of "." with the build image, for both top-level MachineConfig.Files and per-container ContainerConfig.Files
  • This follows the existing convention where container.Image = "." is replaced with the build image during deploy
  • Adds unit tests covering top-level files, container-level files, explicit image refs (left untouched), and files without image_config

Fixes #4771

Test plan

  • Unit tests pass (go test ./internal/command/deploy/ -run TestLaunchInputForUpdate)
  • Preflight tests (run by CI)

🤖 Generated with Claude Code

When deploying multi-container apps, files with `image_config` set to
"." are now substituted with the build image, matching the existing
convention where `container.Image = "."` is replaced. This applies to
both top-level machine config files and per-container files.

Without this change, there was no way to reference the just-built image
in a file's `image_config` field — users had to hardcode the full image
reference, which defeats the purpose of dynamic deploys.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

image_config in files not substituted with build image during deploy

1 participant