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

Move macOS WPT chunk 1 (only) from Buildbot to Taskcluster #22459

Merged
merged 7 commits into from Dec 19, 2018

Conversation

Projects
None yet
5 participants
@SimonSapin
Copy link
Member

SimonSapin commented Dec 14, 2018

This is only one chunk at a time so that we don’t need to double the number of worker nor the cycle time during the migration.

A release build takes ~40 minutes which seems worrying, but this also happens (sometimes) on Buildbot. For example: https://build.servo.org/builders/mac-rel-wpt1/builds/10401


This change is Reviewable

SimonSapin added some commits Dec 14, 2018

Tasckluster: add a second macOS worker
Tweak the Salt definition of /Library/LaunchAgents/net.generic.worker.plist
so that the `user` parameter for the template file is not taken
as a parameter for `file.managed`.
(The first run created a file owned by `worker` instead of `root`.)

@SimonSapin SimonSapin referenced this pull request Dec 14, 2018

Open

Meta: move CI from Buildbot to Taskcluster #22325

15 of 24 tasks complete
@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Dec 14, 2018

@bors-servo try=wpt-mac

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 14, 2018

⌛️ Trying commit 1fcf13c with merge 6e3d83d...

bors-servo added a commit that referenced this pull request Dec 14, 2018

Auto merge of #22459 - servo:tc-macos-wpt, r=<try>
Move macOS WPT chunk 1 (only) from Buildbot to Taskcluster

This is only one chunk at a time so that we don’t need to double the number of worker nor the cycle time during the migration.

A release build takes ~40 minutes which seems worrying, but this also happens (sometimes) on Buildbot. For example: https://build.servo.org/builders/mac-rel-wpt1/builds/10401

<!-- 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/22459)
<!-- Reviewable:end -->
@jdm

jdm approved these changes Dec 14, 2018

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 14, 2018

💔 Test failed - status-taskcluster

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Dec 14, 2018

[taskcluster:error] Uploading error artifact public/filtered-wpt-errorsummary.log from file repo/filtered-wpt-errorsummary.log with message "Could not read file '/Users/worker/tasks/task_1544801404/repo/filtered-wpt-errorsummary.log'", reason "file-missing-on-worker" and expiry 2019-12-14T14:47:58.994Z

[taskcluster:error] TASK FAILURE during artifact upload: file-missing-on-worker: Could not read file '/Users/worker/tasks/task_1544801404/repo/filtered-wpt-errorsummary.log'

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Dec 14, 2018

@bors-servo try=wpt-mac

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 14, 2018

⌛️ Trying commit ed4d122 with merge b804b95...

bors-servo added a commit that referenced this pull request Dec 14, 2018

Auto merge of #22459 - servo:tc-macos-wpt, r=<try>
Move macOS WPT chunk 1 (only) from Buildbot to Taskcluster

This is only one chunk at a time so that we don’t need to double the number of worker nor the cycle time during the migration.

A release build takes ~40 minutes which seems worrying, but this also happens (sometimes) on Buildbot. For example: https://build.servo.org/builders/mac-rel-wpt1/builds/10401

<!-- 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/22459)
<!-- Reviewable:end -->
@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 14, 2018

💔 Test failed - status-taskcluster

@SimonSapin SimonSapin force-pushed the tc-macos-wpt branch from ed4d122 to 7cd6d63 Dec 14, 2018

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Dec 14, 2018

@bors-servo try=wpt-mac

bors-servo added a commit that referenced this pull request Dec 14, 2018

Auto merge of #22459 - servo:tc-macos-wpt, r=<try>
Move macOS WPT chunk 1 (only) from Buildbot to Taskcluster

This is only one chunk at a time so that we don’t need to double the number of worker nor the cycle time during the migration.

A release build takes ~40 minutes which seems worrying, but this also happens (sometimes) on Buildbot. For example: https://build.servo.org/builders/mac-rel-wpt1/builds/10401

<!-- 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/22459)
<!-- Reviewable:end -->
@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 14, 2018

