Project

General

Profile

action #55643

Prevent duplicate codecov reports

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

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Organisational
Target version:
-
Start date:
2019-08-12
Due date:
% Done:

0%

Estimated time:
Difficulty:

Description

Observation

in our pull requests it seems we are getting duplicate (or triple) reports from codecov, e.g. see https://github.com/os-autoinst/openQA/pull/2268#issuecomment-521982535 and two similar comments just below. I understand this can happen if both an old bot are activated along with the github app? As the bot runs under the account "coolo" we can look into this with him.


Related issues

Copied from openQA Project - coordination #55364: [epic] Let's make codecov reports reliableResolved2020-09-24

History

#1 Updated by okurz about 2 years ago

#2 Updated by okurz about 2 years ago

According to https://docs.codecov.io/docs/team-bot#section--github-users-why-am-i-getting-two-codecov-comments-on-my-pr-s- I thought in https://codecov.io/account/gh/os-autoinst/yaml I would need to write

codecov:
  bot: null

but this is not accepted, getting error: "bot must be string"

#3 Updated by okurz about 2 years ago

  • Status changed from New to Feedback
  • Assignee set to okurz

trying with

codecov:
  bot: "null"

#4 Updated by okurz almost 2 years ago

  • Status changed from Feedback to Resolved

The last did not really seem to help. However it seems we do not receive the separate codecov messages anymore at all for openQA since some recent changes regarding how we use circle CI. I first only observed that we do not receive duplicate ones anymore. However I never could make good use of them. The important part is the test details that still show the overall coverage and if the threshold is met, e.g. as seen in https://github.com/os-autoinst/openQA/pull/2462 . For os-autoinst we still have separate codecov reports but only single ones: https://github.com/os-autoinst/os-autoinst/pull/1257

Also available in: Atom PDF