Closed purelind closed 5 months ago
I have already done a preliminary review for you, and I hope to help you do a better job.
Summary of the changes:
main.py
is added to the scripts/ops/parse-jenkins-build-log
directory.requirements.txt
is added to the same directory.main.py
script can analyze the console output of failed builds and save the output to a file.requirements.txt
file lists the required dependencies for the script.Potential problems:
JOB_NAME
exists. It would be better to add some error handling in case these assumptions are not true.failed_builds_output
directory and the summary of failed builds to the failed_builds_summary.csv
file. These directories and files are created in the same directory as the script. It would be better to make these paths configurable so that the user can specify where to save the output and summary files.Suggestions for fixing:
[APPROVALNOTIFIER] This PR is APPROVED
Approval requirements bypassed by manually added approval.
This pull-request has been approved by:
The full list of commands accepted by this bot can be found here.
The pull request process is described here
add scripts parse jenkins build log.