Skip to content

fix: iam oidc thumbprint permission - #16

Merged
Hashversion merged 2 commits into
mainfrom
hotfix/iam-oidc-thumbprint-permission
May 16, 2026
Merged

fix: iam oidc thumbprint permission#16
Hashversion merged 2 commits into
mainfrom
hotfix/iam-oidc-thumbprint-permission

Conversation

@Hashversion

@Hashversion Hashversion commented May 16, 2026

Copy link
Copy Markdown
Owner
  • fix: add iam permission for oidc provider thumbprint update
  • fix: add iam permission to create policy versions

Summary by CodeRabbit

  • Chores
    • Expanded CI/CD infrastructure permissions to support enhanced deployment and infrastructure management capabilities.

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 16, 2026

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Updated GitHub Actions IAM policy document with two new permissions: iam:UpdateOpenIDConnectProviderThumbprint for OIDC provider management and iam:CreatePolicyVersion for policy management. Both additions expand the permissions available to GitHub Actions for Terraform-managed infrastructure operations.

Changes

GitHub Actions Infrastructure Permissions

Layer / File(s) Summary
Expand GitHub Actions IAM permissions
infra/aws/main/iam.tf
IAM policy document updated with two new action permissions: iam:UpdateOpenIDConnectProviderThumbprint in the OIDC provider management statement and iam:CreatePolicyVersion in the IAM policy management statement.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Two permissions, small but mighty fine,
For OIDC thumbprints and policy design,
GitHub Actions now has what it needs,
To manage the infrastructure seeds! 🌱

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: adding an IAM OIDC thumbprint permission, which is the primary focus of the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hotfix/iam-oidc-thumbprint-permission

Comment @coderabbitai help to get the list of available commands and usage tips.

@Hashversion Hashversion changed the title hotfix/iam oidc thumbprint permission fix: iam oidc thumbprint permission May 16, 2026
@Hashversion
Hashversion merged commit 96cf5c7 into main May 16, 2026
2 of 3 checks passed
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.

1 participant