Remove remains of tc-vcs

RESOLVED FIXED in mozilla58

Status

RESOLVED FIXED
a year ago
9 months ago

People

(Reporter: glandium, Assigned: glandium)

Tracking

unspecified
mozilla58

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(6 attachments)

(Assignee)

Description

a year ago
AFAICT, tc-vcs is not used anymore. Time to get rid of the remains we have in the tree.
(Assignee)

Updated

a year ago
Depends on: 1383973
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 7

a year ago
mozreview-review
Comment on attachment 8919145 [details]
Bug 1409260 - Remove base-repository exception in .taskcluster.yml.

https://reviewboard.mozilla.org/r/190058/#review195382
Attachment #8919145 - Flags: review+

Comment 8

a year ago
mozreview-review
Comment on attachment 8919146 [details]
Bug 1409260 - Remove unused scripts from the desktop-build images.

https://reviewboard.mozilla.org/r/190060/#review195384
Attachment #8919146 - Flags: review+

Comment 9

a year ago
mozreview-review
Comment on attachment 8919147 [details]
Bug 1409260 - Remove mozharness tc-vcs support.

https://reviewboard.mozilla.org/r/190062/#review195386
Attachment #8919147 - Flags: review+

Comment 11

a year ago
mozreview-review
Comment on attachment 8919149 [details]
Bug 1409260 - Remove tc-vcs related files from docker images.

https://reviewboard.mozilla.org/r/190066/#review195390

::: taskcluster/docker/desktop-build/Dockerfile
(Diff revision 1)
>  MAINTAINER    Dustin J. Mitchell <dustin@mozilla.com>
>  
>  VOLUME /builds/worker/workspace
>  VOLUME /builds/worker/tooltool-cache
>  VOLUME /builds/worker/checkouts
> -VOLUME /builds/worker/.tc-vcs

Strictly speaking, the previous commit may have broken taskgraph until this commit because there is code that parses for VOLUME and ensures cache definitions are in sync. But as long as this all lands as a single push, we should be fine.
Attachment #8919149 - Flags: review+

Comment 12

a year ago
mozreview-review
Comment on attachment 8919150 [details]
Bug 1409260 - Remove comments about tc-vcs.

https://reviewboard.mozilla.org/r/190068/#review195392

Great cleanup!

FWIW my recollection was also that the android tasks were the last components using tc-vcs. Refactoring those was a bit difficult and I never had the will to do it. I'm glad nalexander tackled it as part of overhauling the android automation!

Also, with tc-vcs gone, I'm pretty sure we can remove node.js from some Docker images. Although with all the recent refactorings, I can't recall exactly where node.js is installed and what other functionality may be using it. IIRC there's a npm package related to tc-vcs that could almost certainly be removed without removing node.
Attachment #8919150 - Flags: review+
(Assignee)

Comment 13

a year ago
> IIRC there's a npm package related to tc-vcs that could almost certainly be removed without removing node.

There's one removed in taskcluster/docker/desktop1604-test/Dockerfile in one of the patches in this bug ;)
(Assignee)

Comment 14

a year ago
Also, it seems node is used for http/2 tests.

Comment 15

a year ago
mozreview-review
Comment on attachment 8919145 [details]
Bug 1409260 - Remove base-repository exception in .taskcluster.yml.

https://reviewboard.mozilla.org/r/190058/#review195504
Attachment #8919145 - Flags: review?(dustin) → review+

Comment 16

a year ago
mozreview-review
Comment on attachment 8919146 [details]
Bug 1409260 - Remove unused scripts from the desktop-build images.

https://reviewboard.mozilla.org/r/190060/#review195506
Attachment #8919146 - Flags: review?(dustin) → review+

Comment 18

a year ago
mozreview-review
Comment on attachment 8919148 [details]
Bug 1409260 - Remove tc-vcs caches.

https://reviewboard.mozilla.org/r/190064/#review195514

I'm grinning like a fool
Attachment #8919148 - Flags: review?(dustin) → review+

Comment 19

a year ago
mozreview-review
Comment on attachment 8919149 [details]
Bug 1409260 - Remove tc-vcs related files from docker images.

