Project

General

Profile

action #122923

coordination #96596: [qe-core][CI] CI/CD and Coding style improvements

[qe-core][CI] Hide passing check messages from the CI logs

Added by szarate 2 months ago. Updated 16 days ago.

Status:
Resolved
Priority:
Normal
Assignee:
rfan1
Category:
Infrastructure
Target version:
QA - QE-Core: Ready
Start date:
2022-10-13
Due date:
% Done:

0%

Estimated time:
Difficulty:
Tags:
qe-core-february-sprint

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:

  1. Tidy checks.
  2. Metadata checks

  3. [yaml checks]

    Acceptance Criteria

  4. Passing checks done on files aren't shown unless --verbose or another flag is passed during the calls to make

See also: https://github.com/os-autoinst/os-autoinst/pull/2235 for silencing Tidy

Related issues

Copied from openQA Tests - action #118588: [qe-core][CI] Anotate output when there are code violations on github's PR.Resolved2022-10-13

History

#1 Updated by szarate 2 months ago

  • Copied from action #118588: [qe-core][CI] Anotate output when there are code violations on github's PR. added

#2 Updated by szarate 2 months 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

#3 Updated by szarate 2 months 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)

#4 Updated by szarate 2 months ago

  • Sprint changed from QE-Core: December Sprint (Dec 07 - Jan 11) to QE-Core: January Sprint (Jan 11 - Feb 08)

#5 Updated by rfan1 2 months ago

  • Assignee set to rfan1

#6 Updated by rfan1 2 months ago

  • Status changed from Workable to In Progress

#7 Updated by rfan1 about 1 month 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.

#8 Updated by rfan1 about 1 month ago

  • Status changed from Feedback to Resolved

#9 Updated by pdostal about 1 month 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 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.

Go for it, just let me know in the pull request.

#10 Updated by rfan1 about 1 month 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/16213

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.

Go for it, just let me know in the pull request.
Sure, I will ask Santiago's help to file a new ticket

#11 Updated by szarate about 1 month 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

#12 Updated by rfan1 about 1 month ago

  • Status changed from Feedback to In Progress

#14 Updated by rfan1 20 days ago

  • Status changed from In Progress to Resolved

#15 Updated by rfan1 16 days ago

  • Description updated (diff)

Also available in: Atom PDF