Hardcode website static assets bucket name in Pulumi #10968
Conversation
vacekj
requested review from
thelostone-mc,
chibie,
nutrina and
tim-schultz
as code owners
| @@ -72,6 +72,7 @@ const staticAssetsBucket = new aws.s3.Bucket("bucket", { | |||
| website: { | |||
| indexDocument: "index.html", | |||
| }, | |||
| bucket: "bucket-0427bb4", | |||
There was a problem hiding this comment.
Would probably be better to but this hard coding in the environment variables section. Just would ensure that it goes to what we want exactly for the time being.
There was a problem hiding this comment.
I think Gerald said while we were pairing that he's gonna change this section later anyway, so if you don't mind could we merge this as-is?
There was a problem hiding this comment.
So, I can approve this as a hot-fix to get the issue resolved now, but we need to make sure to follow-up on this and properly resolve it.
Not sure who will pick up the responsibility of properly fixing this. But I will make an issue in GH for myself as a reminder, and we can follow-up on that @kammerdiener
|
@kammerdiener have created this issue in GH related to getting a proper fix: #10969 |
There are currently two buckets on S3, the new one, bucket-fc46b51, deployed Nov 11. The app is currently mis-configured to upload assets to the new bucket, but the CloudFront CDN still points to the old bucket
bucket-0427bb4. This hardcodes the app to point to the old bucket, which has had assets synced from the new one.