Skip to content

Smoke-Build a Dispatched Run of the Smoke Caller Snippet Past Its Paths Filter #3149

Closed

Description

@ptr727

Context

Raised by CodeRabbit on the develop -> main promotion #3146, against catalog/snippets/workflows/test-pull-request-smoke.yml, which #3113 added. The YAML is older than that file: main's docs/reusable-workflows.md "Release repos with a smoke build" example already carries the same workflow_dispatch: trigger and the same if: needs.changes.outputs.release == 'true' gate on smoke-build, and #3113 copied that example into the catalog, where develop's doc still carries it.

Gap

On a workflow_dispatch run there is no pull request, so dorny/paths-filter compares against a base it picks itself. A dispatch on the default branch compares only that branch's last commit. Where that commit touches no release path, smoke-build skips, and check-workflow-status reads the skip as a pass, so a manual run meant to smoke-build the target builds nothing and reports green.

Possible Direction

Let a dispatch bypass the filter, if: github.event_name == 'workflow_dispatch' || needs.changes.outputs.release == 'true', in the snippet and the docs example together, or drop workflow_dispatch: from the pull request workflow where a dispatch has no use. WORKFLOW.md D1.1 states the filter for a pull request only, so either choice keeps the contract. Which one the fleet wants is the maintainer's call.

Activity

  1. added
    github_actionsPull requests that update GitHub Actions code
    pre-existingReview finding classed pre-existing per local-strict-review Disposing of Findings
    on Oct 16, 2026
  2. ptr727 commented on Oct 16, 2026

    @ptr727
    OwnerAuthor

    Maintainer decision (asked at the close of the engineering round 28 session): a dispatch bypasses the filter. Gate smoke-build on github.event_name == 'workflow_dispatch' || needs.changes.outputs.release == 'true', in catalog/snippets/workflows/test-pull-request-smoke.yml and in the docs/reusable-workflows.md example together, since a manual run is an explicit ask to build. Keep workflow_dispatch:.

  3. added a commit that references this issue on Oct 16, 2026
    7a58899
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    github_actionsPull requests that update GitHub Actions codepre-existingReview finding classed pre-existing per local-strict-review Disposing of Findings

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions