action #119464
closedThe CI badge in https://github.com/os-autoinst/os-autoinst shows failed - cause: long commit msg size:M
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
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
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
Updated by dheidler almost 2 years ago
- Status changed from Workable to In Progress
- Assignee set to dheidler
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.
Updated by okurz almost 2 years ago
- Status changed from Resolved to Feedback
https://github.com/os-autoinst/os-autoinst shows failed batch again
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
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.
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
Updated by okurz almost 2 years ago
I think copying tickets for the sake of it is not efficient but here you go: #120175
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