[Bug 17322] Integrate git/Gerrit and Bugzilla

2014-04-16 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

Bug 17322 depends on bug 47622, which changed state.

Bug 47622 Summary: Bugzilla gerrit bot should mention patch author when 
notifying about a new changeset
https://bugzilla.wikimedia.org/show_bug.cgi?id=47622

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-07-01 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

Bug 17322 depends on bug 47622, which changed state.

Bug 47622 Summary: Bugzilla gerrit bot should mention patch author when 
notifying about a new changeset
https://bugzilla.wikimedia.org/show_bug.cgi?id=47622

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |---

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-06-29 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

christ...@quelltextlich.at changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #34 from christ...@quelltextlich.at ---
Should be fixed since we upgraded gerrit and hooks-bugzilla on 2013-06-27.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-06-28 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

Bug 17322 depends on bug 47622, which changed state.

Bug 47622 Summary: Bugzilla gerrit bot should mention patch author when 
notifying about a new changeset
https://bugzilla.wikimedia.org/show_bug.cgi?id=47622

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-06-28 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

Bug 17322 depends on bug 47621, which changed state.

Bug 47621 Summary: Bugzilla gerrit bot should use a shorter format
https://bugzilla.wikimedia.org/show_bug.cgi?id=47621

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |WORKSFORME

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-04-30 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #33 from christ...@quelltextlich.at ---
(In reply to comment #32)
 (In reply to comment #31)
  (In reply to comment #30)
   (In reply to comment #29)
Of course this doesn't trigger when a new patchset adds a new reference 
to a
bug, which is mildly irritating. :-)
 
 I can't think of a non-hacky way that would allow me to sleep at night.

Would
https://gerrit-review.googlesource.com/#/c/45201/
allow you to sleep at night?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-04-29 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

James Forrester jforres...@wikimedia.org changed:

   What|Removed |Added

 CC||jforres...@wikimedia.org

--- Comment #29 from James Forrester jforres...@wikimedia.org ---
(In reply to comment #28)
 This is now enabled again, with the following config (based on feedback):
 
 Do notify on:
 * New change
 * Restored change
 * Abandoned change
 * Merged change
 
 Do NOT notify on:
 * New patchset
 * New comment
 
 Basically, leave a comment when there's something you should check out in
 Gerrit (someone's started work, someone's giving up, or work is done), but
 not during the normal review cycle (comments  patchets).
 
 I hope this meets everyone's approval.

Of course this doesn't trigger when a new patchset adds a new reference to a
bug, which is mildly irritating. :-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-04-29 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #30 from Chad H. innocentkil...@gmail.com ---
(In reply to comment #29)
 (In reply to comment #28)
  This is now enabled again, with the following config (based on feedback):
  
  Do notify on:
  * New change
  * Restored change
  * Abandoned change
  * Merged change
  
  Do NOT notify on:
  * New patchset
  * New comment
  
  Basically, leave a comment when there's something you should check out in
  Gerrit (someone's started work, someone's giving up, or work is done), but
  not during the normal review cycle (comments  patchets).
  
  I hope this meets everyone's approval.
 
 Of course this doesn't trigger when a new patchset adds a new reference to a
 bug, which is mildly irritating. :-)

This was by request, as many changes go through many patch iterations before
merging.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-04-29 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #31 from James Forrester jforres...@wikimedia.org ---
(In reply to comment #30)
 (In reply to comment #29)
  Of course this doesn't trigger when a new patchset adds a new reference to a
  bug, which is mildly irritating. :-)
 
 This was by request, as many changes go through many patch iterations before
 merging.

Sure; I assume there's no way to trigger only on
new-patchset-which-has-newly-added-bug-reference?

J.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-04-29 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #32 from Chad H. innocentkil...@gmail.com ---
(In reply to comment #31)
 (In reply to comment #30)
  (In reply to comment #29)
   Of course this doesn't trigger when a new patchset adds a new reference 
   to a
   bug, which is mildly irritating. :-)
  
  This was by request, as many changes go through many patch iterations before
  merging.
 
 Sure; I assume there's no way to trigger only on
 new-patchset-which-has-newly-added-bug-reference?
 
 J.

I can't think of a non-hacky way that would allow me to sleep at night.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-04-24 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

Bartosz Dziewoński matma@gmail.com changed:

   What|Removed |Added

 Depends on||47621

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-04-24 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

Bartosz Dziewoński matma@gmail.com changed:

   What|Removed |Added

 Depends on||47622

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-04-04 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #28 from Chad H. innocentkil...@gmail.com ---
This is now enabled again, with the following config (based on feedback):

Do notify on:
* New change
* Restored change
* Abandoned change
* Merged change

Do NOT notify on:
* New patchset
* New comment

Basically, leave a comment when there's something you should check out in
Gerrit (someone's started work, someone's giving up, or work is done), but not
during the normal review cycle (comments  patchets).

I hope this meets everyone's approval.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-04-02 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

Lydia Pintscher lydia.pintsc...@wikimedia.de changed:

   What|Removed |Added

 CC||lydia.pintscher@wikimedia.d
   ||e

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-03-23 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #27 from christ...@quelltextlich.at ---
(In reply to comment #22)
 Per the feedback I got yesterday, we still need to tune down the noisiness on
 new patches (we only want notifs on new changesets, not each patchset
 attached
 to a change)

See https://gerrit-review.googlesource.com/#/c/43843/

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-03-21 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #21 from Andre Klapper aklap...@wikimedia.org ---
Slightly bikesheddy:
* I wonder whether it should also set the patch-in-gerrit keyword (current
  situation/workflow that I'm not a fan of), or whether I should pick up the
  debate again on creating something like a PATCH_IN_GERRIT bug status - I
  consider getting closer to resolving this report by having a patch in
Gerrit
  a status but no constant property (=keyword). Related: bug 39399, bug 39402,
  http://lists.wikimedia.org/pipermail/wikitech-l/2012-December/065046.html - 
  might be best to discuss in bug 39402.
* I wonder whether it should also notify on Abandoned (Somebody could pick up 
  the work notification in the bug report)?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-03-21 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #22 from Chad H. innocentkil...@gmail.com ---
We could turn on notifs for abandoned/restored (I was thinking those might be
useful), in addition to merges.

Per the feedback I got yesterday, we still need to tune down the noisiness on
new patches (we only want notifs on new changesets, not each patchset attached
to a change)

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-03-21 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #23 from christ...@quelltextlich.at ---
(In reply to comment #22)
 We could turn on notifs for abandoned/restored (I was thinking those might be
 useful), in addition to merges.

See https://gerrit.wikimedia.org/r/55048

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-03-21 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #24 from Waldir wal...@email.com ---
Shouldn't Jenkins' messages be filtered out?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-03-21 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #25 from Chad H. innocentkil...@gmail.com ---
We did disable notifications on comments.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-03-21 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #26 from Waldir wal...@email.com ---
I was asking because of bug 34557 comment 5

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-03-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #20 from Chad H. innocentkil...@gmail.com ---
Plugin is live again, and is now only notifying on new patch and merge.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-03-18 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

Andre Klapper aklap...@wikimedia.org changed:

   What|Removed |Added

 Blocks||39402

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-03-18 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #19 from Tim Landscheidt t...@tim-landscheidt.de ---
(In reply to comment #16)
 Example: https://bugzilla.wikimedia.org/show_bug.cgi?id=41889#c6

Cool!  The layout needs some work, though :-).

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-03-16 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #18 from christ...@quelltextlich.at ---
As the patch to adjust spammyness is sitting upstream already for a few days
without comments from the maintainer, I'll link it here, so we can keep track
of it:

https://gerrit-review.googlesource.com/#/c/43280/

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-03-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

Quim Gil q...@wikimedia.org changed:

   What|Removed |Added

 CC||q...@wikimedia.org

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-03-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

jeremyb bugzilla+org.wikime...@tuxmachine.com changed:

   What|Removed |Added

 CC||bugzilla+org.wikimedia@tuxm
   ||achine.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-03-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #16 from Andre Klapper aklap...@wikimedia.org ---
Example: https://bugzilla.wikimedia.org/show_bug.cgi?id=41889#c6

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-03-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #17 from Chad H. innocentkil...@gmail.com ---
Yeah, we got it partially working :) But it's a bit spammy, so I turned it off
until we can fine-tune it a bit further.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-01-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

Andre Klapper aklap...@wikimedia.org changed:

   What|Removed |Added

   Priority|Normal  |High
 CC||aklap...@wikimedia.org,
   ||anja.jentz...@wikimedia.de,
   ||christ...@quelltextlich.at,
   ||dereck...@espace-win.org
   Assignee|wikibugs-l@lists.wikimedia. |christ...@quelltextlich.at
   |org |

--- Comment #15 from Andre Klapper aklap...@wikimedia.org ---
AFAIK Christian is working on this, based on comment 14, hence assigning this
to him.


Note: I'm VERY afraid of duplicated work in this area.

Hence I also CC Anja who's been working on WMDE's wikidata bot (see bug 44100
comment 2 for an example),
and Dereckson who's been working on communicating between Gerrit, Bugzilla and
IRC according to bug 43655.
It would be great if you could share some implementation details so far to see
if some parts are reusable.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2013-01-02 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #14 from Chad H. innocentkil...@gmail.com ---
I began writing Bugzilla support for Gerrit here:
https://gerrit-review.googlesource.com/#/c/40750/

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2012-08-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

Liangent liang...@gmail.com changed:

   What|Removed |Added

   See Also||https://bugzilla.wikimedia.
   ||org/show_bug.cgi?id=39399

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2012-04-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #12 from Tim Landscheidt t...@tim-landscheidt.de 2012-04-19 
17:07:11 UTC ---
Another argument pro auto-closing: Unambiguity.  Bug 35964 just got closed as
RESOLVED FIXED while the commit is still in review. 
https://bugzilla.wikimedia.org/page.cgi?id=fields.html says for FIXED: A fix
for this bug is checked into the tree and tested.  That's technically
obviously true in this case but it doesn't mean that it is resolved in master
so that a user can profit from it without too much hassle.

If there's no auto-closing, it's up to some developer to follow the review of
his commit and then close the bug which will probably lead many developers to
close their bugs prematurely.  Auto-closing would enable them to fix and
forget.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2012-04-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

Liangent liang...@gmail.com changed:

   What|Removed |Added

 CC||liang...@gmail.com

--- Comment #13 from Liangent liang...@gmail.com 2012-04-19 17:13:01 UTC ---
Hmm you're reusing an old bug so the first several comments seem unrelated or
outdated...

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2012-04-13 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #11 from Tim Landscheidt t...@tim-landscheidt.de 2012-04-13 
13:01:48 UTC ---
(In reply to comment #10)
 [...]
 Generally, I'm kind of leery about auto-closing an auto-opening bugs. I may
 commit half a fix for a bug and not want BZ to be closed yet. 
 [...]

I have neither looked at the code I referenced nor worked in a project with
auto-close, but I assume it would usually work in the way that the commit
message would have to include some (semi-natural) phrase that triggers the
close (^This fixes bug #(\d+)\., ^Fixes: (\d+)$, etc.)  So if a commit
doesn't fix a bug, you just don't use the exact phrase but maybe This
partially fixes bug #123. and everything's as you expected.

  Regarding deployment, while I see your point, I think not using this facility
would still leave the ambiguity: Has the deployment *really* been done?  In the
past, there have been delays when difficulties were encountered or manpower was
short.  Another automated comment on a closed bug will cause no harm, but
*actively* inform the reporter who cares about the bug and not require *them*
to do something.

  This of course *could* depend on how the deployment branch is organized, i.
e. how a commit that is reverted is recognizable to some script so I would
consider this part low priority.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2012-04-12 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

Tim Landscheidt t...@tim-landscheidt.de changed:

   What|Removed |Added

 CC||innocentkil...@gmail.com,
   ||rlan...@gmail.com
  Component|Subversion  |Git/Gerrit
 AssignedTo|mediaw...@natividads.com|wikibugs-l@lists.wikimedia.
   ||org
Summary|Integrate SVN and Bugzilla  |Integrate git/Gerrit and
   ||Bugzilla

--- Comment #9 from Tim Landscheidt t...@tim-landscheidt.de 2012-04-12 
20:30:43 UTC ---
git integration: https://wiki.mozilla.org/Bugzilla:Addons

Gerrit integration: http://code.google.com/p/gerrit/issues/detail?id=124

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 17322] Integrate git/Gerrit and Bugzilla

2012-04-12 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322

--- Comment #10 from Chad H. innocentkil...@gmail.com 2012-04-12 22:15:05 UTC 
---
If we make use of the feature described in issue 124 (see comment 9), we could
easily use patchset-created to ping BZ with a A change for this bug was pushed
for review in gerrit 123 [project: foo; branch; bar] We could follow up with
Gerrit change 123 was merged on DATE [project: foo; branch: bar]

Generally, I'm kind of leery about auto-closing an auto-opening bugs. I may
commit half a fix for a bug and not want BZ to be closed yet. 

(In reply to comment #8)
 I don't know whether the configuration management of WMF wikis falls under SVN
 as well, but I'd also find it extremely helpful if a bug not only received a
 Fixed in r4711. comment, but also that the fix went live on labs (or
 whereever) and finally on the main farm.
 

The plan right now is to have deployment-labs running master (so it'll get all
the latest changes) and deploy master to the cluster every 2 weeks. So I think
in the long run this will be a little bit less of an issue. Once we get into
the cycle you'll be able to assume that anything over ~2 weeks old has been
deployed.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l