-
Notifications
You must be signed in to change notification settings - Fork 351
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
Fixing Windows CI build. #1143
Fixing Windows CI build. #1143
Conversation
Take a look at the build output here for the powershell script for the Windows stage: https://oe-jenkins.eastus.cloudapp.azure.com/blue/organizations/jenkins/Bors/detail/staging/289/pipeline/32 |
9c61a26
to
047fff9
Compare
bors try |
(The following comment is no longer valid to the current changes in this PR, but I'm leaving it around for historical purpsoses): Note that the extra commit here (deleting the "linuxbin" directory) is to get CI to pass on Windows. The Jenkins agent on Windows is not cleaning the Jenkins workspace before or after the build properly, even though it is configured to do so. That seems like a bug with the Jenkins agent. To remedy this, we now force remove the "linuxbin" directory before the build to make sure that it is wiped away. |
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.
Thanks for jumping on this, LGTM pending the bors run.
tryBuild failed |
Oh yeah, PowerShell doesn't usually report the exit code of native processes back to the process which spawned PowerShell. This bites everyone. |
You can do it in PowerShell, you just have to wrap the execution of the native binaries and instruct PowerShell to |
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.
But I still think using bat
in this way is simpler 😄
047fff9
to
08ba23d
Compare
bors try |
tryBuild failed |
Something is broken regarding the scripted Jenkinsfile and how it handles updates to the git repository and cleaning the workspace. On our Windows CI, very odd behavior has been seen.
These flags have been temporarily removed because Windows doesn't recognize them and our Windows CI has been broken for the last few days and ignoring builds that would otherwise fail.
08ba23d
to
e36ff75
Compare
bors try |
tryBuild failed |
bors try |
tryBuild failed |
e36ff75
to
42ef11e
Compare
bors try |
tryBuild succeeded |
bors r+ |
1143: Fixing Windows CI build. r=johnkord a=johnkord Right now Windows builds in CI are broken (the tests aren't getting run and powershell is erroring out, yet it believes it to be a successful build), and this PR will revert the changes that broke it. This PR temporarily disables the oeedgr8r tests on Windows, which needs to be re-enabled ASAP. Co-authored-by: John Kordich <johnkord@microsoft.com> Co-authored-by: Anita Govindarajan <v-angovi@microsoft.com>
Build failed |
bors r+ |
1143: Fixing Windows CI build. r=johnkord a=johnkord Right now Windows builds in CI are broken (the tests aren't getting run and powershell is erroring out, yet it believes it to be a successful build), and this PR will revert the changes that broke it. This PR temporarily disables the oeedgr8r tests on Windows, which needs to be re-enabled ASAP. Co-authored-by: John Kordich <johnkord@microsoft.com> Co-authored-by: Anita Govindarajan <v-angovi@microsoft.com>
Build failed |
bors r+ |
1143: Fixing Windows CI build. r=johnkord a=johnkord Right now Windows builds in CI are broken (the tests aren't getting run and powershell is erroring out, yet it believes it to be a successful build), and this PR will revert the changes that broke it. This PR temporarily disables the oeedgr8r tests on Windows, which needs to be re-enabled ASAP. Co-authored-by: John Kordich <johnkord@microsoft.com> Co-authored-by: Anita Govindarajan <v-angovi@microsoft.com>
Build succeeded |
Woo hoo! :) |
Great job John. Windows CI giving a False PASS wasn't good. |
1143: Fixing Windows CI build. r=johnkord a=johnkord Right now Windows builds in CI are broken (the tests aren't getting run and powershell is erroring out, yet it believes it to be a successful build), and this PR will revert the changes that broke it. This PR temporarily disables the oeedgr8r tests on Windows, which needs to be re-enabled ASAP. Co-authored-by: John Kordich <johnkord@microsoft.com> Co-authored-by: Anita Govindarajan <v-angovi@microsoft.com>
Right now Windows builds in CI are broken (the tests aren't getting run and powershell is erroring out, yet it believes it to be a successful build), and this PR will revert the changes that broke it.
This PR temporarily disables the oeedgr8r tests on Windows, which needs to be re-enabled ASAP.