[Bug 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 Quim Gil changed: What|Removed |Added Status|PATCH_TO_REVIEW |RESOLVED Resolution|--- |FIXED --- Comment #17 from Quim Gil --- Yep, thank you! -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 Alvaro changed: What|Removed |Added Status|ASSIGNED|PATCH_TO_REVIEW --- Comment #16 from Alvaro --- All fixed now in korma: http://korma.wmflabs.org/browser/gerrit_review_queue.html -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 --- Comment #15 from Alvaro --- Quim, after reviewing, the problem was the predicted: * For companies we are starting the analysis from startok = "'2013-04-30'" and this is why in April the values are 0. * For global data, the start time analysis is the start of the report, 2010. We have modified the global analysis so it also starts i "startok" and the data should be the same. Also, the start time for both graphs will be moved to May 2013 to remove the empty April 2013 data. -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 --- Comment #14 from Alvaro --- I need to review it, probably, we are filtering the data before Apr 2013. There are troubles with data until May 2013, so this issue could be related to that. -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 --- Comment #13 from Quim Gil --- Thank you! One graph that makes me wonder is "Are Wikimedia's staff and non-staff contributions processed equally?". How can all the values be 0 on April 2013, when in the graph above we see that the general median is 51,1 days? -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 --- Comment #12 from Alvaro --- Ok, updated all in: http://korma.wmflabs.org/browser/gerrit_review_queue.html after yesterday meeting. -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 --- Comment #11 from Alvaro --- Quim, all work done in: http://korma.wmflabs.org/browser/gerrit_review_queue2.html We have included both metrics, for all reviews pending, and only for reviews pending for reviewers. Once you check all, we will remove the all reviews pending and just leave the graphs with "for reviewer". We will wait for you comments for future steps! -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 --- Comment #10 from Nemo --- Created attachment 15238 --> https://bugzilla.wikimedia.org/attachment.cgi?id=15238&action=edit Who is doing better and worse? Sorry, can't parse this. -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 --- Comment #9 from Quim Gil --- This is useful, thank you. "Number of pending submissions" can keep both lines, it is useful to see the difference between the total amount of open reviews and those waiting for a reviewer. "Age of pending submissions" could also have blue/green columns. If only one value could be showcased, then I would rather go for the changesets waiting for reviewer. "Review time for open submissions..." and "Are Wikimedia's staff..." should be calculated based on ReviewsWaitingForReviewer only. In the list of repositories, ideally "pending" and "ReviewsWaitingForReviewer" would be merged with blue/green lines. If this is not possible, then we can keep them separate. "Update time for pending in days" would be calculated based on ReviewsWaitingForReviewer and would be the criteria for sorting the list. I think this approach solves the problem in a better way than I had initially proposed, thanks to your idea of bringing a prototype first. -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 --- Comment #8 from Alvaro --- A first version for getting comments: http://korma.wmflabs.org/browser/gerrit_review_queue2.html The repositories data is now ordered by pending reviews waiting for reviewer and this metric is shown (ReviewsWaitingForReviewer). Next step could be to modify the pending time (Update time for pending reviews waiting for reviewer in days), but I am not sure about it and I prefer to share with you this first approach and decide next steps. -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 Quim Gil changed: What|Removed |Added Priority|Normal |High -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 --- Comment #7 from Quim Gil --- Alright, let's filter changesets with label:Code-Review<=-1. We can add the delay later one if we really need it. -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 Nemo changed: What|Removed |Added CC||federicol...@tiscali.it --- Comment #6 from Nemo --- Just in case it's not clear, what needs to be filtered are -1 and -2 (label:Code-Review<=-1). They've always been filtered in our lists of open commits: https://www.mediawiki.org/wiki/Gerrit/Reports#Commits_lists (query first shared by robla). -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 Quim Gil changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #5 from Quim Gil --- Well ok, after a first wave of replies it looks clear (to me) that we are not going to implement an automatic process for abandoning changes anytime soon. Even if we do it, the margin of time could be e.g. three months, which brings back the problem of not reflecting the patches that are really waiting for a review. Therefore, Alvaro, your further investigation to see how can we fix our metrics without changing our current process is welcome. :) -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 Quim Gil changed: What|Removed |Added Priority|High|Normal --- Comment #4 from Quim Gil --- Two things. This isn't that trivial to implement in Metrics Grimoire. Alvaro can explain the details if needed, but basically what happens is that each upload needs to be checked to make sure that it is the latest of a changeset. The other thing, more important actually, is that perhaps the solution is to change our policy, and abandon -1 changesets after some period. There is a discussion ongoing at http://lists.wikimedia.org/pipermail/wikitech-l/2014-April/075825.html We will decide what to do once we know what happens with the policy. -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 Liangent changed: What|Removed |Added CC||liang...@gmail.com --- Comment #3 from Liangent --- I have some changesets where I uploaded a reworked patchset several weeks or even months after an original -1 was given... -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 --- Comment #2 from Alvaro --- Hmmm, this query detect patchSet with -1 not touched but a submissions could have several patchSets. We should only take care that this corresponds to the last patchSet! -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 --- Comment #1 from Alvaro --- Hi! In OpenStack projec, the policy is that a submissions with -1 and not touched in 1 week is moved automatically to ABANDONED status. Something like that make sense in Wikimedia also? A query to detect some of this submissions: select count(changes.id) as total, issue_id, new_value, status, changed_on from changes, issues where changes.issue_id = issues.id AND status = 'NEW' and (new_value="-1" or new_value="-2") and DATEDIFF(NOW(),changed_on) > 30 group by issue_id having total=2; A list of 143 submissions in this state are detected. We can move all of them to ABANDONED state but maybe, it is better to do it in Gerrit directly. -- 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 63533] Gerrit metrics about open changesets should ignore -1s
https://bugzilla.wikimedia.org/show_bug.cgi?id=63533 Quim Gil changed: What|Removed |Added Priority|Unprioritized |High Assignee|wikibugs-l@lists.wikimedia. |a...@bitergia.com |org | -- 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
