-
Notifications
You must be signed in to change notification settings - Fork 27.3k
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
Google testing failure missing triage info #134668
Comments
@XilaiZhang Could you help take a look? |
This is concerning as the only solution offered is to ask a Googler to investigate or follow the solutions below:
Can we revert this functionality at this time? |
Marking as P1 considering high impact but there is a workaround to reset the test status considering this is a false alarm. |
sent cl/565087659 for review and put more context in #134637 (comment) |
yes it would work even if its deleted and theres no cl |
I tried overriding the Google testing status in FroB and it worked temporarily (said it was passing in the Github checks), but when it tried to autosubmit, it failed with message
|
umm i tried the same with autosubmit but for some reasons autosubmit merged it. Would be great if you could file a bug and document what happened. thank you! |
Latency in reporting the status? |
It went from red -> green -> red again? and persisted red, until Xilai overrode it a second time from FRoB. |
I did not know that. |
This happened again on #134590 cc @XilaiZhang |
I will file a separate issue for the check going from green back to red: #134684 public, with cross-link to Google-internal bug. |
Isn't that part of this issue? Also what is the work around here then? Do you need to go on Frob and toggle the status to land the PR? I am assuming you do not since you can squash and merge in flutter. |
If you're referring to #134684, it is not obvious to me that it is the same issue as this one. I will defer to you however if you think it is the same issue. |
@christopherfujino no I am fine with this, I thought the actual issue was the one you pointed out. My mistake I thought there were two issues beyond the missing triage info. Please keep it open. |
Hi @XilaiZhang, any updates on this? Just caught up on the thread |
Yeah i sent updates in the PSA on Saturday |
Gotcha. Thanks! I will close this, but feel free to reopen if necessary. |
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 |
Example RP: #134637
By clicking
details
, here is the check run page:https://github.com/flutter/flutter/pull/134637/checks?check_run_id=16761209576
The View more details on Flutter Roll on Borg however directs to the wiki page as well.
Not sure how to debug this failure.
The text was updated successfully, but these errors were encountered: