-
Bug
-
Resolution: Done
-
Major
-
None
Description of Problem:
Previously, when a PipelineRun with invalid YAML was detected, the error was
reported in the GitHub Checks tab of the Pull Request. However, after recent
changes, these errors are now only logged in the controller and are not
surfaced to the user in the GitHub Checks API. This results in a poor user
experience, as users are being unaware of the issue unless they check the logs.
Actual Results:
- The GitHub Checks tab does not show any error or failure related to the invalid YAML.
- The error is only logged in the controller logs, with a message like:
🚨 08:39:42 pac-controller prun: tekton validation error: yaml: line 35: could not find expected :
- Users are not notified of the issue in the GitHub UI.
Expected Results:
- The GitHub Checks tab should display an error or a failure status when a PipelineRun contains invalid YAML.
- Users should receive clear feedback about the issue directly in the Pull Request, such as a comment or a failed check.
Reproducibility:
Always
Acceptance Criteria:
Definition of Done:
- Errors in PipelineRun YAML are surfaced in the GitHub Checks tab.
- Users are notified of YAML parsing errors directly in the Pull Request.
- The fix is tested and verified in a real-world scenario.
- Documentation is updated to reflect the new behavior.
Build Details:
- Controller Version: [Insert version here]
- GitHub Integration Version: [Insert version here]
Additional Info (Such as Logs, Screenshots, etc):
Logs:
💡 08:39:41 pac-controller issue_comment: pipelinerun recheck on chmouel/scratchmyback#344 has been requested 💡 08:39:42 pac-controller Using PipelineRun definition from source pull request chmouel/scratchmyback#344 SHA on 407080e913874ed2af476cb9a81a2676bbb8ce3b 💡 08:39:42 pac-controller matching pipelineruns to event: URL=https://github.com/chmouel/scratchmyback, target-branch=main, source-branch=on-assign, target-event=pull_request 💡 08:39:42 pac-controller matched pipelinerun with name: scratchmyback-pull-request on gitops comment: "/assign chmouel\r\n\r\n" 🚨 08:39:42 pac-controller prun: tekton validation error: yaml: line 35: could not find expected : 💡 08:39:42 pac-controller cannot locate templates in .tekton/ directory for this repository in on-assign
- is documented by
-
RHDEVDOCS-6460 Document YAML Parsing Errors in PAC PipelineRun being reported in Pull Request comments as well as logs
-
- Closed
-