Add job audit script #222
Add job audit script #222
Conversation
eshulman2
force-pushed
the
main
branch
2 times, most recently
from
23ee673 to
605a1c9
Compare
stephenfin
left a comment
stephenfin
left a comment
There was a problem hiding this comment.
Could we get a blurb in the README for this new script? Could of additional points inline.
mandre
left a comment
mandre
left a comment
There was a problem hiding this comment.
Can we also see the frequency at which the periodics run?
| from pathlib import Path | ||
|
|
||
| import yaml | ||
| from ruamel.yaml import YAML |
There was a problem hiding this comment.
humm, why do we need 2 yaml libraries? Can't we use PyYAML for writing the report?
There was a problem hiding this comment.
the PyYAML output was less readable, so switched to ruamel to make the output more readable but PyYAML is still required for parsing. I can switch back to only PyYAML but I prefer the output with ruamel.
Add job audit script to create a yaml showcasing conditions for the job to run from config
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: eshulman2, mandre The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
3 participants
Add job audit script to create a yaml showcasing conditions for the job to run from config