⌛️ Trying commit 7cd6d63 with merge a942a63...

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Dec 14, 2018

https://tools.taskcluster.net/groups/f1zynGgKQDe7PVq9ZQZ84w is green! (Hacked to not re-compile and to run fewer tests.)

@SimonSapin SimonSapin force-pushed the tc-macos-wpt branch from 7cd6d63 to e92e8cc Dec 14, 2018

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Dec 14, 2018

Removed the hacks, I think this is ready to land.

@jdm

This comment has been minimized.

Copy link
Member

jdm commented Dec 14, 2018

@SimonSapin SimonSapin force-pushed the tc-macos-wpt branch from e92e8cc to 0f3f253 Dec 17, 2018

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Dec 17, 2018

@bors-servo r=jdm

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 17, 2018

📌 Commit 0f3f253 has been approved by jdm

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 17, 2018

⌛️ Testing commit 0f3f253 with merge f0c0d50...

bors-servo added a commit that referenced this pull request Dec 17, 2018

Auto merge of #22459 - servo:tc-macos-wpt, r=jdm
Move macOS WPT chunk 1 (only) from Buildbot to Taskcluster

This is only one chunk at a time so that we don’t need to double the number of worker nor the cycle time during the migration.

A release build takes ~40 minutes which seems worrying, but this also happens (sometimes) on Buildbot. For example: https://build.servo.org/builders/mac-rel-wpt1/builds/10401

<!-- 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/22459)
<!-- Reviewable:end -->
@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 17, 2018

💔 Test failed - status-taskcluster

@CYBAI

This comment has been minimized.

Copy link
Collaborator

CYBAI commented Dec 18, 2018

@bors-servo retry

bors-servo added a commit that referenced this pull request Dec 18, 2018

Auto merge of #22459 - servo:tc-macos-wpt, r=jdm
Move macOS WPT chunk 1 (only) from Buildbot to Taskcluster

This is only one chunk at a time so that we don’t need to double the number of worker nor the cycle time during the migration.

A release build takes ~40 minutes which seems worrying, but this also happens (sometimes) on Buildbot. For example: https://build.servo.org/builders/mac-rel-wpt1/builds/10401

<!-- 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/22459)
<!-- Reviewable:end -->
@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 18, 2018

⌛️ Testing commit 0f3f253 with merge 13146d0...

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 18, 2018

💔 Test failed - status-taskcluster

@CYBAI

This comment has been minimized.

Copy link
Collaborator

CYBAI commented Dec 19, 2018

@bors-servo retry

{
    "status": "TIMEOUT", 
    "group": "default", 
    "message": "Test timed out", 
    "stack": null, 
    "subtest": "_parent should reuse window.parent context", 
    "test": "/html/browsers/windows/browsing-context-names/choose-_parent-003.html", 
    "line": 132914, 
    "action": "test_result", 
    "expected": "PASS"
}
{
    "status": "TIMEOUT", 
    "group": "default", 
    "message": null, 
    "stack": null, 
    "subtest": null, 
    "test": "/html/browsers/windows/browsing-context-names/choose-_parent-003.html", 
    "line": 132915, 
    "action": "test_result", 
    "expected": "OK"
}
@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 19, 2018

⌛️ Testing commit 0f3f253 with merge f3ca482...

bors-servo added a commit that referenced this pull request Dec 19, 2018

Auto merge of #22459 - servo:tc-macos-wpt, r=jdm
Move macOS WPT chunk 1 (only) from Buildbot to Taskcluster

This is only one chunk at a time so that we don’t need to double the number of worker nor the cycle time during the migration.

A release build takes ~40 minutes which seems worrying, but this also happens (sometimes) on Buildbot. For example: https://build.servo.org/builders/mac-rel-wpt1/builds/10401

<!-- 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/22459)
<!-- Reviewable:end -->
@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 19, 2018

@bors-servo bors-servo merged commit 0f3f253 into master Dec 19, 2018

2 of 3 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details

@SimonSapin SimonSapin deleted the tc-macos-wpt branch Dec 28, 2018

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