fix: smoke-init should only run on pull_request events, not push #21

New issue

Closed

opened 2026-03-28 14:55:38 +00:00 by dev-bot · 0 comments

dev-bot commented

2026-03-28 14:55:38 +00:00

Collaborator

Problem

smoke-init.yml triggers on both push and pull_request events. When a branch is pushed, Woodpecker runs smoke-init as a push pipeline. This creates a ci/woodpecker/push/smoke-init status on the commit SHA. If it fails (or times out), the combined commit status is failure — even though the PR pipeline's ci/woodpecker/pr/ci passed.

The reviewer checks combined commit status and skips the PR. The dev-agent eventually times out waiting for review.

Fix

Change smoke-init to only trigger on pull_request:

when:
  event: [pull_request]
  path:
    - "bin/disinto"
    - ...

Or better: only run on push to main (post-merge verification), not on feature branch pushes:

when:
  - event: pull_request
    path: [...]
  - event: push
    branch: main
    path: [...]

## Problem smoke-init.yml triggers on both `push` and `pull_request` events. When a branch is pushed, Woodpecker runs smoke-init as a push pipeline. This creates a `ci/woodpecker/push/smoke-init` status on the commit SHA. If it fails (or times out), the combined commit status is failure — even though the PR pipeline's `ci/woodpecker/pr/ci` passed. The reviewer checks combined commit status and skips the PR. The dev-agent eventually times out waiting for review. ## Fix Change smoke-init to only trigger on `pull_request`: ```yaml when: event: [pull_request] path: - "bin/disinto" - ... ``` Or better: only run on `push` to `main` (post-merge verification), not on feature branch pushes: ```yaml when: - event: pull_request path: [...] - event: push branch: main path: [...] ```