Add a button to auto-post csets to bugs

RESOLVED INCOMPLETE

Status

Tree Management Graveyard
TBPL
RESOLVED INCOMPLETE
5 years ago
2 years ago

People

(Reporter: cjones, Unassigned)

Tracking

Details

Attachments

(1 attachment)

I find it rather tiresome to manually post these.  (I wrote "List changeset URLs" to solve a similar problem.)

But now that tbpl can talk to bugzilla, we should just make a button that posts the cset URL to the affected bugs and do away with "List changeset URLs".

Comment 1

5 years ago
(In reply to Chris Jones [:cjones] [:warhammer] from comment #0)
> But now that tbpl can talk to bugzilla

TBPL can't actually talk to bugzilla (at least not yet), we are using http://www.graememcc.co.uk/m-cmerge/ (repo: https://bitbucket.org/graememcc/m-cmerge/) to mark merges. (In the future I imagine we'll import the m-cMerge functionality into the TBPL repo, rather than hosting separately).

There is an existing issue open in the bitbucket tracker (https://bitbucket.org/graememcc/m-cmerge/issue/8/glandium-mark-m-i-pushes) for making m-cMerge support marking inbound pushes too.

Once that's fixed, we could just replace the "List changeset URLs" link with one pointing to something like http://www.graememcc.co.uk/m-cmerge/?cset=foo&branch=bar
This features is now implemented in m-cmerge.  Example:
http://www.graememcc.co.uk/m-cmerge/?cset=4888b8395089&tree=mozilla-inbound
m-cMerge is fairly heavyweight: for single changeset pushes, it'll still be faster to flip over to a Bugzilla tab and cut-and-paste. I think we'll want a different solution for that case.

That said it should be useful for pushes with multiple changesets. I'm happy to put together a patch to add the relevant links to TBPL if you think it'll be a useful first step.
Created attachment 660072 [details] [diff] [review]
Add m-cMerge links

Haven't requested review, as I'm still not sure how useful it would be given comment 3, but this adds m-cMerge links to non-Try trees.

Updated

5 years ago
Summary: Feature request: add a button to auto-post csets to bugs → Add a button to auto-post csets to bugs

Comment 5

5 years ago
Comment on attachment 660072 [details] [diff] [review]
Add m-cMerge links

lgtm :-)

Whilst we could maybe do things like only showing on mozilla-central, or for pushes over size N; I think it's not worth making the UI inconsistent - so as is wfm. (With revised commit message :-))
Attachment #660072 - Flags: review+
http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/e32299202235

Leaving this open for now. We can do better for small pushes.

Updated

5 years ago
Depends on: 791282

Comment 7

5 years ago
(In reply to Graeme McCutcheon [:graememcc] from comment #6)
> http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/
> e32299202235

This part is now in production.

Comment 8

5 years ago
I'd like to close this bug now to make things easier to track. cjones, did you still want something to cover the smaller pushes use-case? If so, I'll file another for that and morph the summary of this bug to be just about m-cMerge.
I've pulled together some thoughts on small pushes, thoughts/comments welcome:
https://etherpad.mozilla.org/oTyMkUxGzE
(Assignee)

Updated

3 years ago
Product: Webtools → Tree Management
TBPL has been EOLed (bug 1054977).
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → INCOMPLETE
(Assignee)

Updated

2 years ago
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.