Skip to content

fix: handle update event for the github customer resource handler - #409

Merged
xuezhma merged 1 commit into
mainfrom
fix/handle-update
Jan 29, 2026
Merged

fix: handle update event for the github customer resource handler#409
xuezhma merged 1 commit into
mainfrom
fix/handle-update

Conversation

@xuezhma

@xuezhma xuezhma commented Jan 27, 2026

Copy link
Copy Markdown
Contributor

Describe your changes

  • added UPDATE handler support for sagemaker-templates GitHub repository custom resource:
    • CodeConnectionArn changes: updates CodeBuild source credentials
    • S3BucketObjectKey changes (seed code updates): creates branch and pull request with new code
    • repository_owner changes: creates new repository under new owner with seed code, keeps old repository intact
  • added codebuild:DeleteSourceCredentials permission to sagemaker-templates GitHub repository Lambda role

Issue ticket number and link

Checklist before requesting a review

  • I updated CHANGELOG.MD with a description of my changes
  • If the change was to a module, I ran the code validation script (scripts/validate.sh)
  • If the change was to a module, I have added thorough tests
  • If the change was to a module, I have added/updated the module's README.md
  • If a module was added, I added a reference to the module to the repository's README.md
  • I verified that my code deploys successfully using seedfarmer apply

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@xuezhma
xuezhma merged commit 895fea2 into main Jan 29, 2026
6 checks passed
@kwangaws kwangaws mentioned this pull request Feb 9, 2026
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