Skip to content

/ servo

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

Reduce intermittent failures for taskcluster #22493

Closed
wants to merge 7 commits into from
Closed

Reduce intermittent failures for taskcluster #22493

wants to merge 7 commits into from

Conversation

@jdm
Copy link
Member

jdm commented Dec 19, 2018 •
edited by SimonSapin

These changes have passed without any intermittent failures on four runs in #22482, which is a better track record than any other PRs at this point.

This change is Reviewable
@highfive
Copy link

highfive commented Dec 19, 2018

Heads up! This PR modifies the following files:
@jdm
Copy link
Member Author

jdm commented Dec 19, 2018

@bors-servo p=100 r+
@bors-servo
Copy link
Contributor

bors-servo commented Dec 19, 2018

📌 Commit 3be665f has been approved by jdm
@highfive highfive assigned jdm and unassigned paulrouget Dec 19, 2018
This was referenced Dec 19, 2018
Closed
Closed
Merged
Merged
@jdm jdm force-pushed the jdm-patch-27 branch from 3be665f to 9e7187c Dec 19, 2018
@servo-wpt-sync
Copy link
Collaborator

servo-wpt-sync commented Dec 19, 2018

Error syncing changes upstream. Logs saved in error-snapshot-1545193011502.
@jdm jdm force-pushed the jdm-patch-27 branch from 9e7187c to eab33fd Dec 19, 2018
@jdm
Copy link
Member Author

jdm commented Dec 19, 2018

@bors-servo r+
@bors-servo
Copy link
Contributor

bors-servo commented Dec 19, 2018

📌 Commit eab33fd has been approved by jdm
@bors-servo
Copy link
Contributor

bors-servo commented Dec 19, 2018

⌛ Testing commit eab33fd with merge e1cdf00...
bors-servo added a commit that referenced this pull request Dec 19, 2018
@bors-servo
Auto merge of #22493 - servo:jdm-patch-27, r=jdm
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
e1cdf00 
Reduce intermittent failures for taskcluster

These changes have passed without any intermittent failures on four runs in #22482, which is a better track record than any other PRs at this point.

<!-- 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/22493)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Dec 19, 2018

💔 Test failed - status-taskcluster
@CYBAI
Copy link
Collaborator

CYBAI commented Dec 19, 2018

@bors-servo retry

There're a bunch of tests under /eventsource/ CRASHed with incompatible library. It seems this is my first time to see this kind of failure. Let's try again!

  ▶ CRASH [expected OK] /eventsource/dedicated-worker/eventsource-close.htm
  │ 
  │ dyld: Library not loaded: /Users/worker/homebrew/opt/harfbuzz/lib/libharfbuzz.0.dylib
  │   Referenced from: /Users/worker/tasks/task_1545195545/repo/target/release/servo
  └   Reason: Incompatible library version: servo requires version 20201.0.0 or later, but libharfbuzz.0.dylib provides version 20102.0.0
@bors-servo
Copy link
Contributor

bors-servo commented Dec 19, 2018

💣 Failed to start rebuilding: Unknown error
@bors-servo
Copy link
Contributor

bors-servo commented Dec 19, 2018

⌛ Testing commit eab33fd with merge e6109e1...
@jdm
Copy link
Member Author

jdm commented Dec 21, 2018

@bors-servo retry
@bors-servo
Copy link
Contributor

bors-servo commented Dec 21, 2018

⌛ Trying commit ee0de01 with merge 36c8d3c...
bors-servo added a commit that referenced this pull request Dec 21, 2018
@bors-servo
Auto merge of #22493 - servo:jdm-patch-27, r=<try>
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
36c8d3c 
Reduce intermittent failures for taskcluster

These changes have passed without any intermittent failures on four runs in #22482, which is a better track record than any other PRs at this point.

<!-- 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/22493)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Dec 21, 2018

💔 Test failed - status-taskcluster
@jdm
Copy link
Member Author

jdm commented Dec 21, 2018

