-
Notifications
You must be signed in to change notification settings - Fork 232
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
Flaky test: testTwoBPInstancesWithVideo #469
Comments
@RainNapper FYI since you worked on this feature. |
I think disabling it sounds good. I thought it was fixed in XC12, but it
looks to have gone back. This has been working in production, so I'm not
sure what's different about the test configuration.
…On Tue, Jan 19, 2021 at 12:07 PM Xiao Chen ***@***.***> wrote:
@RainNapper <https://github.com/RainNapper> FYI since you worked on this
feature.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#469 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AALB5OE4524Y6QW5EKINTXDS2XQ7ZANCNFSM4WJK2PKQ>
.
|
I wonder if it's just memory on the macOS VMs or some other resource that gets consumed... we could try running |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
testTwoBPInstancesWithVideo has been very flaky in the checks.
I briefly looked into it and have these findings
xcrun simctl io
, which could be pretty expensiveFor now I think we should disable it and try to root cause it in a branch.
The text was updated successfully, but these errors were encountered: