-
Notifications
You must be signed in to change notification settings - Fork 49
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
PACKAGE_INSTALLATION_FAILURE happening on some Android work items #597
Comments
It looks like this might be machine/device specific. Namely |
I logged in 4 machines to check. DNCENGWIN-50 and DNCENGWIN-120 were affected but they are ok now. DNCENGWIN-20 and DNCENGWIN-71 were not resolved. I see that there are adb server running on them and I think this is the root cause. It may not be killed since previous run so when we get this error we should try to kill it, then start and try to install apk. |
This is a nice find. Side note: In iOS we kill the So you should decide where you draw the line between what XHarness does and what is the desired machine state and put that in the layer that is Helix SDK. Maybe developers using XHarness don't want their ADB servers to be killed because they have some other things running or maybe they would actually appreciate. |
What if to kill adb only in case when |
Yeah, sure, makes sense then. I don't know the Android world much so just adding my two cents. If we already handle adb like this then it's good to do the same for this case as well |
I cannot reproduce it to test my fix, so will compare statistics after merge. I considered three types of solutions for this issue:
|
Version with fix 21275.2 |
From my observation today, this seems pretty flaky and hasn't yet impacted too many work items.
The failures occur on the Android arm64 (device) leg. Here are examples:
https://helixre107v0xdeko0k025g8.blob.core.windows.net/dotnet-runtime-refs-heads-main-8f7622f0765e44d792/System.ComponentModel.EventBasedAsync.Tests/console.6f6858ae.log?sv=2019-07-07&se=2021-06-02T19%3A06%3A59Z&sr=c&sp=rl&sig=sAw%2FaG5opryYuZsSLmO2g6m3SoYh3ip3Wo%2F9d24zYmw%3D
https://helixre107v0xdeko0k025g8.blob.core.windows.net/dotnet-runtime-refs-heads-main-8f7622f0765e44d792/System.Globalization.Extensions.Tests/console.23ae2187.log?sv=2019-07-07&se=2021-06-02T19%3A06%3A59Z&sr=c&sp=rl&sig=sAw%2FaG5opryYuZsSLmO2g6m3SoYh3ip3Wo%2F9d24zYmw%3D
The text was updated successfully, but these errors were encountered: