-
Notifications
You must be signed in to change notification settings - Fork 28.7k
Switch to Linux orchestrators for Windows releasers. #168941
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
Switch to Linux orchestrators for Windows releasers. #168941
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -500,54 +500,18 @@ targets: | ||
- cpu=x86 | ||
|
||
- name: Linux windows_android_aot_engine | ||
recipe: engine_v2/engine_v2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you deleting everything but the name because of the weird merged namespaces in ci.yaml we have with release builders?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NO, something else is happening here. Dont waste your time getting me up to speed.
autosubmit label was removed for flutter/flutter/168941, because - The status or check suite Windows module_host_with_custom_build_test has failed. Please fix the issues identified (or deflake) before re-applying this label.
|
CI had a failure that stopped further tests from running. We need to investigate to determine the root cause. SHA at time of execution: a79bc2c. Possible causes:
A blank commit, or merging to head, will be required to resume running CI for this PR. Error Details:
Stack trace:
|
Let's hold this PR until after the I/O stable |
I will wait until I come back from vacation to do this. |
@eyebrowsoffire Can you sign off (approve)? When I merge this, it will need to be CPd into beta/stable immediately. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Towards #168934.
/cc @reidbaker as release engineer
/cc @zanderso (we talked about this offline)