Uploaded image for project: 'OpenShift Pipelines'
  1. OpenShift Pipelines
  2. SRVKP-7670

Merge w/ final non-matching commit doesn't trigger a build

XMLWordPrintable

    • 3
    • False
    • Hide

      None

      Show
      None
    • False
    • 3
    • Pipelines Sprint Crookshank 21

      Request type: Error / Bug
      Reason of urgency: I could push another commit if I had to...

      Request info

      Team: gatekeeper
      Tenant: gatekeeper
      Application: gatekeeper
      Component: gatekeeper

      Details:

              Rebase merge w/ multiple commits doesn't seem to trigger a built
              This PR just merged:
      • https://github.com/stolostron/gatekeeper/pull/271
      It resulted in these commits:
      • https://github.com/stolostron/gatekeeper/commits/release-3.17/?since=2025-01-28&until=2025-01-28
      Of these three commits, the last one updated files not included in the CEL expression annotation. As expected, the build ran on the PR since the other 2 commits updated relevant files. However, on push, the build didn't run.
      
      I'm hoping it's just a hiccup or I misconfigured something in the CEL, but my fear is that since it's a rebase merge, Konflux is only looking at the final commit and skipping the push build, but help would be much appreciated in identifying the cause!
      
      CEL expression (modified from the original for readability):
      ```pipelinesascode.tekton.dev/on-cel-expression:
        event == "pull_request" && 
        target_branch == "release-3.17" &&
        (
          ".tekton/gatekeeper-*.yaml".pathChanged() || 
          "Dockerfile".pathChanged() || 
          "build/Dockerfile.rhtap".pathChanged() || 
          "go.(mod|sum)".pathChanged() || 
          "main.go".pathChanged() || 
          "(apis|pkg|vendor)/***".pathChanged()
        )```
      ref: https://github.com/stolostron/gatekeeper/blob/2e142402ae3631eeb10b191f1b271766b0bc02cc/.tekton/gatekeeper-3-17-push.yaml#L10
              

              rh-ee-zashaikh Zaki Shaikh
              rh-ee-dhaiduce Dale Haiducek
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Created:
                Updated: