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

CloudFormation template for managing the project's AWS CodeBuild workflows #159

Merged
merged 3 commits into from
Aug 27, 2022
Merged

CloudFormation template for managing the project's AWS CodeBuild workflows #159

merged 3 commits into from
Aug 27, 2022

Conversation

bmoffatt
Copy link
Collaborator

@bmoffatt bmoffatt commented Aug 22, 2022

Description of changes:

Validating #154 threw me down a rabbit hole of regression testing. The biggest change I made was to resolve constantly waiting for AWS SDK build, by moving the step out of any ci/codebuild/*.yml and into the respective ci/docker/* - to cache it for personal sanity. moved to #162 - This PR focuses now only on codifying the existing AWS CodeBuild CI workflows into a CloudFormation template

Adds:

This PR is for visibility - I am expecting some of the existing workflows to break, and need still need to synthesize the new CodeBuild project resources into the account used for this project before the web hooks take effect (edit: I have deployed this stack, web hooks can be seen in this PR and other recent PRs). Testing of the new workflows was initially done by synthesizing into a separate account, and setting the target repository to the fork/branch this PR is based on.

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

@bmoffatt bmoffatt marked this pull request as ready for review August 22, 2022 02:47
@marcomagdy
Copy link
Contributor

marcomagdy commented Aug 22, 2022

I love the changes. Thank you for doing all that work.
Would it be too much to ask to split this PR into many smaller ones? perhaps one that achieves each of the bullet points you outlined? it would make it a lot easier to review/digest.

@bmoffatt
Copy link
Collaborator Author

Can do!

@bmoffatt
Copy link
Collaborator Author

split out the other changes, and will get around later to updating this PR to focus only on ci/codebuild.yml and ci/update-images.sh

to help maintainers with adding/modifying the CI workflows

also adds helper script `ci/update-images.sh` for re-building and re-uploading the `ci/docker/*` images
@bmoffatt bmoffatt changed the title CodeBuild CI Updates CloudFormation template for managing the project's AWS CodeBuild workflows Aug 23, 2022
@bmoffatt bmoffatt merged commit 9dd0bdf into awslabs:master Aug 27, 2022
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