In the latest test failure, I have these logs:

{"source": "web-platform-tests", "test": "/html/semantics/forms/textfieldselection/selection-start-end-extra.html", "thread": "TestRunnerManager-4", "time": 1545411063750, "action": "test_start", "pid": 30}
{"source": "web-platform-tests", "thread": "TestRunnerManager-4", "time": 1545411063798, "action": "log", "message": "Got command: u'log'", "level": "DEBUG", "pid": 30}
{"thread": "TestRunnerManager-4", "process": "21309", "pid": 30, "source": "web-platform-tests", "command": "/repo/target/release/servo --hard-fail -u Servo/wptrunner -Z replace-surrogates -z http://web-platform.test:8000/html/semantics/forms/textfieldselection/selection-start-end-extra.html --user-stylesheet /repo/resources/ahem.css --certificate-path /tmp/tmpdPn8OY/cacert.pem", "time": 1545411063798, "action": "process_output", "data": "VMware, Inc."}
{"source": "web-platform-tests", "thread": "TestRunnerManager-4", "time": 1545411063799, "action": "log", "message": "Got command: u'log'", "level": "DEBUG", "pid": 30}
{"thread": "TestRunnerManager-4", "process": "21309", "pid": 30, "source": "web-platform-tests", "command": "/repo/target/release/servo --hard-fail -u Servo/wptrunner -Z replace-surrogates -z http://web-platform.test:8000/html/semantics/forms/textfieldselection/selection-start-end-extra.html --user-stylesheet /repo/resources/ahem.css --certificate-path /tmp/tmpdPn8OY/cacert.pem", "time": 1545411063799, "action": "process_output", "data": "softpipe"}
{"source": "web-platform-tests", "thread": "TestRunnerManager-4", "time": 1545411063799, "action": "log", "message": "Got command: u'log'", "level": "DEBUG", "pid": 30}
{"thread": "TestRunnerManager-4", "process": "21309", "pid": 30, "source": "web-platform-tests", "command": "/repo/target/release/servo --hard-fail -u Servo/wptrunner -Z replace-surrogates -z http://web-platform.test:8000/html/semantics/forms/textfieldselection/selection-start-end-extra.html --user-stylesheet /repo/resources/ahem.css --certificate-path /tmp/tmpdPn8OY/cacert.pem", "time": 1545411063799, "action": "process_output", "data": "softpipe"}
{"source": "web-platform-tests", "thread": "TestRunnerManager-4", "time": 1545411063799, "action": "log", "message": "Got command: u'log'", "level": "DEBUG", "pid": 30}
{"thread": "TestRunnerManager-4", "process": "21309", "pid": 30, "source": "web-platform-tests", "command": "/repo/target/release/servo --hard-fail -u Servo/wptrunner -Z replace-surrogates -z http://web-platform.test:8000/html/semantics/forms/textfieldselection/selection-start-end-extra.html --user-stylesheet /repo/resources/ahem.css --certificate-path /tmp/tmpdPn8OY/cacert.pem", "time": 1545411063799, "action": "process_output", "data": "3.3 (Core Profile) Mesa 18.3.0-devel"}
{"source": "web-platform-tests", "thread": "TestRunnerManager-4", "time": 1545411063915, "action": "log", "message": "Got command: u'log'", "level": "DEBUG", "pid": 30}
{"thread": "TestRunnerManager-4", "process": "21309", "pid": 30, "source": "web-platform-tests", "command": "/repo/target/release/servo --hard-fail -u Servo/wptrunner -Z replace-surrogates -z http://web-platform.test:8000/html/semantics/forms/textfieldselection/selection-start-end-extra.html --user-stylesheet /repo/resources/ahem.css --certificate-path /tmp/tmpdPn8OY/cacert.pem", "time": 1545411063915, "action": "process_output", "data": "[2018-12-21T16:51:03Z INFO  net::http_loader] request for http://web-platform.test:8000/html/semantics/forms/textfieldselection/selection-start-end-extra.html (GET)"}
{"source": "web-platform-tests", "thread": "TestRunnerManager-4", "time": 1545411078783, "action": "log", "message": "Got command: 'test_ended'", "level": "DEBUG", "pid": 30}
{"source": "web-platform-tests", "thread": "TestRunnerManager-4", "time": 1545411078785, "action": "log", "message": "Unexpected count in this thread 4", "level": "DEBUG", "pid": 30}
{"status": "TIMEOUT", "thread": "TestRunnerManager-4", "extra": {"test_timeout": 10}, "pid": 30, "source": "web-platform-tests", "test": "/html/semantics/forms/textfieldselection/selection-start-end-extra.html", "time": 1545411078785, "action": "test_end", "expected": "OK"}

