Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

ci: add pilot auto-dispatch entry points for approved PRs#32053

Open
gkarat wants to merge 6 commits into
openssl:masteropenssl/openssl:masterfrom
gkarat:ct-validation-dispatchgkarat/openssl:ct-validation-dispatchCopy head branch name to clipboard
Open

ci: add pilot auto-dispatch entry points for approved PRs#32053
gkarat wants to merge 6 commits into
openssl:masteropenssl/openssl:masterfrom
gkarat:ct-validation-dispatchgkarat/openssl:ct-validation-dispatchCopy head branch name to clipboard

Conversation

@gkarat

@gkarat gkarat commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

This is a pilot for automatically running selected CI workflows after a PR is approved.

The pilot intentionally includes only three workflows:

  • ct-validation-daily
  • run-checker-daily
  • valgrind-daily

It covers only the automatic on-approval path. The proposed /ci comment command for manually requesting CI will be implemented separately.

For bot dispatches, the workflows accept the PR number, exact head SHA, and check-run ID. A shared reusable workflow rejects incomplete or malformed input before checkout. Jobs then check out the specified SHA with read-only permissions and without persisted credentials.

The run name carries the check-run correlation data, and PR-keyed concurrency cancels obsolete runs.

Existing scheduled and Actions-tab manual execution remain unchanged: leave the internal inputs empty to run normally. Existing fork behavior is also preserved.

Additive: schedule and bare-manual runs unchanged. Adds validated,
fail-closed pr/head_sha/check_run_id inputs so a caller can dispatch against
an exact commit; run-name echoes them back for correlation.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…st a PR head SHA

Additive: schedule and bare-manual runs unchanged. Adds validated,
fail-closed pr/head_sha/check_run_id inputs so a caller can dispatch each
against an exact commit; run-name echoes them back for correlation.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@openssl-ci-bot openssl-ci-bot Bot added the approval: review pending This pull request needs review by a committer label Jul 23, 2026
gkarat and others added 3 commits July 23, 2026 15:00
Clarify the bot-only workflow inputs and validate any supplied tuple through a shared reusable workflow without interpolating input values into shell source.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@gkarat gkarat changed the title ci: allow ct-validation-daily to be dispatched against a PR head SHA ci: allow daily workflows to be dispatched against a PR head SHA Jul 23, 2026
@gkarat gkarat changed the title ci: allow daily workflows to be dispatched against a PR head SHA ci: add pilot auto-dispatch entry points for approved PRs Jul 23, 2026
@gkarat
gkarat marked this pull request as ready for review July 23, 2026 13:20
@gkarat
gkarat requested a review from quarckster as a code owner July 23, 2026 13:20
@gkarat
gkarat requested review from a team, bob-beck, levitte, nhorman, npajkovsky and t8m July 23, 2026 13:20
@mbroz mbroz added branch: master Applies to master branch triaged: test The issue/pr requests or implements test enhancements without any other code changes. labels Jul 24, 2026
mbroz
mbroz previously approved these changes Jul 24, 2026

@mbroz mbroz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, just a comment.

Comment thread .github/workflows/validate-dispatch-inputs.yml Outdated
@mbroz
mbroz requested a review from jogme July 24, 2026 10:36
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

@jogme jogme left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, thank you!

For a little more context here: The CI will be dispatched by the ci_bot, therefore this PR adds only variables that helps the bot to be plumbed in to the workflows.

@mbroz mbroz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@openssl-ci-bot openssl-ci-bot Bot added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Jul 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals branch: master Applies to master branch triaged: test The issue/pr requests or implements test enhancements without any other code changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.