Skip to content

fix: update bucket IAM member keys for consistency in generated lists#34

Merged
filippolmt merged 3 commits intomainfrom
fix/update_bucket_iam_member_keys
Jan 22, 2026
Merged

fix: update bucket IAM member keys for consistency in generated lists#34
filippolmt merged 3 commits intomainfrom
fix/update_bucket_iam_member_keys

Conversation

@filippolmt
Copy link
Copy Markdown
Contributor

No description provided.

@filippolmt filippolmt force-pushed the fix/update_bucket_iam_member_keys branch from 22f9c47 to 80c7586 Compare January 22, 2026 14:32
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request fixes an issue where bucket IAM member resources would fail with "Invalid for_each argument" errors when append_random_suffix = true was used. The fix changes the for_each keys from using dynamically generated bucket names (with random suffixes) to using static input bucket names.

Changes:

  • Modified local variables to use input bucket names instead of generated names with random suffixes
  • Updated IAM member resource references from each.value.name to each.value.bucket_name for consistency
  • Renamed the admin member key from bucket_obj_adm to bucket_obj_admin in the local variable

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
main.tf Updated local variables to use static bucket names instead of generated names, and fixed resource references to use bucket_name consistently
CHANGELOG.md Added version 0.12.0 release notes documenting the breaking change to IAM member for_each keys

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

Comment thread main.tf
@filippolmt filippolmt merged commit c99808d into main Jan 22, 2026
7 checks passed
@filippolmt filippolmt deleted the fix/update_bucket_iam_member_keys branch January 22, 2026 16:20
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