action #122923
closedcoordination #96596: [qe-core][CI] CI/CD and Coding style improvements
[qe-core][CI] Hide passing check messages from the CI logs
0%
Description
Whenever a check fails in the CI, it's often difficult to directly find the file causing the failure, for instance here.
At the moment, things that come to mind are:
- Tidy checks.
- Metadata checks
- [yaml checks]
Acceptance Criteria¶
- Passing checks done on files aren't shown unless
--verboseor another flag is passed during the calls tomake
See also: https://github.com/os-autoinst/os-autoinst/pull/2235 for silencing Tidy
Updated by szarate over 2 years ago
- Copied from action #118588: [qe-core][CI] Anotate output when there are code violations on github's PR. added
Updated by szarate over 2 years ago
- Subject changed from [qe-core][CI] Anotate output when there are code violations on github's PR to [qe-core][CI] Hide passing checks from the CI
Updated by szarate over 2 years ago
- Subject changed from [qe-core][CI] Hide passing checks from the CI to [qe-core][CI] Hide passing check messages from the CI logs
- Description updated (diff)
Updated by szarate over 2 years ago
- Sprint changed from QE-Core: December Sprint (Dec 07 - Jan 11) to QE-Core: January Sprint (Jan 11 - Feb 08)
Updated by rfan1 over 2 years ago
- Status changed from In Progress to Feedback
PR is merged, Let me monitor the following commits to see if any issues.
I don't change the logic for yamllint CI checks for now, we can file a new ticket and check with the maintainer to see if we can do the same thing for it.
Updated by pdostal over 2 years ago
rfan1 wrote:
PR is merged, Let me monitor the following commits to see if any issues.
Can you link the PR?
I don't change the logic for
yamllintCI checks for now, we can file a new ticket and check with the maintainer to see if we can do the same thing for it.
Go for it, just let me know in the pull request.
Updated by rfan1 over 2 years ago
pdostal wrote:
rfan1 wrote:
PR is merged, Let me monitor the following commits to see if any issues.
Can you link the PR?
oops, I missed the PR https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/16213I don't change the logic for
yamllintCI checks for now, we can file a new ticket and check with the maintainer to see if we can do the same thing for it.Go for it, just let me know in the pull request.
Sure, I will ask Santiago's help to file a new ticket
Updated by szarate over 2 years ago
- Sprint changed from QE-Core: January Sprint (Jan 11 - Feb 08) to QE-Core: February Sprint (Feb 08 - Mar 08)
- Tags set to qe-core-february-sprint
- Status changed from Resolved to Feedback
Reopened during the Planning, Richard has some more ideas on where things can improve
Updated by rfan1 over 2 years ago
New PR for yaml check
https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/16356