Closed
Bug 1277957
Opened 9 years ago
Closed 9 years ago
Update intree configs to no longer reference git.m.o
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(firefox50 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox50 | --- | fixed |
People
(Reporter: u429623, Assigned: jgriffin)
References
Details
Attachments
(1 file)
Moving into a separate bug from bug 1277838 comment #3, where :Callek noted:
>
> We do have a few things specifying git.m.o in tree though for tooltool:
>
> https://dxr.mozilla.org/mozilla-central/search?q=git.mozilla.org%2Fbuild%2Ftooltool&redirect=true
>
> Such as:
> https://dxr.mozilla.org/mozilla-central/source/testing/taskcluster/scripts/builder/install-packages.sh#10
And there may be non-tooltool reference as well.
Assignee | ||
Comment 1•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/60906/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/60906/
Attachment #8765626 -
Flags: review?(dustin)
Comment 2•9 years ago
|
||
Comment on attachment 8765626 [details]
Bug 1277957 - Point to tooltool at GitHub,
https://reviewboard.mozilla.org/r/60906/#review57736
Attachment #8765626 -
Flags: review?(dustin) → review+
Pushed by jgriffin@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/912ea9267130
Point to tooltool at GitHub, r=dustin
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → jgriffin
Backed out for failures like https://treeherder.mozilla.org/logviewer.html#?job_id=30727922&repo=mozilla-inbound
https://hg.mozilla.org/integration/mozilla-inbound/rev/7bcb335a9bc5
Flags: needinfo?(jgriffin)
Assignee | ||
Comment 5•9 years ago
|
||
Not sure what happened here, maybe Dustin can tell.
Flags: needinfo?(jgriffin) → needinfo?(dustin)
Comment 6•9 years ago
|
||
Those tasks are using tc-vcs to check out that repository, and that requires a primed cache to be in place first. I think greg's the resident expert in creating those caches.
Flags: needinfo?(dustin) → needinfo?(garndt)
Comment 7•9 years ago
|
||
New release has been published (and hooks updated) to include this repo:
https://github.com/taskcluster/taskcluster-vcs/releases/tag/v2.3.37
Manually created a task to cache it in the meantime:
https://tools.taskcluster.net/task-inspector/#e0vSm-UcSG2wMZyyiCLgyw/0
Flags: needinfo?(garndt)
Comment 8•9 years ago
|
||
Thanks greg! With that done, I think this patch can re-land (although it's worth a try push)
Assignee | ||
Comment 9•9 years ago
|
||
(In reply to Dustin J. Mitchell [:dustin] from comment #8)
> Thanks greg! With that done, I think this patch can re-land (although it's
> worth a try push)
TaskCluster doesn't pick up the failed jobs in a try push (see https://treeherder.mozilla.org/#/jobs?repo=try&revision=ef57a93e79b0), so I'll just have to try landing again.
Comment 10•9 years ago
|
||
Pushed by jgriffin@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/13fa96518284
Point to tooltool at GitHub, r=dustin
Comment 11•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/310e0872791410454b67ec5edebb49e654055cbf
Bug 1277957: fix tooltool URLs to not include .git; r=garndt a=bustage
Comment 12•9 years ago
|
||
bugherder |
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•