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

ci: add script to help launch stuck codebuild jobs #5004

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

lrstewart
Copy link
Contributor

@lrstewart lrstewart commented Jan 8, 2025

Release Summary:

Description of changes:

We have enough codebuild jobs now that manually launching them, like we have to do when we modify a buildpsec, is very slow and tedious. I was very annoyed trying to get 7220e23 merged.

This just adds a simple script that will launch all the codebuild jobs for you, as long as you have the aws cli setup right. I could put this script somewhere else, but committing it seems like our best bet to keep it up to date.

Call-outs:

I'd love a way to enforce that the script is always up-to-date with all our Codebuild jobs, but I'm not sure how to do that easily 🤔

Testing:

I ran it locally and verified that the builds were successfully launched. Anyone with access to the Codebuild jobs can too, to verify. This is the output of a run:

%  ./codebuild/bin/start_codebuild.sh pr/5004
Builds:
aws --region us-west-2 codebuild start-build-batch --source-version pr/5004
        --project-name AddressSanitizer
aws --region us-west-2 codebuild start-build-batch --source-version pr/5004
        --project-name S2nIntegrationV2SmallBatch
aws --region us-west-2 codebuild start-build-batch --source-version pr/5004
        --project-name Valgrind
aws --region us-west-2 codebuild start-build-batch --source-version pr/5004
        --project-name s2nFuzzBatch
aws --region us-west-2 codebuild start-build-batch --source-version pr/5004
        --project-name s2nGeneralBatch
aws --region us-west-2 codebuild start-build-batch --source-version pr/5004
        --project-name s2nUnitNix
aws --region us-east-2 codebuild start-build-batch --source-version pr/5004
        --project-name Integv2NixBatchBF1FB83F-7tcZOiMDWPH0
aws --region us-west-2 codebuild start-build --source-version pr/5004
        --project-name kTLS
1) Start 8 builds
2) Exit
#? 1
Starting builds...
        "id": "AddressSanitizer:d8ee382f-c514-49c7-bf7c-6e7f85f97c3f",
        "id": "S2nIntegrationV2SmallBatch:03004132-4aa8-49cf-9c8f-409e4513fb94",
        "id": "Valgrind:61ce8e36-5d64-42b3-be2b-eaa2c7b74b70",
        "id": "s2nFuzzBatch:181468d7-ce88-42e2-af70-105871a3d900",
        "id": "s2nGeneralBatch:b85fbb1c-a2eb-4243-97af-892cd5f21c4f",
        "id": "s2nUnitNix:98accd76-0fe3-4cb6-a5ac-d8abb413df5d",
        "id": "Integv2NixBatchBF1FB83F-7tcZOiMDWPH0:1bd41ce3-ac35-4782-a02b-fc53e2eea0d6",
        "id": "kTLS:835eacfc-9ba2-49b6-bba0-65141f4b01f3",
All builds successfully launched.

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

@github-actions github-actions bot added the s2n-core team label Jan 8, 2025
@lrstewart lrstewart marked this pull request as ready for review January 8, 2025 01:09
@lrstewart lrstewart requested a review from dougch as a code owner January 8, 2025 01:09
@lrstewart lrstewart requested a review from jouho January 8, 2025 01:09
Copy link
Contributor

@jouho jouho left a comment

Choose a reason for hiding this comment

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

Tested and confirmed it works. Thank you for adding this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants