Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cl-724 Adding support for codedeploy deployment configs and codepipeline custom action types and webhooks #20

Merged

Conversation

swhite-oreilly
Copy link

@swhite-oreilly swhite-oreilly commented Oct 31, 2023

This PR deployment config support for CodeDeploy deployment configs and groups along with CodePipeline custom action types and webhooks.

Tests scripts are included in the related PR

Note: Due to errors with using the cloud control resource types AWS::CodeDeploy::DeploymentConfig and AWS::CodePipeline::CustomActionType, creating this custom nuke modules for these resource types.

Due to errors with using the cloud control resource type AWS::CodeDeploy::DeploymentConfig, creating custom nuke module for CodeDeploy Deployment Configs.
Copy link

@gsoria gsoria left a comment

Choose a reason for hiding this comment

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

I haven't finished trying all the changes from the 3 PRs but I thought it could be useful sending an initial review 🙂

resources/codedeploy-deployment-configs.go Outdated Show resolved Hide resolved
@swhite-oreilly swhite-oreilly changed the title cl-724 Adding support for codedeploy deployment configs cl-724 Adding support for codedeploy deployment configs and codepipeline custom action types Nov 1, 2023
@swhite-oreilly swhite-oreilly changed the title cl-724 Adding support for codedeploy deployment configs and codepipeline custom action types cl-724 Adding support for codedeploy deployment configs and codepipeline custom action types and webhooks Nov 2, 2023
Copy link

@gsoria gsoria left a comment

Choose a reason for hiding this comment

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

This looks good to me, confirmed locally that accounts are cleanup up correctly:

us-east-1 - CodeDeployDeploymentGroup - CodeDeployDemo_DG - [ApplicationName: "CodeDeployDemoApplication-20fdbddbb85b101b5943", DeploymentGroupName: "CodeDeployDemo_DG"] - removed
us-east-1 - CodeDeployDeploymentConfig - ThreeQuartersHealthy - [DeploymentConfigName: "ThreeQuartersHealthy"] - removed

@swhite-oreilly swhite-oreilly merged commit 04688f8 into oreilly-main Nov 3, 2023
1 check passed
corybekk pushed a commit that referenced this pull request Nov 6, 2024
* chore: update to actions/checkout@v4

* chore: update to actions/configure-pages@v4

* chore: update to actions/deploy-pages@v4

* chore: actions/setup-go@v5

* chore: update to actions/upload-artifact@v4

* chore: actions/setup-python@v5

* chore: actions/upload-pages-artifact@v3

* chore: update to docker/login-action@v3

* chore: update docker actions

* update goreleaser-action to v5
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