[ARM/CI] Fix arm32 CI git check failure #9798

Merged
merged 1 commit into from Feb 25, 2017

Projects

None yet

3 participants

@hqueue
Contributor
hqueue commented Feb 25, 2017 edited

Because layout in corefx binary has been changed recently, it causes arm CI failure.

This is a part of #9445 which contains huge change in arm CI.

BTW #9445 can not be merged yet, since discussion for new arm CI is not done at dotnet/core-setup#790

@hqueue
Contributor
hqueue commented Feb 25, 2017 edited

@jkotas PTAL. I hope this can fix another failure in arm32 CI. Sorry to bother, but I can't test it locally since netci.groovy can not be tested unless it is mereged.
I hope this can fix current problem. And it can be resolved completely with new arm32 CI #9445.

@hqueue hqueue changed the title from [ARM/CI] Update netci.groovy for arm CI to [ARM/CI] Fix arm32 CI failure Feb 25, 2017
@hqueue hqueue changed the title from [ARM/CI] Fix arm32 CI failure to [ARM/CI] Fix arm32 CI git check failure Feb 25, 2017
@hqueue hqueue [ARM/CI] Fix arm32 CI git check failure
Because layout in corefx binary has been changed, it causes arm CI failure.
And skip arm32 tests because test is not working recently.

Signed-off-by: Hyung-Kyu Choi <[email protected]>
9659d37
@jkotas jkotas merged commit 55382c5 into dotnet:master Feb 25, 2017

0 of 13 checks passed

Linux ARM Emulator Cross Debug Build Build finished.
Details
Linux ARM Emulator Cross Release Build Build finished.
Details
CentOS7.1 x64 Debug Build and Test Started.
Details
FreeBSD x64 Checked Build Started.
Details
OSX x64 Checked Build and Test Started.
Details
Ubuntu x64 Checked Build and Test Started.
Details
Ubuntu x64 Formatting Started.
Details
Windows_NT arm Cross Debug Build Started.
Details
Windows_NT arm Cross Release Build Started.
Details
Windows_NT x64 Debug Build and Test Started.
Details
Windows_NT x64 Formatting Started.
Details
Windows_NT x64 Release Priority 1 Build and Test Started.
Details
Windows_NT x86 Checked Build and Test Started.
Details
@jkotas
Member
jkotas commented Feb 25, 2017

@hqueue Thank you for working through this!

@hqueue
Contributor
hqueue commented Feb 25, 2017

@jkotas No problem :)
BTW does it take some time before netci.groovy is applied ? Because I found that CI still use old netci.groovy, and arm CI is still failing with old command.

@jkotas
Member
jkotas commented Feb 25, 2017

The updated netci.groovy is only picked up by new PRs. The usual trick is to close the PR and immediately reopen it - it will cause a new netci.groovy to be picked up.

I have done it in #9788. Unfortunately, it looks like it is still failing later. From https://ci.dot.net/job/dotnet_coreclr/job/master/job/arm_emulator_cross_debug_ubuntu_prtest/3312/console

11:29:10 ERROR: There are some uncommited changes. To avoid losing these changes commit them and try again.
11:29:10 
11:29:10 HEAD detached at fbc7e97
11:29:10 Untracked files:
11:29:10   (use "git add <file>..." to include
@hqueue
Contributor
hqueue commented Feb 25, 2017 edited

@jkotas I found that CI still used 00:01:31.455 + tar -xf ./bin/build.tar.gz command wihtou -C option for the failure. I will trigger arm CI agin for the #9788.

@hqueue
Contributor
hqueue commented Feb 25, 2017 edited

@jkotas I think arm32 CI works now. arm32 CI for #9788 have been completed :)
If you don't mind, can you rerun CI for other PRs too ?

@hqueue
Contributor
hqueue commented Feb 25, 2017

@jkotas I've re-triggered failed arm32 CI for other PRs. Thank you for merging quick fixes :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment