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

Update CodeBuild Runner integration to include the run id and attempt #738

Closed
wants to merge 33 commits into from

Conversation

popematt
Copy link
Contributor

@popematt popematt commented Apr 9, 2024

Issue #, if available:

None

Description of changes:

According to https://docs.aws.amazon.com/codebuild/latest/userguide/action-runner.html, we need to start including the run id and run attempt in the "runs on" configuration for CodeBuild-hosted runners.
This will also make it impossible to create a branch protection rule on those particular tasks because we don't have a stable job name now that the run id, etc. are now part of the strategy matrix and therefore part of the parameterized task name, so I've created a job that functions as a "join" operation for all of the build strategies.

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

@popematt popematt force-pushed the main branch 3 times, most recently from 94acfc5 to 0c8e6cf Compare April 9, 2024 21:56
@popematt popematt closed this Apr 10, 2024
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