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

[Travis problem] Demonstrate double builds for travis, Do not merge #51

Closed
wants to merge 1 commit into from

Conversation

asmacdo
Copy link
Contributor

@asmacdo asmacdo commented Mar 25, 2019

re #4495

Please be sure you have read our documentation on creating PRs:
https://docs.pulpproject.org/en/3.0/nightly/contributing/pull-request-walkthrough.html

@codecov
Copy link

codecov bot commented Mar 25, 2019

Codecov Report

Merging #51 into master will decrease coverage by 0.33%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #51      +/-   ##
==========================================
- Coverage   66.39%   66.05%   -0.34%     
==========================================
  Files          65       65              
  Lines        2949     2949              
==========================================
- Hits         1958     1948      -10     
- Misses        991     1001      +10
Impacted Files Coverage Δ
pulpcore/content/handler.py 46.89% <0%> (-5.65%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3c63e42...beb54c4. Read the comment docs.

@asmacdo
Copy link
Contributor Author

asmacdo commented Mar 25, 2019

I've opened a new PR: #52 to workaround the CI freeze.

On this PR, you can see that Travis is performing 2 checks:

  1. https://github.com/pulp/pulpcore/pull/51/checks?check_run_id=84742379
  2. https://github.com/pulp/pulpcore/pull/51/checks?check_run_id=84742382

However, if you check each of these checks, they both refer to a single build:
https://travis-ci.com/pulp/pulpcore/builds/105747056

The build is green, but only one of the checks shows green, the other is permanently stuck yellow.

@dralley
Copy link
Contributor

dralley commented Mar 25, 2019

Travis y u do this :(

@asmacdo asmacdo changed the title Rename wsgi to match latest ansible-pulp [Travis problem] Demonstrate double builds for travis, Do not merge Mar 26, 2019
@asmacdo
Copy link
Contributor Author

asmacdo commented Mar 26, 2019

Closing since leaving it open is not actually helpful for solving the issue.

@asmacdo asmacdo closed this Mar 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants