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

Xbox One disconnects from test runner #4234

Open
joeyparrish opened this issue May 17, 2022 · 1 comment
Open

Xbox One disconnects from test runner #4234

joeyparrish opened this issue May 17, 2022 · 1 comment
Assignees
Labels
component: tests The issue involves our automated tests (generally; otherwise use a more specific component) platform: Xbox Issues affecting Xbox priority: P2 Smaller impact or easy workaround type: code health A code health issue
Milestone

Comments

@joeyparrish
Copy link
Member

The Xbox One in our lab disconnects from the test runner most of the time. The rate of this occurring is not consistent, but I have not been able to get 5 successful test runs in a row on Xbox. When the disconnect occurs, the device appears to briefly hang. There is no log to indicate what test was running at the time because log delivery to the test runner is not synchronous.

Through trial and error disabling certain subsets of the tests and doing 5 test passes, I found that disabling streaming_engine_unit.js alone is enough to get the tests to complete successfully. So the trigger is some or all of the tests in that file.

@joeyparrish joeyparrish added type: code health A code health issue priority: P2 Smaller impact or easy workaround component: tests The issue involves our automated tests (generally; otherwise use a more specific component) platform: Xbox Issues affecting Xbox labels May 17, 2022
@joeyparrish joeyparrish self-assigned this May 17, 2022
@github-actions github-actions bot added this to the Backlog milestone May 17, 2022
@joeyparrish
Copy link
Member Author

I found that excluding EMSG tests from streaming_engine_unit.js seems to make Xbox reliably complete the tests without disconnecting. Stopping emsg parsing right before calling box.reader.readBytes() also seems to do this.

Sadly, the problem goes away when I add console logs to readBytes(). So it looks like this is not going to be a straightforward problem to solve.

joeyparrish added a commit that referenced this issue Aug 31, 2022
Since we have been unable to solve #4234 (Xbox One disconnects from test runner), let's temporarily disable XboxOne from the lab to cut down on noise in our nightly tests.
joeyparrish added a commit that referenced this issue Aug 31, 2022
Since we have been unable to solve #4234 (Xbox One disconnects from test runner), let's temporarily disable XboxOne from the lab to cut down on noise in our nightly tests.
JulianDomingo pushed a commit that referenced this issue Oct 6, 2022
Since we have been unable to solve #4234 (Xbox One disconnects from test runner), let's temporarily disable XboxOne from the lab to cut down on noise in our nightly tests.
JulianDomingo pushed a commit that referenced this issue Oct 6, 2022
Since we have been unable to solve #4234 (Xbox One disconnects from test runner), let's temporarily disable XboxOne from the lab to cut down on noise in our nightly tests.
JulianDomingo pushed a commit that referenced this issue Oct 7, 2022
Since we have been unable to solve #4234 (Xbox One disconnects from test runner), let's temporarily disable XboxOne from the lab to cut down on noise in our nightly tests.
JulianDomingo pushed a commit that referenced this issue Oct 7, 2022
Since we have been unable to solve #4234 (Xbox One disconnects from test runner), let's temporarily disable XboxOne from the lab to cut down on noise in our nightly tests.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: tests The issue involves our automated tests (generally; otherwise use a more specific component) platform: Xbox Issues affecting Xbox priority: P2 Smaller impact or easy workaround type: code health A code health issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant