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

Test Android x86 on CI #21166

Merged
merged 3 commits into from Jul 12, 2018

Conversation

Projects
None yet
5 participants
@SimonSapin
Copy link
Member

SimonSapin commented Jul 12, 2018

Fixes #21124


This change is Reviewable

@highfive

This comment has been minimized.

Copy link

highfive commented Jul 12, 2018

Heads up! This PR modifies the following files:

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Jul 12, 2018

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Jul 12, 2018

⌛️ Trying commit 50affaf with merge c4a3092...

bors-servo added a commit that referenced this pull request Jul 12, 2018

Auto merge of #21166 - servo:android-x86, r=<try>
Test Android x86 on CI

Fixes #21124
@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Jul 12, 2018

💔 Test failed - mac-rel-css2

@SimonSapin SimonSapin force-pushed the android-x86 branch from 50affaf to 41cb270 Jul 12, 2018

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Jul 12, 2018

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Jul 12, 2018

⌛️ Trying commit 41cb270 with merge 72b1f5f...

bors-servo added a commit that referenced this pull request Jul 12, 2018

Auto merge of #21166 - servo:android-x86, r=<try>
Test Android x86 on CI

Fixes #21124

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/21166)
<!-- Reviewable:end -->
@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Jul 12, 2018

💔 Test failed - android-x86

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Jul 12, 2018

./mach clean-cargo-cache --keep 3 --force
[…]
"/home/servo/buildbot/slave/android-x86/build/python/_virtualenv/bin/python2.7" "-m" "pip" "install" "-I" "-r" "/home/servo/buildbot/slave/android-x86/build/tests/wpt/web-platform-tests/tools/wptrunner/requirements.txt" failed with error code -11:
Output:
Collecting html5lib==1.0.1 (from -r /home/servo/buildbot/slave/android-x86/build/tests/wpt/web-platform-tests/tools/wptrunner/requirements.txt (line 1))
  Using cached https://files.pythonhosted.org/packages/a5/62/bbd2be0e7943ec8504b517e62bab011b4946e1258842bc159e5dfde15b96/html5lib-1.0.1-py2.py3-none-any.whl
[…]
Collecting pycparser (from cffi>=1.7; platform_python_implementation != "PyPy"->cryptography>=1.3.4; python_version <= "2.7" and extra == "secure"->urllib3[secure]==1.22->-r /home/servo/buildbot/slave/android-x86/build/tests/wpt/web-platform-tests/tools/wptrunner/requirements.txt (line 5))
Installing collected packages: webencodings, six, html5lib, mozfile, mozinfo, blessings, mozterm, mozlog, mozdebug, idna, pycparser, cffi, enum34, asn1crypto, ipaddress, cryptography, pyOpenSSL, certifi, urllib3
Successfully installed asn1crypto-0.24.0 blessings-1.7 certifi-2018.4.16 cffi-1.11.5 cryptography-2.2.2 enum34-1.1.6 html5lib-1.0.1 idna-2.7 ipaddress-1.0.22 mozdebug-0.1 mozfile-1.2 mozinfo-0.10 mozlog-3.7 mozterm-1.0.0 pyOpenSSL-18.0.0 pycparser-2.18 six-1.11.0 urllib3-1.23 webencodings-0.5.1
Error:
program finished with exit code 1
elapsedTime=3.951225

:/

@jdm

This comment has been minimized.

Copy link
Member

jdm commented Jul 12, 2018

Pip segfaulting has been a problem on other linux build machines before. I've not yet figured out how to avoid it properly; upgrading pip made it go away but broke some other part of the build that I can't recall right now.

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Jul 12, 2018

This segfault is very reproducible on that machine:

