1157693 - Ability to exclude trees from being posted

Status: RESOLVED FIXED

(Developer Services :: Mercurial: bzpost, defect)

Description

[Andrew Halberstadt [:ahal]]

I don't want all my try pushes being recorded to bugs, especially when debugging automation only failures that require many pushes. There should be a way to exclude trees in the config, e.g:

[bzpost]
excludetrees = try,cedar

Comment 1

[Andrew Halberstadt [:ahal]]

Attachment: MozReview Request: bz://1157693/ahal

/r/7501 - Bug 1157693 - [bzpost] Add ability to exclude trees, r=gps

Pull down this commit:

hg pull -r 2efa4e174e3c8678ec029a95ab369c6ad0aad0d1 https://reviewboard-hg.mozilla.org/version-control-tools/

Comment 2

[Andrew Halberstadt [:ahal]]

Please note, I *haven't* run the test yet because I get the following failure:
(venv)[~/.mozbuild/version-control-tools]$ pyenv local 2.7.8
(venv)[~/.mozbuild/version-control-tools]$ ./run-tests hgext/bzpost
Traceback (most recent call last):
  File "./run-tests", line 178, in <module>
    with docker.auto_clean_orphans():
  File "/usr/lib64/python2.7/contextlib.py", line 19, in __enter__
    raise RuntimeError("generator didn't yield")
RuntimeError: generator didn't yield

So please either run the test on my behalf or help me figure out why my test environment isn't working.

I did test this manually though. I added `excludetrees = try` and the following push did not post anything to this bug:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=cc0b78e01d70

Comment 3

[Gregory Szorc [:gps]]

The issue in comment #2 is fixed in the tip of version-control-tools. That error stems from not having Docker running. Not having Docker means tests requiring Docker will be skipped.

Comment 4

[Gregory Szorc [:gps]]

Comment on attachment 8596583 [details]
MozReview Request: bz://1157693/ahal

https://reviewboard.mozilla.org/r/7499/#review6403

There is a minor test failure that I'll fix and land for you.

Thanks for the patch!

Comment 5

[Gregory Szorc [:gps]]

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

Comment 8

[Andrew Halberstadt [:ahal]]

Attachment: MozReview Request: Bug 1157693 - [bzpost] Add ability to exclude trees, r=gps