[00:09] Good evening #-reviews! [00:09] hi [00:10] anyone able to link me to the QA graphs page for Cleansweep? [00:20] bobbo: topic? [00:25] aha, thanks yofel, I need to improve my reading! [00:26] np ;) [00:50] a lot of the patches are very old, and a fair few of them already seem to have got merged or fixed sometime [00:53] crimsun_: Knock knock [07:32] good morning [09:21] vish: um, we probably need to do that discussion here. [09:22] vish: the 400 something count which I'm talking about includes only patches with the bucket tags [09:23] nigelb: I understand that [09:23] nigelb: we should first get them there [09:23] nigelb: and then wait [09:23] vish: if you want you can go to http://blog.qa.ubuntu.com/ [09:24] bilalakhtar: yes, so upstream has ot make a call. Where does regression in ubuntu come into the picture at all? [09:24] nigelb: let me give you an example [09:24] bilalakhtar: what we're doing (or trying to do) is not have people write patches and they stale in launchpad. [09:24] nigelb: how many of the ones are we *really* forwarding? which have not gotten attention? [09:24] vish: we have forwarded 200 patches [09:24] atleast [09:25] nigelb: some patches get accepted upstream and then they create regressions [09:25] nigelb: what i'm saying is tht we are claiming success from the work seb128 and similar teams have been doing for ages [09:25] bilalakhtar: how can that be our fault? Upstream has to make a call. [09:25] nigelb: GNOME bug #625675 [09:25] Its their code. They should know. [09:25] no bot in here? [09:25] nigelb: of those 200 how much have we really improved due to clean sweep? [09:25] bilalakhtar: our duty is to foward them upstream [09:25] vish: Do you have of any method to track that? [09:25] nigelb: fine, you win [09:25] I dont [09:26] bilalakhtar: Its not a win/fail thing. [09:26] bilalakhtar: I'm only trying to explain. [09:26] nigelb: I meant you;re right [09:26] but I cannot mention it, so wrote 'win' instead [09:26] bilalakhtar, the issue is usually rather changes pushed to the distro without waiting for upstream review [09:26] bilalakhtar, upstream tend to know their code better and fix issues they create [09:26] seb128 is right, that's what is happening, and its my fault as well [09:27] nigelb: no i dont, but i think its wrong to claim others work as our success, my best guess is that we would have had atleast 150 forwarded are the work that seb128 and others teams have been doing already [09:27] if upstream is introducing regressons, we can take blame for that too. Besides all the ^**^% we get blamed for. [09:27] ok, seb128 and nigelb , see this one https://bugzilla.gnome.org/show_bug.cgi?id=625675 [09:27] vish: That is him helping us out in cleansweep! How can I not acknowledge that as contribution? [09:28] seb128 and nigelb : its regression https://bugzilla.gnome.org/show_bug.cgi?id=626620 [09:28] nigelb: how is that? he has been doing that for ages..! why is that our success? [09:28] vish: there isn't a condition that states its only number patches reviewed by volunteers or people other than desktop team that counts. [09:28] vish: Why NOT? [09:28] He's reviewing patches. The goal of cleansweep is to review patches. Where are you getting confused? [09:29] it's rather how you message it than what you do [09:29] because you claim those patches have been review in the cleansweep effort [09:29] nigelb: you are just increasing his workflow and claiming that he has helped you? why? [09:29] but they have not, they have been reviewed by people doing their usual work they were doing before cleansweep [09:29] +1 [09:30] you should rather say than ubuntu forwarded nnn changes and that cleansweep contributed to that effort [09:30] seb128 is right, most of the [people on OC are long-time patch reviewers [09:30] that is exactly what i'm saying , total reviewed patches != Operation Cleensweep success [09:31] http://bit.ly/ddQFie is the list of bugs with patches, however many of them have debdiffs rather than patches [09:31] bilalakhtar: if you can give a list to bdmurray he can tweak the script. [09:32] We have a new script that is a bit "too" active. [09:32] nigelb: yup, the new script is marking merges also with 'patch' [09:32] vish: but how do you know which is which? [09:32] would be nice if the script stopped changing bugs already fixed for example [09:32] it's tagging patches since which have been uploaded [09:32] nigelb: for one , the whole process is a shaddy [09:32] seb128: we were losing bugs through cracks when we did only LP [09:33] seb128: yes, like https://bugs.edge.launchpad.net/ubuntu/+source/ddclient/+bug/619141 nigelb [09:33] vish: I'm not the one doing the tagging. [09:33] vish: brian is the one doing the tagging. [09:33] Talk to him. Please. [09:33] nigelb: but you know about it right? [09:33] vish: yes, but I don't know how to fix it. [09:33] nigelb: you suggested the changes? [09:34] vish: No, you were on the mail. Brain suggested the changes. [09:34] bdmurray: ping [09:34] Wait, you suggested the changes. [09:34] We missed a few bugs and you alerted me. Remember? [09:34] nigelb: it wasnt me.. i have at many times suggested several changes and you have just stuck to the currrent process [09:34] I wrote a mail to Brian copying you and we have a new script that uses the bugs list. [09:34] vish: which was discussed at uds, and noone raised issues. [09:35] It was also mailed to -devel. [09:35] nigelb: you should not have been adding more bugs and subscribing the team to new bugs was someone else's idea.. not mine [09:35] vish: subscribing team to new bugs was what was decided at UDS. [09:35] It was not my decision. [09:35] nigelb: half the people who have tried to suggest changes have given up arguing with you, and just went with "meh!" [09:36] vish: Only you suggested changes. [09:36] ugh, I wish there was a mailing list. [09:36] nigelb: several have suggested and i dont want to name them [09:36] vish: Everyone wanted $foo, $foo1, $foo2. [09:37] I had to stick with a $foo and get going. [09:37] REmember, no one wanted to do this at all. [09:37] nigelb: they have tried to question several workflow and you have argued for statistics and they eventually just gave up [09:37] so I was more interested in getting the thing launched rather than dicussing. [09:37] nigelb: yes, you did a great job is bringing attention.. i dont deny it :) [09:37] if someone wants to make changes and feel it can be effective, I'm open to it. [09:38] vish: If there was ever any reason was because it was I need to kickstart it rather than keep discussing [09:38] we went for about a month talking. [09:38] Finally, I stopped talking started doing. [09:38] nigelb: doing? [09:38] * bilalakhtar will now leave [09:39] vish: When the workflow was written, there were lot of oppositions. [09:39] Most of it was, this is hard. [09:39] I asked, do you have a better way [09:39] Then it would squard back to, "this is hard" [09:39] * nigelb needs to get back to work. [09:40] If someone came up with a meaningful change, I haven't rejected it. [09:40] (maybe I've forgotten about it due to lack of time) [09:41] nigelb: several times people have tried to reason with you, but since you have great interest they have just left it to your judgement [09:41] since they didnt have much stake.. [09:42] vish: The time that the entire process was discussed, there were back and forth mails for months. [09:42] can we try to calm down a bit? [09:42] *month. [09:42] nigelb: lets try this now.. :) firstly, you should not have been adding more bugs and subscribing the team.. that is a way to get a more realistic number.. [09:42] vish: why didn't you raise it at uds? [09:42] you were there [09:42] It there it was discussed. [09:42] nigelb: i wasnt in every session [09:43] It was /not/ my call. [09:43] seb128, vish, nigelb: we all put loads of hours into getting the number of patches down, we all worked very hard, what is the bottom line of what you want to change? [09:43] * nigelb isn't very sure what just happened. [09:43] dholbach, I don't want to change anything, I'm not in this discussion ;-) [09:43] dholbach: nigelb: even now.. i dint bring up the whole discussion.. :) [09:43] the end goal being: more patches getting reviewed, forwarded and integrated and bugs closed [09:43] I don't even know what the discussion is about [09:44] ok, what are practical suggestions right now? [09:44] dholbach: nigelb: i just wanted to mention it as a we need to actually do work.. somehow this got twisted.. ;) [09:44] I just made a comment saying that if anybody has a perception issue that cleansweep is claiming work it should do change the wording to say "ubuntu did and cleansweep helped" [09:44] parts of this discussion read like a heated argument and if there's no practical suggestions, we can as well all go back to doing some real work [09:44] should->shouldn't [09:45] dholbach, indeed [09:45] seb128, ok, I'm sure nobody tried to steal anybody's credit, but we can message that out more clearly - I'll try to do that when I blog about it next again [09:45] seb128: ack. agreed. [09:45] dholbach: indeed.. :) i just made a passing observation in -motu which lead to a lot of chatter :( [09:45] M turn next week. I'll do it. [09:45] * nigelb gets back to work. [09:46] dholbach, I was not the one who complained, that's just what I got from the discussion [09:46] yeah, let's back to work ;-) [09:46] seb128, gotcha, thanks for your comment [09:46] * dholbach hugs you all [09:46] not worth discussing [09:46] for what is worth I'm fine cleansweep counting patches I forwarded as cleansweep effort [09:46] as long as it motivates people to work on cleaning [09:47] numbers are always complicated [09:47] you never get them right, somebody will always be pissed off [09:49] dholbach: i actually dint get pissed off :s i just wanted to say we need to do more reviews and that our numbers arent really very good right now :( [09:49] dholbach: pls read my para on -motu .. [09:49] vish, totally agreed - we need to work harder :) [10:12] vish: you've got time for a logo design? [10:13] http://pollka.net if you're interested. [10:13] nigelb: we already have one, right? needs any update or a new one? [10:13] oh! [10:13] Its from the UW competition that happened a few months back. [10:13] elky opensourced the app and called for contributors who would be mentored in PHP :) [10:16] nigelb: its a standalone app or only runs on sites? [10:16] vish: its a standalone app for quick polling [10:16] * nigelb points to launchpad.net/pollka [10:17] cool! [10:17] Actually, alanbell wanted the code [10:17] and it ended up being on elky's junk folder [10:17] and then as a new project and mentoring platform [10:18] * nigelb kicks google maps meanwhile. === yofel_ is now known as yofel [11:33] hmm.. [12:00] just for the record, i think nigelb is great ! :D [12:00] * vish just got pm'd by someone thinking i hate nigelb :s [13:03] * nigelb hugs vish [13:03] belatedly [13:03] Today is just a bad day. At least my foosball luck is improving. [13:23] hm, won all ames today [13:34] * penguin42 closes another bug-with-patch [13:34] a lot fo them are so old that someone somewhere has pushed the bug upstream already [13:36] yep, low hanging fruit. [13:36] Like I said, we need more manpower. [13:37] * penguin42 has this week off so I'm happy to pick of the low hangers off and anything I understand [14:17] nigelb: heh , yeah , somehow they thought i was angry at you.. [oh the joys of irc ;p ] and wanted to use me as a mouth piece for their frustration! === maco2 is now known as maco [14:36] vish: ftr, if anyone has any issues, I'll just wait for him/her to have the guts to mail/ping me. [14:36] nigelb: yay! mud fight? ;p [14:38] vish: hahaha [16:07] vish: what do you want me to do? [16:07] unassign the bug from me and say I won't be able to do it? [16:08] nigelb: hmm, i just entered... dont know what the context is.. [16:08] haha [16:08] cheeeese! [16:09] nigelb: ah! yeah, just add a comment on the bug about what pitti told you , kermiac will pick it up from there.. [16:09] vish: um, what pitti told me.... [16:09] * nigelb has no recollection. [16:10] The logs are on my laptop which is in the shop :( [16:10] * nigelb notes the "..." in the comments :p [16:12] nigelb: yeah, that was an oopsie! i was about to write something .. then pinged you and not sure what i did there..