https://reviewboard.mozilla.org/r/190066/#review195516

::: taskcluster/docker/desktop1604-test/Dockerfile:76
(Diff revision 1)
>  RUN usermod -a -G video worker
>  
>  RUN mkdir Documents; mkdir Pictures; mkdir Music; mkdir Videos; mkdir artifacts
>  
> -# install tc-vcs and tc-npm-cache
> -RUN npm install -g taskcluster-vcs@2.3.12 \
> +# install tc-npm-cache
> +RUN  npm install -g taskcluster-npm-cache@1.1.14 \

I have no idea what this is, so best to leave it in. ..
Attachment #8919149 - Flags: review?(dustin) → review+

Comment 20

a year ago
mozreview-review
Comment on attachment 8919150 [details]
Bug 1409260 - Remove comments about tc-vcs.

https://reviewboard.mozilla.org/r/190068/#review195518
Attachment #8919150 - Flags: review?(dustin) → review+

Comment 21

a year ago
mozreview-review-reply
Comment on attachment 8919149 [details]
Bug 1409260 - Remove tc-vcs related files from docker images.

https://reviewboard.mozilla.org/r/190066/#review195516

> I have no idea what this is, so best to leave it in. ..

Hrm, I vaguely recall that this was a b2g thing.

Comment 22

a year ago
mozreview-review-reply
Comment on attachment 8919149 [details]
Bug 1409260 - Remove tc-vcs related files from docker images.

https://reviewboard.mozilla.org/r/190066/#review195516

> Hrm, I vaguely recall that this was a b2g thing.

taskcluster-vcs provided tc-vcs IIRC.

While I'm here, there's an outside chance that someone is relying on tc-vcs to perform Git-based automation on GitHub. I recall someone having such one-off automation ~1 year ago. Can't remember who. If that's still true, I suppose removing tc-vcs will shine a spotlight on them and enlighten all of us of their requirements.

Comment 23

a year ago
mozreview-review-reply
Comment on attachment 8919149 [details]
Bug 1409260 - Remove tc-vcs related files from docker images.

https://reviewboard.mozilla.org/r/190066/#review195516

> taskcluster-vcs provided tc-vcs IIRC.
> 
> While I'm here, there's an outside chance that someone is relying on tc-vcs to perform Git-based automation on GitHub. I recall someone having such one-off automation ~1 year ago. Can't remember who. If that's still true, I suppose removing tc-vcs will shine a spotlight on them and enlighten all of us of their requirements.

Ah, I was referring to taskcluster-npm-cache.

I also recall some git automation using it, but we should probably be able to check some of our org repos for the use of it.  We can also audit the index service to see if it's being hit for those paths.
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s 4841347dc20c -d 3915770a8869: rebasing 427140:4841347dc20c "Bug 1409260 - Remove base-repository exception in .taskcluster.yml. r=dustin,gps"
merging .taskcluster.yml
warning: conflicts while merging .taskcluster.yml! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 31

a year ago
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/autoland/rev/e325fb428095
Remove base-repository exception in .taskcluster.yml. r=dustin,gps
https://hg.mozilla.org/integration/autoland/rev/82781cec272f
Remove unused scripts from the desktop-build images. r=dustin,gps
https://hg.mozilla.org/integration/autoland/rev/b445d6f3372f
Remove mozharness tc-vcs support. r=dustin,gps
https://hg.mozilla.org/integration/autoland/rev/915021e13b67
Remove tc-vcs caches. r=dustin,gps
https://hg.mozilla.org/integration/autoland/rev/afb002c9c180
Remove tc-vcs related files from docker images. r=dustin,gps
https://hg.mozilla.org/integration/autoland/rev/4375412d40e3
Remove comments about tc-vcs. r=dustin,gps

Comment 33

a year ago
mozreview-review
Comment on attachment 8919147 [details]
Bug 1409260 - Remove mozharness tc-vcs support.

https://reviewboard.mozilla.org/r/190062/#review196078

lgtm
Attachment #8919147 - Flags: review?(jlund) → review+

Updated

9 months ago
Product: TaskCluster → Firefox Build System
You need to log in before you can comment on or make changes to this bug.