-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Fix Android Tests in CI #22464
Fix Android Tests in CI #22464
Conversation
PR #22464: Size comparison from 383c416 to 3bcca58 Increases (8 builds for nrfconnect, psoc6, telink)
Decreases (2 builds for bl602, esp32)
Full report (46 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
Marking as draft for now - we have smoke tests and full tests ... I am unclear why full tests would not run. I think the original idea was that smoke tests should run (faster) on PRs and then full tests on master. |
https://github.com/project-chip/connectedhomeip/actions/workflows/full-android.yaml says workflow was disabled manually ... that is concerning. |
PR #22464: Size comparison from 383c416 to f1550be Increases (9 builds for bl602, cc13x2_26x2, nrfconnect, psoc6, telink)
Decreases (3 builds for cc13x2_26x2, esp32)
Full report (46 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
Accepting as is for now: this fixes checkout submodules at least, even though master CI may still fail (this was disabled for some time now ... still trying to figure out how long and when/why) |
* Enable android CI on push as well * Revert "Enable android CI on push as well" This reverts commit 3bcca58. * Add submodule checkout to full android: needed to set secure.directory * Distinct name for full android builds * Place the submodule checkout to the right location * Undo the wandalen retry removal Co-authored-by: Andrei Litvin <[email protected]>
* Fix Android Tests in CI (#22464) * Enable android CI on push as well * Revert "Enable android CI on push as well" This reverts commit 3bcca58. * Add submodule checkout to full android: needed to set secure.directory * Distinct name for full android builds * Place the submodule checkout to the right location * Undo the wandalen retry removal Co-authored-by: Andrei Litvin <[email protected]> * Fix android full build in CI (#22486) * Split out android builds to not use as much space during building * Ensure we can test android builds with act * Since we use checkout_submodules, submodules are not needed on the other checkout * Add some documentation on using act to test * Fix ENOBUFS being undefined when compiling tv-casting-app on arm Co-authored-by: Andrei Litvin <[email protected]>
* Enable android CI on push as well * Revert "Enable android CI on push as well" This reverts commit 3bcca58. * Add submodule checkout to full android: needed to set secure.directory * Distinct name for full android builds * Place the submodule checkout to the right location * Undo the wandalen retry removal Co-authored-by: Andrei Litvin <[email protected]>
Android CI was manually disabled.
When trying to re-enable it, it turns out it does not properly compile.
Fixes #22463