Report not loading

s0undt3ch · October 12, 2019, 11:23am

Anyone knows why the following PR report isn’t even loading?

drazisil · October 12, 2019, 11:39am

I think this is a bug where we time out when there is a large number of files in the diff. Can you please share the commit sha from the head?

s0undt3ch · October 13, 2019, 6:08pm

Yes, there is.

This is a work in progress so the SHA keeps changing as I move along.

The tip SHA is currently Skip tests which are not supposed to pass under windows. · saltstack/salt@a5aade4 · GitHub

s0undt3ch · October 16, 2019, 7:01am

At the bare minimum we should get a timeout message of some sort, however, I’d really like to be able to see the impact my PR is causing while working on it.

Any ETA for any of the above?

drazisil · October 23, 2019, 4:03pm

It’s should be fixed in our new frontend, which is coming “soon”

If GitHub can’t process it, they may not even be returning the data to us. Do you happen to have a newer commit SHA? The above one appears to be over 2 weeks old and isn’t in the logs anymore.

s0undt3ch · October 26, 2019, 7:21pm

Here’s another one:

This one doesn’t change any “coverable” code, and also doesn’t load.

Current HEAD of the PR:

Also, what do you mean, if GH can’t process it?

s0undt3ch · October 27, 2019, 10:06am

Another one which does not load:

https://codecov.io/github/saltstack/salt/commit/2690f0ecfdfe5b13a3c62e717714a9f0a30fcc9f

drazisil · October 30, 2019, 3:35pm

I was referring to to GitHub saying the diff has too many files to show.

It’s timing out for us in displaying. Why is the diff so large, is there a way to make it smaller?

s0undt3ch · October 30, 2019, 9:18pm

Unfortunately, these are big PRs(can’t be smaller), at least the ones linked, and yes on at least one GH warns it won’t display the whole diff.

s0undt3ch · November 1, 2019, 7:53am

This is now the HEAD of our master branch, and we can’t load the report.

Do we have something wrong with our configuration?

github.com

saltstack/salt/blob/master/.codecov.yml

codecov:
  ci:
    - jenkins.saltproject.io
    - github.com

  max_report_age: 24           # The age you want coverage reports to expire at, or if you
                               # want to disable this check. Expired reports will not be processed by codecov.
  require_ci_to_pass: yes      # Less spammy. Only notify on passing builds.


#  notify:
#    after_n_builds: 25           # Only notify after N builds
#                                 # This value is the output of:
#                                 #    sh -c 'echo "$(ls .ci/ | grep kitchen | wc -l)"'
#    wait_for_ci: yes             # Should Codecov wait for all CI statuses to complete before sending ours.
#                                 # Note: Codecov considers all non-codecov statues to be CI statuses


ignore:
  - ^*.py$                       # python files at the repo root, ie, setup.py

This file has been truncated. show original

jerrod · November 1, 2019, 6:46pm

@s0undt3ch some of these uploads (in the hundreds) are definitely stretching our current processing ability, hence why you are seeing timeouts. Directionally, we are making good progress on expanding our ability to process larger and larger sets of uploads, but it will take time.

I don’t see anything actively wrong with your configuration. You can always validate the YAML here: Validate the yaml

cat codecov.yml | curl --data-binary @- https://codecov.io/validate

Use the shell script above to post your yaml to Codecov to be checked for syntax and layout issues.

Topic		Replies	Views
Coverage Igrnored? Support python	4	361	October 30, 2019
GitHub codecov check in pull request suddenly stuck Support github	2	438	July 17, 2020
Specific pull request report hanging/causing internal server error Bug Fixes	7	500	March 2, 2020
Gh PR comment not updating Support github	5	504	November 24, 2020
All repos in organization are stuck on 'Waiting for CI to complete' for ~week Support golang , github	2	1355	February 28, 2020

Report not loading

Related topics