Skip to content

ci: Add CI guardrail for BoringSSL fork#5715

Merged
kaukabrizvi merged 10 commits intoaws:mainfrom
kaukabrizvi:ci-stale-boring
Feb 17, 2026
Merged

ci: Add CI guardrail for BoringSSL fork#5715
kaukabrizvi merged 10 commits intoaws:mainfrom
kaukabrizvi:ci-stale-boring

Conversation

@kaukabrizvi
Copy link
Copy Markdown
Contributor

@kaukabrizvi kaukabrizvi commented Jan 29, 2026

Goal

Resolves #5692 by adding a workflow that runs on pull requests to check the status of the BoringSSL fork used in integration tests.

Why

To add a layer of assurance that our integration tests run with a recent state of BoringSSL.

How

Added a script that compares the most recent commit date on the fork to the current date. The script fails if the time exceeds a set time threshold (currently set to 60 days) and is called from a github workflow.

Callouts

I am working on granting s2n-core access to the fork repository so that all team members can refresh the repository if it has gone stale.

Testing

I ran the workflow in this PR, once with the threshold at 180 days to observe the success case and once with the threshold at 1 day to observe the faliure case.

Related

Resolves #5692, the BoringSSL fork dependency was added in #5659.

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 29, 2026
@kaukabrizvi kaukabrizvi marked this pull request as ready for review February 9, 2026 21:56
@kaukabrizvi kaukabrizvi added this pull request to the merge queue Feb 17, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 17, 2026
@kaukabrizvi kaukabrizvi added this pull request to the merge queue Feb 17, 2026
Merged via the queue into aws:main with commit bbfd46e Feb 17, 2026
54 checks passed
@kaukabrizvi kaukabrizvi deleted the ci-stale-boring branch February 17, 2026 18:52
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.

Add CI guardrail to detect stale BoringSSL fork

3 participants