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: Disable functional tests on mac host #17240

Merged
merged 1 commit into from Oct 24, 2019

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Oct 24, 2019

Judging from the lack of responses to #15400 (comment), no one can reproduce the failures locally. Thus, disable the tests on the ci mac host. Otherwise they cause ci failures to be ignored or overwritten by a blind re-run.

@fanquake fanquake added the Tests label Oct 24, 2019
@laanwj
Copy link
Member

laanwj commented Oct 24, 2019

Are all tests flakey on MacOS or is it just the mentioned one (if so, we could consider skipping only that)?

Copy link
Contributor

@ryanofsky ryanofsky left a comment

Choose a reason for hiding this comment

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

Code review ACK fa71006. I've been blindly restarting these but can confirm it seemed like random test failures.

@laanwj
Copy link
Member

laanwj commented Oct 24, 2019

ok, ACK fa71006 in that case
can't help with testing locally

laanwj added a commit that referenced this pull request Oct 24, 2019
fa71006 ci: Disable functional tests on mac host (MarcoFalke)

Pull request description:

  Judging from the lack of responses to #15400 (comment), no one can reproduce the failures locally. Thus, disable the tests on the ci mac host. Otherwise they cause ci failures to be ignored or overwritten by a blind re-run.

ACKs for top commit:
  laanwj:
    ok, ACK fa71006 in that case
  ryanofsky:
    Code review ACK fa71006. I've been blindly restarting these but can confirm it seemed like random test failures.

Tree-SHA512: 805f552124409f6e79b16b419826d334307202b8e3636edc8a91dadffe9949554d924c00cb16bb09473ec1aaaf2d8eaf7a97c7c2b8c11bf6372135d39bba802b
@laanwj laanwj merged commit fa71006 into bitcoin:master Oct 24, 2019
@maflcko maflcko deleted the 1910-ciNoFunMac branch October 24, 2019 18:55
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Dec 16, 2021
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

4 participants