1116861 - Implement |hg fxtrees| to pull from every label that |hg fxheads| knows about

Status: RESOLVED FIXED

(Developer Services :: Mercurial: firefoxtree, defect)

Description

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

It'd be nice to have some command that could pull all of the labels that fxheads knows about, so that one command can get all of the new incoming revisions from all of the known branches.

Comment 1

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

This should NOT pull from everything that https://hg.mozilla.org/hgcustom/version-control-tools/file/default/pylib/mozautomation/mozautomation/repository.py knows about, only the ones that have actively been pulled from in the past and are listed by the fxheads command.

Comment 2

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Attachment: WIP v1

This is about as far as I've come implementing something like this. 

My python-foo and mercurial-extension-writing-foo is weak, so I'm unsure how to use the output of `hg fxheads` inside of this new command, so I've just hardcoded an array of the heads I care about in my day to day job.

Comment 3

[Gregory Szorc [:gps]]

Comment on attachment 8544254 [details]
WIP v1

Congratulations on writing your first Mercurial extension!

But as you said, you want some more advanced functionality than what you've implemented. I hope you don't mind if I bake up something more robust and integrate it as part of firefoxtree.

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Fine with me. I'd also not mind working through to the final product myself, but that's probably more hand-holding than we'd want to get this done anytime soon.

Comment 5

[Gregory Szorc [:gps]]

Attachment: MozReview Request: bz://1116861/gps

Comment 6

[Gregory Szorc [:gps]]

/r/2047 - firefoxtree: allow pulling from aliases that map to multiple trees (bug 1116861)

Pull down this commit:

hg pull review -r f45c5587f836d8139b3cac72bda7b08c7cd50f7e

Comment 7

[Ed Morley [:emorley]]

https://reviewboard.mozilla.org/r/2045/#review1291

::: pylib/mozautomation/mozautomation/repository.py
(Diff revision 1)
>      'integration': ('inbound', 'fx-team', 'services', 'build', 'b2ginbound'),

I'd quite like to remove services and build from the list of integration repos (just the multi repo alias, not their entries entirely) - they are dormant and removed from buildbot/treeherder. 

This would mean sheriffs could do |hg pull integration| and actually get only the main repos. Plus it would mean fewer repos have to get cloned as part of the version-control-tools test run, though it's probably a drop in the ocean for run-time.

Is up to you :-)

Comment 8

[Ed Morley [:emorley]]

https://reviewboard.mozilla.org/r/2045/#review1293

Ship It!

Comment 9

[Gregory Szorc [:gps]]

https://reviewboard.mozilla.org/r/2045/#review1301

> I'd quite like to remove services and build from the list of integration repos (just the multi repo alias, not their entries entirely) - they are dormant and removed from buildbot/treeherder. 
> 
> This would mean sheriffs could do |hg pull integration| and actually get only the main repos. Plus it would mean fewer repos have to get cloned as part of the version-control-tools test run, though it's probably a drop in the ocean for run-time.
> 
> Is up to you :-)

Send me a patch to remove them and I'll r+ it.

Comment 10

[Ed Morley [:emorley]]

The only reason I mentioned it here is because you added a bunch of clones for those dormant repos to firefoxrepos.sh, presumably as test fixup - when the alternative fix would be to remove them from the alias. Again, don't mind either way :-)

Comment 11

[Gregory Szorc [:gps]]

https://hg.mozilla.org/hgcustom/version-control-tools/rev/435f3e909575

Comment 12

[Gregory Szorc [:gps]]

I'll file a new bug to remove build-system and services-central from the integration list.

Comment 13

[Gregory Szorc [:gps]]

https://hg.mozilla.org/hgcustom/version-control-tools/rev/ec56399bb1c8

Comment 14

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Cleaning up the title to better reflect what actually landed.

Comment 15

[Ed Morley [:emorley]]

(In reply to Wes Kocher (:KWierso) from comment #14)
> Cleaning up the title to better reflect what actually landed.

That's not what landed - see:
https://hg.mozilla.org/hgcustom/version-control-tools/rev/ec56399bb1c8#l1.15

Comment 17

[Gregory Szorc [:gps]]

Attachment: MozReview Request: firefoxtree: allow pulling from aliases that map to multiple trees (bug 1116861)