Skip to content

fix(cdk): reference existing GitHub OIDC provider via fromAccount#735

Merged
stabbylambda merged 1 commit into
mainfrom
fix/oidc-v4-from-account
Jun 28, 2026
Merged

fix(cdk): reference existing GitHub OIDC provider via fromAccount#735
stabbylambda merged 1 commit into
mainfrom
fix/oidc-v4-from-account

Conversation

@stabbylambda

Copy link
Copy Markdown
Owner

aws-cdk-github-oidc v4 switched GithubActionsIdentityProvider from a
Lambda-backed custom resource (Custom::AWSCDKOpenIdConnectProvider) to the
native AWS::IAM::OIDCProvider. Reusing the same logical id across that type
change made CloudFormation reject the change set ("Update of resource type is
not permitted").

The provider is an account-level singleton, so reference the existing one with
fromAccount instead of managing it in this stack. Migration was done in two
deploys: first retaining the v2 custom resource's physical provider, then
removing it here so the retained provider survives and UploadRole's trust
policy points at the existing provider ARN.

Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com

aws-cdk-github-oidc v4 switched GithubActionsIdentityProvider from a
Lambda-backed custom resource (Custom::AWSCDKOpenIdConnectProvider) to the
native AWS::IAM::OIDCProvider. Reusing the same logical id across that type
change made CloudFormation reject the change set ("Update of resource type is
not permitted").

The provider is an account-level singleton, so reference the existing one with
fromAccount instead of managing it in this stack. Migration was done in two
deploys: first retaining the v2 custom resource's physical provider, then
removing it here so the retained provider survives and UploadRole's trust
policy points at the existing provider ARN.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@stabbylambda stabbylambda merged commit 954bff7 into main Jun 28, 2026
1 check passed
@stabbylambda stabbylambda deleted the fix/oidc-v4-from-account branch June 28, 2026 18: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.

1 participant