-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Disable testPackageManagerDefineAndXArgs #2769
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
Disable testPackageManagerDefineAndXArgs #2769
Conversation
This test fails spuriously but rather frequently on linux recently. SR-12902
@swift-ci Please test |
Thanks for this, @aschwaighofer. I was tearing my hair out trying to figure out how I’d somehow caused this error. It’s a relief to find out it’s not unique to my PR. |
@swift-ci Please test os x platform |
Disabling this test makes sense if it's generating noise, but I thought that it was an issue that could hit any test. Do we know if this test is doing something wrong that's triggering it, or is being affected by an underlying issue? |
I have only seen it affect this one test on 3 of my PRs and 2 or 3 other PRs that I looked at yesterday I think it still makes sense to disable this test until we have a fix if that reduces the number of failing PRs. |
@swift-ci Please test |
Great, thanks for the context. I absolutely agree that we should disable the test to avoid more confusion and wasted time, but wanted to understand if we needed to also fix the test. Hopefully if this test triggers it, then it might make it easier to figure out the underlying cause. Thanks! |
Do you have a JIRA for the underlying bug? I want to create a JIRA for enabling this test and block it on the underlying one. |
I don't have a bug for the underlying bug. @slavapestov Do you have one? |
@swift-ci Please clean test linux |
@swift-ci Please clean test os x |
@swift-ci Please clean test |
@swift-ci Please test |
Thanks. I created https://bugs.swift.org/browse/SR-12924 to reenable it, and will block that on the underlying JIRA when I find it. |
Merged to unblock CI |
This test fails spuriously but rather frequently on linux recently.
SR-12902