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

Dynamic variable `pull_request_number` is not populated intermittently for push events related to pull-request merge

XMLWordPrintable

    • 3
    • False
    • Hide

      None

      Show
      None
    • False
    • Hide
      Before this update, Pipelines-as-Code occasionally failed to identify pull request numbers when using the Merge Commit strategy due to GitHub API indexing delays. This has been fixed by implementing an exponential backoff retry mechanism that detects merge commits and retries the API call over several seconds if the association isn't found immediately. This ensures that the `pull_request_number` variable is correctly populated even when there is minor provider latency.
      Show
      Before this update, Pipelines-as-Code occasionally failed to identify pull request numbers when using the Merge Commit strategy due to GitHub API indexing delays. This has been fixed by implementing an exponential backoff retry mechanism that detects merge commits and retries the API call over several seconds if the association isn't found immediately. This ensures that the `pull_request_number` variable is correctly populated even when there is minor provider latency.
    • Bug Fix
    • Proposed
    • Pipelines Sprint CrookShank 44, Pipelines Sprint CrookShank 45, Pipelines Sprint CrookShank 46

      Description of problem:

      The Dynamic variable pull_request_number is not populated for push events that occur as part of a pull-request merge. The issue is observed intermittently, causing the following tasks to fail. this is causing the when pull request is merged with a merge-commit strategy.

      Workaround

      Prerequisites (if any, like setup, operators/versions):

      Steps to Reproduce

       # <steps>

       

      Actual results:

      Expected results:

      Reproducibility (Always/Intermittent/Only Once):

      Acceptance criteria: 

       

      Definition of Done:

      Build Details:

      Additional info (Such as Logs, Screenshots, etc):

       

       *

              rh-ee-zashaikh Zaki Shaikh
              rhn-support-jyarora Jyotsana Arora
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Created:
                Updated:
                Resolved: