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

ci: Re-run wine tests once if they fail #21138

Merged
merged 1 commit into from Feb 10, 2021
Merged

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Feb 10, 2021

Works around the intermittent wine issue: #21122 (comment)

@fanquake fanquake added the Tests label Feb 10, 2021
@laanwj laanwj changed the title ci: Re-run wine tests once if they fail ci: Re-run wine tests once if they fail Feb 10, 2021
@maflcko maflcko force-pushed the 2102-ciWin branch 3 times, most recently from 4ee29ad to faa8afe Compare February 10, 2021 11:10
@maflcko
Copy link
Member Author

maflcko commented Feb 10, 2021

I've reset the build 10 times and they all passed the unit tests.

I've added one bug to check that it is still caught: https://cirrus-ci.com/task/5411828982349824?command=ci#L573

Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK faa8afe - thanks for following up with this.

@fanquake fanquake merged commit cda9142 into bitcoin:master Feb 10, 2021
@maflcko maflcko deleted the 2102-ciWin branch February 10, 2021 12:00
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants