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

Fix wheel deploys under CI. #8188

Merged
merged 1 commit into from Aug 21, 2019

Conversation

@jsirois
Copy link
Member

commented Aug 20, 2019

No description provided.

@jsirois

This comment has been minimized.

Copy link
Member Author

commented Aug 20, 2019

Morning email:
image

Logs showed:
image

@stuhood
Copy link
Member

left a comment

Thank you @jsirois!

@cosmicexplorer
Copy link
Contributor

left a comment

Thanks a lot for showing how you diagnosed this!

@Eric-Arellano
Copy link
Contributor

left a comment

Thanks!

@stuhood

This comment has been minimized.

Copy link
Member

commented Aug 20, 2019

Hm. One of your shards was segfaulting left and right in a bunch of tests. Have seen that before, but haven't identified any sort of useful pattern to which tests it occurs under. In this case, it felt environmental. Restarted.

@jsirois

This comment has been minimized.

Copy link
Member Author

commented Aug 20, 2019

Hm. One of your shards was segfaulting left and right ...

Sortof thanks - I was digging on that but had not captured logs yet. If it reds again similarly, I'll handle.

@stuhood

This comment has been minimized.

Copy link
Member

commented Aug 20, 2019

It recurred, which leads me to believe that might have something to do with the cache entri(es)?

@jsirois : While I think it would be great to figure out what's going on with this shard, how would you feel about merging this to unblock consumption of wheels? The nightly jobs that consume these provide useful sanity checking that can catch cases that don't have upstream tests yet.

@stuhood

This comment has been minimized.

Copy link
Member

commented Aug 21, 2019

Merging to fix master. Will leave the branch and failure where they are in case that allows for a useful repro.

@stuhood stuhood merged commit 7346b0b into pantsbuild:master Aug 21, 2019

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.