Skip to content

infra: update default S3 bucket name per environment#104

Merged
kou merged 1 commit into
enactic:mainfrom
abetomo:update-default-value
Jul 3, 2026
Merged

infra: update default S3 bucket name per environment#104
kou merged 1 commit into
enactic:mainfrom
abetomo:update-default-value

Conversation

@abetomo

@abetomo abetomo commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings July 3, 2026 01:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Terraform environment defaults so staging and production deployments point at distinct S3 buckets, reducing the chance of cross-environment data mixing when the app reads/writes to S3 via S3_BUCKET_NAME.

Changes:

  • Change staging default s3_bucket_name from openeval to openeval-staging.
  • Change production default s3_bucket_name from openeval to openeval-production.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
infra/environments/staging/variables.tf Updates the staging environment’s default S3 bucket name.
infra/environments/production/variables.tf Updates the production environment’s default S3 bucket name.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kou kou merged commit bc96951 into enactic:main Jul 3, 2026
2 checks passed
@abetomo abetomo deleted the update-default-value branch July 3, 2026 01:14
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.

3 participants