fix(aws static-site): aws-cli in runner image + honour baseDnsZone#282
Closed
smecsia wants to merge 2 commits into
Closed
fix(aws static-site): aws-cli in runner image + honour baseDnsZone#282smecsia wants to merge 2 commits into
smecsia wants to merge 2 commits into
Conversation
`pkg/clouds/pulumi/aws/static_website.go` shells out to `aws s3 sync` via Pulumi `local.NewCommand`, but neither the prod nor staging Dockerfile installed the AWS CLI — every static-site stack run under the simplecontainer/github-actions image failed with `/bin/sh: aws: not found`. Adding the alpine `aws-cli` package (community repo, python-based) to the runtime layer of both images + extending the build-time smoke test to cover `aws --version`. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…ured aws.StaticSiteInput embeds api.StackConfigStatic (which carries Site. BaseDnsZone) but never exposed an OverriddenBaseZone() method, so the type assertion in pulumi/deploy.go fell through and the Cloudflare registrar fell back to the parent stack's default zone. Records intended for e.g. simple-forge.com landed in the simple-container.com zone and Cloudflare appended the suffix → simple-forge.com.simple- container.com. GCP, Lambda and ECS Fargate inputs already implement the interface; this just restores parity. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Contributor
Author
|
Closing per request — branch kept for manual PR. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.