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

PaC Structured Logs Missing Source Repo Context

XMLWordPrintable

    • 2
    • False
    • Hide

      None

      Show
      None
    • False
    • Structured logs in Pipelines as Code now correctly include the source repository context. Previously, structured logs emitted by the PaC controller were missing details such as repository name/URL.
    • Pipelines Sprint Tekshift 32, Pipelines Sprint Tekshift 33

      Description of problem:

      The structured logs for Pipelines as Code miss important source repository details from the initiating webhook request. Such information would make it easier for operators to identify user-reported issues.

      Workaround: Grepping of individual log files

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

      OpenShift Pipelines on Konflux (latest?)

      Steps to Reproduce

      Run a Pipeline with Pipelines as Code
       

      Actual results:

      The "pac-watcher" and "pac-controller" logs contain an `event-sha` key that contains the commit ID for the webhook event. It does not contain other information reporting the origin of the webhook event.

      Expected results:

      The pac-watcher and pac-controller logs should report enough context for an operator to identify where a webhook event came from, such as:

      • Source repository URL
      • Branch name, if provided
      • Event type (this is not always consistent)
      • Commit SHA (also not always consistent)

      Reproducibility (Always/Intermittent/Only Once):

      Always

      Acceptance criteria: 

       

      Definition of Done:

      Build Details:

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

       

       *

              rh-ee-shubbhar Shubham Bhardwaj
              adkaplan@redhat.com Adam Kaplan
              Votes:
              1 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated:
                Resolved: