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

Run a decision task on both TC deployments #24662

Merged
merged 4 commits into from Nov 6, 2019
Merged

Run a decision task on both TC deployments #24662

merged 4 commits into from Nov 6, 2019

Conversation

@SimonSapin
Copy link
Member

SimonSapin commented Nov 5, 2019

No description provided.

@highfive
Copy link

highfive commented Nov 5, 2019

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

SimonSapin commented Nov 5, 2019

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

bors-servo commented Nov 5, 2019

🙁 There is no try chooser try-wpt-mac for this repo, try one of: mac, magicleap, arm, android, wpt-android, wpt-mac, wpt, windows, linux
@SimonSapin
Copy link
Member Author

SimonSapin commented Nov 5, 2019

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

bors-servo commented Nov 5, 2019

⌛ Trying commit 8eff0ae with merge 6d5ea1a...
bors-servo added a commit that referenced this pull request Nov 5, 2019
@bors-servo
Auto merge of #24662 - servo:tc-urls, 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…
6d5ea1a 
Fix Taskcluster API URLs
@SimonSapin
Copy link
Member Author

SimonSapin commented Nov 5, 2019

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

bors-servo commented Nov 5, 2019

⌛ Trying commit 30acde9 with merge 54a76e7...
bors-servo added a commit that referenced this pull request Nov 5, 2019
@bors-servo
Auto merge of #24662 - servo:tc-urls, 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…
54a76e7 
Fix Taskcluster API URLs
@SimonSapin SimonSapin force-pushed the tc-urls branch from 30acde9 to 49b8dde Nov 5, 2019
@SimonSapin
Copy link
Member Author

SimonSapin commented Nov 5, 2019

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

bors-servo commented Nov 5, 2019

⌛ Trying commit 49b8dde with merge dd28f57...
bors-servo added a commit that referenced this pull request Nov 5, 2019
@bors-servo
Auto merge of #24662 - servo:tc-urls, 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…
dd28f57 
Fix Taskcluster API URLs
SimonSapin added 3 commits Nov 5, 2019
@SimonSapin
Always construct TC API URLs from $TASKCLUSTER_PROXY_URL or $TASKCLUS…
0870534 
…TER_ROOT_URL
@SimonSapin
Don’t fail tasks if a branch was fast-forward-pushed since the trigge…
7cf489c 
…ring push event
@SimonSapin
Update the decision task’s Docker image
694e38e 
servo/taskcluster-bootstrap-docker-images@47b7864...5a2588e
@SimonSapin
Copy link
Member Author

SimonSapin commented Nov 5, 2019

https://tools.taskcluster.net/groups/EkcR2QAMQEu0rVOhZcMCoQ is green.

(WPT tasks previously failed to download the release build artifact.)
@bors-servo
Copy link
Contributor

bors-servo commented Nov 5, 2019

☀️ Test successful - status-taskcluster
State: approved= try=True
@SimonSapin SimonSapin force-pushed the tc-urls branch from 49b8dde to 6068842 Nov 5, 2019
@SimonSapin SimonSapin changed the title Fix Taskcluster API URLs Run a decision task on both TC deployments Nov 5, 2019
@jdm
Copy link
Member

jdm commented Nov 5, 2019

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

bors-servo commented Nov 5, 2019

📌 Commit 6068842 has been approved by jdm
@highfive highfive assigned jdm and unassigned Manishearth Nov 5, 2019
@SimonSapin SimonSapin mentioned this pull request Nov 5, 2019
Merged
@SimonSapin
Copy link
Member Author

SimonSapin commented Nov 5, 2019

@bors-servo p=1
@bors-servo
Copy link
Contributor

bors-servo commented Nov 5, 2019

⌛ Testing commit 6068842 with merge e918dd6...
bors-servo added a commit that referenced this pull request Nov 5, 2019
@bors-servo
Auto merge of #24662 - servo:tc-urls, 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…
e918dd6 
Run a decision task on both TC deployments
@bors-servo
Copy link
Contributor

bors-servo commented Nov 5, 2019

💔 Test failed - linux-rel-wpt
@jdm
Copy link
Member

jdm commented Nov 5, 2019

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

bors-servo commented Nov 5, 2019

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

bors-servo commented Nov 5, 2019

⌛ Testing commit 6068842 with merge 6e2d6d0...
bors-servo added a commit that referenced this pull request Nov 5, 2019
@bors-servo
Auto merge of #24662 - servo:tc-urls, 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…
6e2d6d0 
Run a decision task on both TC deployments
armenzg added a commit to mozilla/treeherder that referenced this pull request Nov 5, 2019
@armenzg
Support ingesting a Github PR + refactor
Loading status checks…
f80a47c 
Now you need to specify the type of ingestion you want (hg-push or task)
and makes it simpler to separate which options are used for what.

To ingest an Hg push you can do this:

```bash
./manage.py ingest_push_and_tasks hg-push --project autoland --commit 1f17d60de61efeebaf9e4d04ce2abc4290179d00
```

To ingest a task you can do this:

```bash
./manage.py ingest_push_and_tasks task --task-id QGZSGHnaSRWRFk6TZlNMHQ
```

To ingest a Github PR you can do this:
```
./manage.py ingest_push_and_tasks pr --pr-url servo/servo#24662
```
@bors-servo
Copy link
Contributor

bors-servo commented Nov 5, 2019

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

jdm commented Nov 5, 2019 

Name                               Stmts   Miss  Cover   Missing
----------------------------------------------------------------
etc/taskcluster/decision_task.py     182      1    99%   29
@SimonSapin
Run a decision task on both deployments for push events
Loading status checks…
6125fe8
@SimonSapin SimonSapin force-pushed the tc-urls branch from 6068842 to 6125fe8 Nov 5, 2019
@SimonSapin
Copy link
Member Author

SimonSapin commented Nov 5, 2019

Eh, maybe I was a bit too eager with requiring 100% coverage :)

@bors-servo r=jdm
@bors-servo
Copy link
Contributor

bors-servo commented Nov 5, 2019

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

bors-servo commented Nov 5, 2019

⌛ Testing commit 6125fe8 with merge 748f10d...
bors-servo added a commit that referenced this pull request Nov 5, 2019
@bors-servo
Auto merge of #24662 - servo:tc-urls, 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…
748f10d 
Run a decision task on both TC deployments
@bors-servo
Copy link
Contributor

bors-servo commented Nov 6, 2019

☀️ Test successful - linux-rel-css, linux-rel-wpt, status-taskcluster
Approved by: jdm
Pushing 748f10d to master...
@bors-servo bors-servo merged commit 6125fe8 into master Nov 6, 2019
5 checks passed
5 checks passed
Community-TC (pull_request) TaskGroup: success
Details
Community-TC (push) TaskGroup: success
Details
Taskcluster (pull_request) TaskGroup: success
Details
Taskcluster (push) TaskGroup: success
Details
homu Test successful
Details
@bors-servo bors-servo deleted the tc-urls branch Nov 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jdm jdm

Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
@SimonSapin @highfive @bors-servo @jdm @Manishearth
You can’t perform that action at this time.