Project

General

Profile

Actions

action #55643

closed

Prevent duplicate codecov reports

Added by okurz over 4 years ago. Updated over 4 years ago.

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

0%

Estimated time:

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 1 (0 open1 closed)

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

Actions
Actions #1

Updated by okurz over 4 years ago

Actions #2

Updated by okurz over 4 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"

Actions #3

Updated by okurz over 4 years ago

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

trying with

codecov:
  bot: "null"
Actions #4

Updated by okurz over 4 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

Actions

Also available in: Atom PDF