As far as I can tell, Servo requested the test file and never received a response from the WPT server before the test timeout elapsed.
@jdm
Add hyper logging.
Loading status checks…
85f2d1e
@jdm
Copy link
Member Author

jdm commented Dec 21, 2018

@bors-servo try=wpt
@jdm
Copy link
Member Author

jdm commented Dec 21, 2018

@bors-servo ping
@bors-servo
Copy link
Contributor

bors-servo commented Dec 21, 2018

😪 I'm awake I'm awake
@jdm
Copy link
Member Author

jdm commented Dec 21, 2018

@bors-servo try=wpt
@bors-servo
Copy link
Contributor

bors-servo commented Dec 21, 2018

⌛ Trying commit 85f2d1e with merge 3b1c4df...
bors-servo added a commit that referenced this pull request Dec 21, 2018
@bors-servo
Auto merge of #22493 - servo:jdm-patch-27, r=<try>
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
3b1c4df 
Reduce intermittent failures for taskcluster

These changes have passed without any intermittent failures on four runs in #22482, which is a better track record than any other PRs at this point.

<!-- 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/22493)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Dec 21, 2018

💔 Test failed - status-taskcluster
@jdm
Copy link
Member Author

jdm commented Dec 21, 2018

@bors-servo retry
@bors-servo
Copy link
Contributor

bors-servo commented Dec 21, 2018

⌛ Trying commit 85f2d1e with merge e3605ab...
bors-servo added a commit that referenced this pull request Dec 21, 2018
@bors-servo
Auto merge of #22493 - servo:jdm-patch-27, r=<try>
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
e3605ab 
Reduce intermittent failures for taskcluster

These changes have passed without any intermittent failures on four runs in #22482, which is a better track record than any other PRs at this point.

<!-- 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/22493)
<!-- Reviewable:end -->
@jdm
More processes!
Loading status checks…
36465f1
@jdm
Copy link
Member Author

jdm commented Dec 21, 2018

@bors-servo try=wpt
@bors-servo
Copy link
Contributor

bors-servo commented Dec 21, 2018

⌛ Trying commit 36465f1 with merge 22a7440...
bors-servo added a commit that referenced this pull request Dec 21, 2018
@bors-servo
Auto merge of #22493 - servo:jdm-patch-27, r=<try>
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
22a7440 
Reduce intermittent failures for taskcluster

These changes have passed without any intermittent failures on four runs in #22482, which is a better track record than any other PRs at this point.

<!-- 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/22493)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Dec 21, 2018

💔 Test failed - status-taskcluster
@bors-servo
Copy link
Contributor

bors-servo commented Dec 22, 2018

☔ The latest upstream changes (presumably #22354) made this pull request unmergeable. Please resolve the merge conflicts.
@jdm jdm removed the S-awaiting-review label Feb 1, 2019
@jdm jdm closed this Nov 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SimonSapin SimonSapin

Assignees

@jdm jdm

Labels
S-needs-rebase S-tests-failed
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

8 participants
@jdm @highfive @bors-servo @servo-wpt-sync @CYBAI @KiChjang @SimonSapin @paulrouget
You can’t perform that action at this time.