$ gdb --args "/home/servo/buildbot/slave/android-x86/build/python/_virtualenv/bin/python2.7" "-m" "pip" "install" "-I" "-r" "/home/servo/buildbot/slave/android-x86/build/tests/wpt/web-platform-tests/tools/wptrunner/requirements.txt"
[…]
(gdb) where
#0  0x000000000049d684 in visit_decref () at ../Modules/gcmodule.c:360
#1  0x000000000049d805 in dict_traverse () at ../Objects/dictobject.c:2144
#2  0x000000000049c137 in subtract_refs () at ../Modules/gcmodule.c:385
#3  collect.lto_priv () at ../Modules/gcmodule.c:925
#4  0x00000000004f9b3e in PyGC_Collect () at ../Modules/gcmodule.c:1440
#5  0x00000000004f8e0f in Py_Finalize () at ../Python/pythonrun.c:448
#6  0x000000000051dc18 in Py_Exit (sts=sts@entry=0) at ../Python/pythonrun.c:1783
#7  0x000000000051b1b7 in handle_system_exit () at ../Python/pythonrun.c:1151
#8  0x000000000051aadd in PyErr_PrintEx () at ../Python/pythonrun.c:1161
#9  0x0000000000519a53 in RunModule.lto_priv.1256 () at ../Modules/main.c:194
#10 0x0000000000493b06 in Py_Main () at ../Modules/main.c:587
#11 0x00007ffff7810830 in __libc_start_main (main=0x4934c0 <main>, argc=7, argv=0x7fffffffe5a8, init=<optimized out>, fini=<optimized out>, rtld_fini=<optimized out>, stack_end=0x7fffffffe598) at ../csu/libc-start.c:291
#12 0x00000000004933e9 in _start ()
@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Jul 12, 2018

Looks like https://bugs.python.org/issue27542, but it’s closed as "works for me". The work-around proposed at https://bugs.python.org/issue27542#msg280612 does not help.

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Jul 12, 2018

Removing the virtualenv on the worker seems to fix this.

@bors-servo retry

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Jul 12, 2018

⌛️ Trying commit 41cb270 with merge 01f9240...

bors-servo added a commit that referenced this pull request Jul 12, 2018

Auto merge of #21166 - servo:android-x86, r=<try>
Test Android x86 on CI

Fixes #21124

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/21166)
<!-- Reviewable:end -->
@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Jul 12, 2018

💔 Test failed - android-x86

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Jul 12, 2018

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Jul 12, 2018

⌛️ Trying commit c1aaeb1 with merge e8e6840...

bors-servo added a commit that referenced this pull request Jul 12, 2018

Auto merge of #21166 - servo:android-x86, r=<try>
Test Android x86 on CI

Fixes #21124

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/21166)
<!-- Reviewable:end -->
@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Jul 12, 2018

Disabled pipefail for the yes program, despite what tidy might think.

@bors-servo r=jdm

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Jul 12, 2018

📌 Commit 877f76d has been approved by jdm

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Jul 12, 2018

⌛️ Testing commit 877f76d with merge 7b3d4cc...

bors-servo added a commit that referenced this pull request Jul 12, 2018

Auto merge of #21166 - servo:android-x86, r=jdm
Test Android x86 on CI

Fixes #21124

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/21166)
<!-- Reviewable:end -->
@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Jul 12, 2018

💔 Test failed - linux-dev

@SimonSapin SimonSapin force-pushed the android-x86 branch from 877f76d to bfb3f2f Jul 12, 2018

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Jul 12, 2018

I love you too, tidy.

@bors-servo r=jdm

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Jul 12, 2018

📌 Commit bfb3f2f has been approved by jdm

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Jul 12, 2018

⌛️ Testing commit bfb3f2f with merge 8af9409...

bors-servo added a commit that referenced this pull request Jul 12, 2018

Auto merge of #21166 - servo:android-x86, r=jdm
Test Android x86 on CI

Fixes #21124

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/21166)
<!-- Reviewable:end -->
@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Jul 12, 2018

💔 Test failed - mac-rel-wpt1

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Jul 12, 2018

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Jul 12, 2018

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Jul 12, 2018

@bors-servo bors-servo merged commit bfb3f2f into master Jul 12, 2018

3 of 5 checks passed

Taskcluster (pull_request) TaskGroup: failure
Details
Tidelift Checking Dependencies
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details

@SimonSapin SimonSapin deleted the android-x86 branch Jul 13, 2018

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