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".
(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.
Summary: Feature request: add a button to auto-post csets to bugs → Add a button to auto-post csets to bugs
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.
Depends on: 791282
(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.
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.