-
Notifications
You must be signed in to change notification settings - Fork 28.6k
Linux_samsung_a02 openpay_benchmarks__scroll_perf
keeps failing
#108391
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
Comments
Seems like the error is:
|
@keyonghan looks like pub is failing to resolve dependencies from github. This works for me locally. Are we allowed to clone from github.com in the LUCI bots? @dnfield if we are not allowed to clone from github.com, is |
Its not published yet, but we could publish a new version to work around this issue |
Our bots are not configured to clone from github.com by default:
|
@jonahwilliams could you please publish @keyonghan in that case I'll wait for the package to be published and update |
I've published 0.0.1 for all vector graphics packages, you should be able to update the pubspec to reference these. There may be an additional git constraint for visibility_detector, but that can be removed as long as the package is updated to the latest version |
Updated OpenPay here: https://dash-internal-review.googlesource.com/c/openpay/+/1080, @keyonghan once this lands can we rerun one of the jobs and see if the issue gets resolved? |
With the above CL, the test still fails: https://ci.chromium.org/ui/p/flutter/builders/staging/Linux_samsung_a02%20openpay_benchmarks__scroll_perf/209/overview, but it is not clear to me what the error is as we are suppressing logs. |
@keyonghan is there a way to enable the logs for some users? to enable debugging this? |
Here is one led run with logs: https://luci-milo.appspot.com/raw/build/logs.chromium.org/flutter/led/keyonghan_google.com/43a322025aa9e31c86541b291f1490ca2b3361cd40c872c22100a45d0547ed75/+/build.proto
|
https://dash-internal-review.googlesource.com/c/openpay/+/1140 fixes this error. |
@keyonghan looks like it's still failing :(.
Thanks. |
Commenting out https://cs.opensource.google/flutter/recipes/+/main:recipes/devicelab/devicelab_drone.py;l=78 will enable the logs. Then you can run led based on the local changes.
You can try replacing
|
I finally got around to trying it, I modified the properties of an existing builder and added the relevant bits to checkout openpay. I'm running into errors like so:
Maybe I need some permissions to launch tasks via CLI in @keyonghan could you please help me with this? |
Try is expected to work. How about staging? Could you share the command? |
I used
To
|
Line 26 is still using You can try:
|
I did change line 26 to try before running it :), still failed. update: it works now! |
https://dash-internal-review.googlesource.com/c/openpay/+/1160 should likely fix this issue, but I found another one while debugging this. @keyonghan looks like |
We should be good on that. Approved https://flutter-review.googlesource.com/c/recipes/+/32340. |
@keyonghan , this test seems to succeed now, but its still flaky on one of the bots: https://paste.googleplex.com/5051984135585792 Any reason why the URL in the above mentioned logs wouldn't resolve? |
There is no difference between linux18 and linux27. I would wait for a couple of runs to see if this is a network flake. |
The test is failing consistently again on both linux18 and linux27: https://ci.chromium.org/p/flutter/builders/staging/Linux_samsung_a02%20openpay_benchmarks__scroll_perf |
Linux_samsung_a02 openpay_benchmarks__scroll_perf
keeps timing out Linux_samsung_a02 openpay_benchmarks__scroll_perf
keeps failing
This is still failing every commit, no logs though: |
@iskakaushik Shall we skip the test from CI before it is ready? |
@keyonghan yes, that sounds good to me |
@iskakaushik created PR to skip the test for now here #111415 |
@iskakaushik Do we still need this test? |
Closing as obsolete. |
This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of |
https://ci.chromium.org/p/flutter/builders/staging/Linux_samsung_a02%20openpay_benchmarks__scroll_perf
@iskakaushik could you help take a look? This has been causing testbed quarantine: https://chromium-swarm.appspot.com/bot?id=flutter-devicelab-linux-18 (/cc on call @godofredoc )
The text was updated successfully, but these errors were encountered: