Project

General

Profile

Actions

action #119464

closed

The CI badge in https://github.com/os-autoinst/os-autoinst shows failed - cause: long commit msg size:M

Added by okurz almost 2 years ago. Updated almost 2 years ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
Regressions/Crashes
Target version:
Start date:
2022-10-26
Due date:
% Done:

0%

Estimated time:

Description

Observation

The CI badge in
https://github.com/os-autoinst/os-autoinst shows failed

Acceptance criteria

  • AC1: The CI badge shows passed

Suggestions

  • Follow the link behind the badge, review CI failures, fix them
  • Looks like the codecov stuff is broken and might be behind the errors

Related issues 1 (0 open1 closed)

Copied to openQA Project - action #120175: [sporadic]The CI badge in https://github.com/os-autoinst/os-autoinst shows failed - github <--> codecov api 404 size:MResolvedkraih2022-10-262022-11-29

Actions
Actions #1

Updated by jbaier_cz almost 2 years ago

Error: The maximum subject line length of 72 characters is exceeded.
Codecov: Failed to properly upload: The process '/home/runner/work/_actions/codecov/codecov-action/v2/dist/codecov' failed with exit code 255
Actions #2

Updated by livdywan almost 2 years ago

  • Subject changed from The CI badge in https://github.com/os-autoinst/os-autoinst shows failed to The CI badge in https://github.com/os-autoinst/os-autoinst shows failed size:M
  • Description updated (diff)
  • Status changed from New to Workable
Actions #3

Updated by dheidler almost 2 years ago

  • Status changed from Workable to In Progress
  • Assignee set to dheidler
Actions #4

Updated by dheidler almost 2 years ago

  • Status changed from In Progress to Resolved

Doesn't happen anymore.

I think this was due to https://github.com/os-autoinst/os-autoinst/actions/runs/3227670202

echo "Merge pull request #2193 from FroggyFlox/2190_clicktime_ignored_assert_and_click" | wc -c
81

So the automated merge commit message doesn't respect the commit message length.
So we simply shouldn't use too long branch names to avoid this, as I see no way to fix the merge commit message generation.

Another option would be to exclude merge commit from that checks.
There is already an issue open for this:
https://github.com/GsActions/commit-message-checker/issues/75

I opened https://github.com/os-autoinst/os-autoinst/pull/2199 which will only do anything if that PR for the testsuite repo got merged.

Actions #5

Updated by okurz almost 2 years ago

  • Status changed from Resolved to Feedback
Actions #6

Updated by dheidler almost 2 years ago

This is a different issue now.
Looks like some communication problems between codecov and github.
Did they change their API?
https://github.com/os-autoinst/os-autoinst/actions/runs/3395504590/jobs/5645471927

Actions #7

Updated by dheidler almost 2 years ago

  • Status changed from Feedback to Resolved

Seems to have been some kind of hickup.
Rerun works: https://github.com/os-autoinst/os-autoinst/actions/runs/3395504590/jobs/5713516952
Anyway out of scope of this ticket.
It it happens again, we should open a new ticket.

Actions #8

Updated by okurz almost 2 years ago

  • Copied to action #120175: [sporadic]The CI badge in https://github.com/os-autoinst/os-autoinst shows failed - github <--> codecov api 404 size:M added
Actions #9

Updated by okurz almost 2 years ago

I think copying tickets for the sake of it is not efficient but here you go: #120175

Actions #10

Updated by dheidler almost 2 years ago

  • Subject changed from The CI badge in https://github.com/os-autoinst/os-autoinst shows failed size:M to The CI badge in https://github.com/os-autoinst/os-autoinst shows failed - cause: long commit msg size:M
Actions

Also available in: Atom PDF