Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

codecov integration is broken #2997

Closed
asvetlov opened this issue May 14, 2018 · 23 comments
Closed

codecov integration is broken #2997

asvetlov opened this issue May 14, 2018 · 23 comments

Comments

@asvetlov
Copy link
Member

I've tried both bash script and codecov package from PyPI.
Both ways are failing on uploading data to codecov server.

Looks like it is a result of migration to travis-ci.com but I don't sure.

Any ideas?

@webknjaz
Copy link
Member

Show me the log link. I'd probably be able to fix it.
codecov script relies on certain env vars to identify whether it's running in CI and uses a shared token to upload reports, but this only happens if it identifies itself as running in a known CI env.

@webknjaz webknjaz self-assigned this May 14, 2018
@asvetlov
Copy link
Member Author

@webknjaz
Copy link
Member

It's weird, this https://codecov.io/gh/aio-libs/aiohttp/settings says that:

GitHub Integration is installed. However, this repository is not enabled.

Click here to your installation listings
Find Repository access and add aiohttp to the list.
May take a couple minutes for our system to get notices of new projects added.

And I don't have enough privileges to check/fix this. It suggests you to go to https://github.com/organizations/aio-libs/settings/installations/175267 and add aiohttp there. I don't have rights to do this.

@asvetlov
Copy link
Member Author

The page reports that codecov is enabled for all repos :(

@webknjaz
Copy link
Member

Try doing reset @ https://codecov.io/gh/aio-libs/aiohttp/settings

@webknjaz
Copy link
Member

It looks like you're right and codecov's backend supposingly tries to go to the old platform's API (.org) and fails (but that's just a guess).
Have you tried resetting it yet?

@asvetlov
Copy link
Member Author

Just did it.

Sorry, there was an issue: Authorization declined by github to create a webhook

@webknjaz
Copy link
Member

I was hoping that you'd not get that error, having better privileges 😞

@webknjaz
Copy link
Member

I've emailed them.

@asvetlov
Copy link
Member Author

I've tried to uninstall codecov and install it back again a couple days ago -- no luck.
Will do uninstalling now and install next day -- maybe the tool needs time to sync internal data.

@asvetlov
Copy link
Member Author

Thanks

@asvetlov
Copy link
Member Author

Recreated codecov project.
The history has gone away but nothing is fixed: https://travis-ci.com/aio-libs/aiohttp/jobs/124021867

@webknjaz
Copy link
Member

@asvetlov Let's try https://coveralls.io instead. It's functionally equivalent, lots of cool projects seem to use it and I just love their UI

@webknjaz
Copy link
Member

I've posted a question about whether they support new platform: lemurheavy/coveralls-public#1119

@asvetlov
Copy link
Member Author

The project used coveralls.io but switched too codecov for better UI/UX

@webknjaz
Copy link
Member

Anyway let's try it out while codecov doesn't work for us. They say we need:

service_name: travis-pro

in .coveralls.yml.

@asvetlov
Copy link
Member Author

Maybe we need to provide a secret key for codecov.
The documentation says that it is not needed for open sourced projects but maybe the tool assumes that .com means commercial support.

@webknjaz
Copy link
Member

yes, it looks so, but supposingly it shouldn't rely on that.
anyway, try doing that locally first.

@samuelcolvin
Copy link
Member

coveralls was much worse, no way to compare PRs, no diff coverage, harder to navigate code coverage.

@stevepeak has always been very helpful in the past for me. Perhaps he could help on this?

@stevepeak
Copy link

Looks like it's the new Travis-CI.org to .com switch. Let me see if we can patch that up quickly.

@stevepeak
Copy link

Waiting to get news from Travis-CI. Their old API endpoint on .org was public, which we used to confirm uploads. Now it's private on .com which broke our process and requiring the upload token.

@asvetlov
Copy link
Member Author

Fixed by the service provider

@lock
Copy link

lock bot commented Oct 28, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a [new issue] for related bugs.
If you feel like there's important points made in this discussion, please include those exceprts into that [new issue].
[new issue]: https://github.com/aio-libs/aiohttp/issues/new

@lock lock bot added the outdated label Oct 28, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 28, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants