Skip to content

Conversation

@takaokouji
Copy link
Contributor

No description provided.

takaokouji and others added 2 commits September 20, 2025 19:22
Fix the IAM role trust relationship to properly support both main branch
pushes and pull request events. The previous configuration was too restrictive
and only allowed main branch access.

Changes:
- Update trust relationship to include pull_request events
- Fix Action from sts:AssumeRole to sts:AssumeRoleWithWebIdentity
- Add comprehensive troubleshooting section to OIDC_SETUP.md
- Include debug commands for common OIDC issues

Root cause of deployment failure:
- Trust policy was missing pull_request condition
- Action type was incorrect for OIDC authentication

This fix enables GitHub Actions to authenticate via OIDC for both
main branch deployments and pull request validation.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Update OIDC setup documentation to reflect the final working configuration:

- Update trust relationship to use wildcard pattern: repo:smalruby/smalruby-infra:*
- Remove temporary JSON files and related troubleshooting references
- Replace CLI commands with AWS Console instructions for better usability
- Simplify troubleshooting section to focus on essential steps

Final configuration:
- Trust relationship allows all repository events (*)
- Comprehensive IAM permissions for CloudFormation, Lambda, API Gateway, S3, IAM
- Secure OIDC authentication with automatic token rotation

The configuration has been tested and confirmed working for automatic deployment.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@takaokouji takaokouji merged commit a34596b into main Sep 20, 2025
2 checks passed
@takaokouji takaokouji deleted the feature/enable-automatic-deployment branch September 20, 2025 12:46
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.

2 participants