nigelbabu82, yaay!00:07
nigelbabunhandler: you're on in 1 hour00:07
nhandlernigelbabu: Aren't I going on 01:00 UTC, which is 1.75 hours from how00:20
nigelbabunhandler: wht is utc time now?00:21
nhandlernigelbabu: 23:2100:21
nigelbabunhandler: oh no, can you start frm next session, i need to get to work00:21
ajmitchnigelbabu: wouldn't you need sleep or something first? :)00:22
nigelbabuajmitch: sleep is over. 2200 to 023000:22
nhandlernigelbabu: I can try, but I'll probably be eating dinner in the middle00:22
nigelbabunhandler: *grin*  ajmitch will be around :)00:23
* ajmitch may not be around00:23
ajmitchthat'll be when I head out for lunch (12:00 here)00:23
nigelbabu83 bugs more!00:24
nhandlerSomehow we'll cover any questions that get asked during that hour, but they might need to wait for answers00:24
maconigelbabu: im helping a guy fix his centos00:26
macoim failing :(00:26
macoim *trying* to help him fix his centos00:26
macobut its a yum b0rk not a things-that-exist-here-too b0rk00:26
macoakk: i think the list is "stuff ubuntu-reviewers is subscribed to" and that the magic date is simply the date from which the script that autosubscribes the team was set to run00:27
nigelbabumaco: lol, ok.  we're having a gap in coverage due to my excellent math skills thinking gmt = utc00:28
macoah daylight savings got ya?00:30
nigelbabui think so, why is there no desktop tool for time conversion?00:31
macodate -u00:31
maconigelbabu: the date command accepts format strings, such as %Z for the timezone name to be listed00:32
nigelbabumaco: ok, so basically its always -4:30 for me00:33
nigelbabuwell, I'm always at +4:30 from UTC00:33
macoright, whereas gmt moves about for british summer time00:33
nigelbabuyeah, I usually keep a gmt thingie on my clock and forgot it moved around00:34
nigelbabucan someone review 3 more bugs?00:36
nigelbabuso we can get to 70?00:36
nigelbabuat least I want to make sure we reach the 40s for this patch day :)00:37
nigelbabupersia: we need some system to deal with packages that are ubuntu-specific00:38
nigelbabualso, can someone test and review bug 56469800:40
ubot3Malone bug 564698 in util-linux "lscpu command shows "???" when messages are translated in Japanese" [Undecided,New] https://launchpad.net/bugs/56469800:40
ajmitchpersia would be good to look at that :)00:40
nigelbabuyes, hopefully he will when he sees it :)00:40
nigelbabumaco: can take a look at the k-reltaed packages in the list?00:41
nigelbabuwow, I spell *bad*00:41
nigelbabuajmitch: wise to defer eucalyptus to server team?00:42
nigelbabujust unsubscribe then?00:42
ajmitchI assume so, you're the boss ;)00:42
nigelbabulol, no I'm not :)00:43
ajmitchbut that bug looks very actively looked at00:43
nigelbabuyeah, its assigned too00:43
nigelbabuok, so I'm off, someone please cover and answer questions00:46
nigelbabuwe're at 8200:46
nigelbabubug 56944200:47
ubot3Malone bug 569442 in ttf-wqy-zenhei "After upgradeing to Lucid, unexpectedly-using bitmap font in Japanese Environment (upgrading regression)" [Undecided,New] https://launchpad.net/bugs/56944200:47
* nigelbabu points to maco ^00:47
macoew yuck. bitmapped japanese. that has to be hard to read00:48
nigelbabubitmapped chineese actually I think00:48
nigelbabucheck out the bug00:48
macojapanese uses chinese characters00:49
Paddy_NIHey maco00:49
nigelbabumaco: o_0 thats news to me01:12
maconigelbabu: japanese didnt have a written language. chinese monks gave them literacy01:16
nigelbabumaco: wow01:20
ajmitchalmost down to 1 page of stuff to look at01:24
nigelbabuYAAY! We have breached 8001:25
nigelbabuOnly 79 more bugs to look at!01:25
macoPaddy_NI: https://wiki.ubuntu.com/ReviewersTeam/ReviewGuide01:47
Paddy_NIthanks maco I was wondering what I should be doing.. perhaps reading the topic :)01:48
macoum hmmm01:59
macoso one of the wiki pages links to bugs that have been marked accepted upstream or accepted debian01:59
macomaybe this query should filter out "fix committed" bugs though, since if its committed in ubuntu, its just waiting for a freeze to be lifted, which most people cant really do anything about02:00
nhandlerSo I guess I'm up now02:00
nhandlerNice and quiet now02:23
* ajmitch is working...02:25
nhandlerHmm...Does anyone remember what bugs the script looks for to subscribe the reviewers team to?02:41
persianhandler: I believe the script looks for any bugs with attached patches that have the patch added since a certain date and aren't subscribed to a blacklist of teams and aren't bugs against a blacklist of packages.04:49
nhandlerpersia: Do you know what statuses it looks at?04:50
persiaI believe all open statuses (not invalid, won't fix, fix released).04:51
nhandlerOk, thanks04:51
persiaThe interesting question is: if a bug is Fix Committed and has a patch attached, is this confusion on the part of the person setting the status?04:51
macopersia: bdrung told me today that if i upload a package but its sitting in the queue waiting to be approved, then its fix committed04:54
macofor -proposed stuff and during freezes04:54
persiaI guess, but why should it both have an attached patch and *not* be subscribed to a team?04:55
persiaI think the majority of cases are mistakes: the rare case where it's correct, we can just unsubscribe the team.04:56
persiaAnd it's still Patch Day!  We're down to 79 bugs to review, which indicates that we're 58% done.05:00
macowhat team would be subscribed when its uploaded and sitting in a queue?05:01
macoarchive admins or something?05:01
maco(do they have a team?)05:01
persiaDepends on the reason for the freeze.05:01
persiaI'd say the Release Team, generally.05:02
macoi uploaded a merge to maverick today thats waiting for toolchain freeze to go away05:02
persiaSo, bug #538774 is an example of a bug in our queue that's Fix-Committed.  Why shouldn't that get patch-accepted-debian and be unsubscribed?05:02
ubot3Malone bug 538774 in python-scipy "scipy.io.loadmat() excessively slow (regression)" [Undecided,Fix committed] https://launchpad.net/bugs/53877405:02
persiaAlso, even if an upload is prepared for Ubuntu, should we not be looking at pushing the patch somewhere else (or verifying that this has been done)?05:03
macothat's a misuse05:03
macothe person put fix committed when they committed to their own branch on lp, but if theyre going to use that it should be when it gets committed to lp:ubuntu/<package>05:04
persiaHow is that different than the case you describe?  It will be closed with autosync in a couple days, much like the case you descibe will be closed with unapproved queue clear.05:04
persiaOh, right.  Timing miss.05:05
macotiming miss?05:05
persiaYeah.  The status was changed for the incorrect reason (making it good we caught it), *but* in later analysis, it has reached a state that matches what you describe.05:05
macooh yeah i looked at the activity log to see how it got there05:06
macoi dont think we're doing it like this yet, mostly (though kubuntu does) but committing to lp:ubuntu/<package> bunches of fixes and then only doing a dpkg-buildpackage -S once seems like something thatll be futury05:07
persiaRight.  Anyway, my point is that in case 1: where it's Fix Committed for the wrong reasons, we want it anyway, and in case 2: where it's Fix Committed because it's been uploaded and pending processing, it's not bad for us to get it.05:07
maco"get it"?05:07
persia"receive the bug in our review queue"05:07
persiaI've worked on a bunch of packages that do VCS packaging and upload containing lots of fixes by lots of folks, and yes, in that case, Fix Committed makes sense, but I think there's still value in us tracking the patches to ensure that patch communication is happening.05:09
macoyou mean in terms of upstreams?05:15
macopersia: patch communication in terms of what?05:21
persiaBasically, I'm not sure how the stuff we do is in any way affected by Fix Committed.05:22
persiabecause none of the automated patch communication stuff takes effect until the upload is processed.05:22
macoi think ill go through all the fix-committed's and check to see if theyre improper use and if so put them to the proper status05:23
persiaI recommend against that.  You'll bump into the Desktop Team, which abuses that status regularly.05:23
persiaThere aren't any more Fix Committed ones in our queue.05:23
macoyeah just noticed that when i saw jon the taco and seb128 both use it weirdly05:26
macoseems that team uses fix committed in ubuntu in place of upstream bug watches05:26
macopersia: my email says to tell you im gonna expire from sponsors soon05:32
persiaI suspect your email tells you that you're expiring from the obsolete sponsors team, and suggests telling me under the false impression I'm going to let anyone renew :)05:33
persiaWould you like to join the new, cool, shiny, real sponsors team?05:33
* persia finds the .m and prepares to click the button, waiting for a response05:35
macohehe yes :)05:37
macothe .m?05:37
macois that because lp/~maco != me ?05:38
persiaThe .m is essential05:38
macoyeah i dont know lp/maco05:38
macojust that they took my name05:38
maco(just got email)05:43
persiaThanks for helping with sponsoring :)05:44
persiaOK.  5:00 UTC, and we're only down to 78, which is the one I did.  There's lots there, and we've only 6 hours left, which means we need to push fairly hard to finish within the day!05:57
macopersia: it went back up to 7906:35
persiaIt did indeed, but shishire is working 485225, and I expect it to go back down soon.06:37
macopersia: if a not-right patch is attached to a bug, should we unmark it as a patch or delete it or what?06:39
ubot3Malone bug 512395 in openoffice.org "Openoffice.org's .desktop files do not contain translation domain info" [Wishlist,New]06:39
macowell in this case there are a bunch of "oh no wait this time i got it right..." type things06:40
persiaI'd call that patch-needswork06:40
macoshould the old patches be removed?06:40
dholbachgood morning06:40
persiaI'd leave the old patches for historical purposes.06:40
macoshould they be unmarked so that the patch list on the side only lists the one thats being suggested?06:41
persiaHmm.  That's an interesting question.06:41
persiaI think "No", because I think there's sufficient value in reading the bug log to understand the background.06:42
persiaThat said, it does add apparent difficulty if one assumes folks just grab from the patch lists and push, but I think doing so is uninformed behaviour, and don't want to encourage it.06:42
persiaNote that I feel this is a very subjective opinion on my part: I'm not certain it should be policy without wider discussion.06:43
macousually when im going "oops that patch was wrong, heres a fixed one" i delete my previous one from the bug. wasnt sure if that was just me or not06:43
dholbachDevelopment and MOTU Q&A Session in #ubuntu-classroom in 17m06:43
macodholbach: pfft nobody in this channel has any interest in development :P06:44
macothis channel is for kitten photos!06:44
dholbachmaco: I hope you're wrong ;-)06:44
persiaAnd we kinda compete with MOTU in some ways: seeing who can get the patches applied first06:44
persia(to different targets)06:44
maco(actually no, #ubuntu-women was the kitten-photo channel earlier)06:44
macocuz we're trying to make it so that all the patches reach ubuntu by syncing from debian?06:45
persiaOr, better, by inclusion by the original software authors, yeah.06:45
persiaMind you, it's a very freindly and cooperative competition :)06:46
macowell if its in upstream, itll hit debian then sync to here in most cases06:46
macoi dont /think/ there's a whole lot that goes straight into ubuntu without passing through debian06:46
macoexcept kde. we get that packaged within a day or two of a new release and then i have no idea how long til debian's got it06:46
persiaDepends on the area of focus.  Some areas have lots, some not so much.06:47
macomm and the kernel06:47
macoso where do OOo patches get pushed to? OOo-proper or Go-OOo?06:49
persiaAsk ccheney :)06:50
persiaAnd we're still at 79 at 6:00 :(07:11
* persia needs to stop digging at hard ones, and hunt through for easy ones07:11
macopersia: unsubscribe the reviewers team after marking patch-needswork?07:15
persiamaco: Please, and please give the patch submitter as *much* detail as you can about how to fix it.07:16
persiaI'm not sure we have a good model for what to do once the patches get fixed: maybe hint that they can resubscribe us.07:16
macopersia: a comment has already done that07:16
persianigelbabu: When you get back: this is a case that deserves some thought: how to we recapture patches that needed work to ensure that they don't get lost.07:17
macopersia: dont suppose a script could be written to run periodically that churns through all bugs tagged "patch-needswork" and check the activity log to see if a patch has been added since the tag was and if so, replace the patch-needswork with patch?07:25
maco(no idea if lplib exposes enough info for that though)07:25
persiaThat might work, but we'd need to find a way to put it in a workqueue.07:26
persiaMaybe have that script remove patch-needswork and subscribe the reviewers team again?07:26
ajmitchoops, looks like I missed dholbach's session :)07:40
macoajmitch: its still going07:40
macohe's doing a Q&A07:40
macoand im learning more about this UDD stuff and how it makes everything ive been doing obsolete and wrong07:41
* ajmitch goes to read the backscroll07:41
* ajmitch needs to learn how to be a rock star 07:41
persiamaco: It's not obsolete and wrong.  I do nearly everything that way.  There are alternatives.07:42
macoapparently i fail at humour this far past bedtime07:43
ajmitcheven though I've been using bze since 0.0.x, I'm still getting used to the Proper way of using it for packages07:43
* persia isn't convinced there exists *the* proper way.07:44
persiaI've used it in at least 3 different ways, depending on the set of packages with which I was interacting.07:45
ajmitchyeah, I've used it with having upstream+debian+integration branches07:45
ajmitchthat made for fun times07:45
=== gotunandan_ is now known as gotunandan
persiaHey gotunandan!07:53
gotunandanhi persia07:53
persiaIt's Patch Day! We're following https://wiki.ubuntu.com/ReviewersTeam/ReviewGuide#Workflow : just ask if you have any questions, or need an opinion or a hand.07:53
gotunandani do not know how useful I can be right now, since I am on a windows PC (at work :P )07:53
persiaAh, yeah, that makes it tricky to test the patches :)07:54
gotunandanAlso I think I'll need to do some reading up on the entire packaging process, pretty keen on doing something at least in the maverick cycle !07:55
persiaI'd actually recommend against "reading up" too much.  It's easy to get stuck in circles of ever-more-complex documentation for specialised corner cases.07:56
persiaIf you just want to do packaging work for Ubuntu, I'd suggest you'd do better to hang out in #ubuntu-motu, try to work on some of the MOTU TODO tasks, and ask questions whenever you get stuck (this will probably be *lots* at the beginning).  Folks will likely point you at specific, targeted, documentation that helps you understand the problem at hand, rather than you needing to learn everything first.07:57
gotunandanoh, but its been a while since I last did some packaging, my ppa has been stale for a few months now :| Just need to refresh a bit, I guess07:57
persiaEven our most knowledgeable packagers don't pretend to know everything about packaging.07:57
gotunandanright, thanks :)07:57
ajmitchnot even persia knows all :)07:58
* persia encounters unknown stuff *every day*07:58
persiaOK.  7:00 and we're down to 76.  I'll blame that on the recent -classroom session, but let's get back to Patch Day!08:00
* ajmitch hasn't been working on any for several hours 08:01
* imbrandon grabs the list-o-bugs08:07
persiaWe're down to one page :)08:07
ajmitchand some of the bugs have 2 tasks listed there08:09
persiaHey etretyak!08:11
etretyakpersia, hi08:12
persiaWe're following https://wiki.ubuntu.com/ReviewersTeam/ReviewGuide#Workflow and down to 75 bugs in our current worklist.  Jump in, grab one, and ask if you have questions.08:12
etretyakok.. let me see08:13
persiaWelcome ddecator :)08:17
ddecatorthanks persia =)08:17
ddecatorhopefully you guys can introduce me to what patch reviewing entails this weekend. is there a wiki page i can look over?08:18
persiahttps://wiki.ubuntu.com/ReviewersTeam is probably a good place to start.08:19
persiahttps://wiki.ubuntu.com/ReviewersTeam/ReviewGuide#Workflow is the core of what we do.08:19
ddecatorperfect, thanks08:21
persiaHey zus!08:23
zuspersia, helllo08:23
* zus waves to persia08:25
persiaDid you change your mind, and are staying up to review a few patches, or just stopping by?08:25
persiaAh, the latter, I think :(08:25
ddecatorhow long does it take to review a patch?08:27
persiaMinutes to hours, depending on the complexity of the patch, etc.08:27
ddecatorif you got one that'd only take a few minutes, i can see about helping out real quick before i go to bed =)08:27
persiaSometimes we get lucky, and the patch is submitted by upsteam, so it's just a quick test and see about getting into Ubuntu.08:28
persiaSometimes we're unlucky, and the patch is confusing, does something subtle that's tricky to reproduce, and has an usptream with an irregular bug tracker with a complicated signup process.08:28
ddecatori'll let you guys handle those for now..08:28
persiaBug #527639 looks like it's not terrible.08:30
ubot3Malone bug 527639 in boinc "boinc should use chrt instead of schedtool" [Wishlist,Triaged] https://launchpad.net/bugs/52763908:30
persiaShould be easy to test, and seek upstream comment.08:30
ddecatorso since it's not necessarily a bug to reproduce, i just get the software, see how it works, apply the patch, and make sure it doesn't break the functionality at all?08:34
ddecatorpersia: ^08:37
persiaBasically, yeah.08:38
persiaAlso, check some upstream docs on schedtool and chrt and make sure that it actually makes at least some sense (no reason to bother upstream if it's pointless).08:39
persiaI suspect it does make sense because without having thought about it previously, I have chrt on my system, and don't have schedtool, but that may be coincidence.08:39
ddecatorright, which seems to be the reasoning behind the suggested change08:40
persiaIndeed :)08:40
persiaYou did ask for an easy one :)08:41
ddecatoralright, so i have some more questions about what i should do exactly, but i'm afraid that'll take up too much time and i've already stayed up later than i should have, so i'll return to this patch tomorrow and ask for some help in here =)08:41
ddecatorthanks for the help persia08:41
ddecatorg'night guys08:41
persiaNote that someone else might grab it first though, and you'll have to get a new one.08:41
ddecatorthat's fine08:42
ddecatori'll watch to see what the person did in that case08:42
persiaHey RAOF.  We're almost done, but can always use a bit more on the last push :)09:01
persiaOK.  75 bugs.  3 hours.  It's a push, but  possible.  Now we just need a *heap* of reviewers in the eastern pacific :)09:09
persiaHey alourie|work.09:27
persiaStill 75 at 8:00 :(09:58
nigelbpersia: how are we doing on the count?10:39
persiaNot wonderfully :(10:39
nigelbbreached 70s?10:39
persiaWe're down to 75, but we've been holding there for a good while now.10:40
nigelbouch, was 79 when I left10:40
persiaSo it's nice to have only one page of reports, but I think not many people still find it 5th May :)10:40
nigelbwell, patch day gets over in about 20 days and we can review what we did wrong10:41
nigelbI have a couple of complaints with the workflow which I need to correct10:41
nigelbI feel we should never unsubscribe ~ubuntu-reviewers unless we don't intent to review that patch at all10:41
nigelbalso, I'm thinking of defering a few packages to the server folks, the ones that are too hot to handle for us10:42
persiaHrm?  80 minutes I thought.10:42
nigelb1100 UTC is when?10:42
persia78 minutes from now.10:42
nigelbok, my time calculation is just broken :)10:42
persiaSo, my thought on unsubscription is that if we have already reviewed a bug, why do we need to stay subscribed?10:43
nigelbalso we need a way for people to say, I got this, the ~ubuntu-reviewers need not get into this bug10:43
nigelbthe unsub is because I'm giving other folks can option to say ^10:44
nigelbso, all the sub'd bugs would be taken care of by us, and others would be taken care of someone else10:44
nigelbgreat, now I have putty, my private key, but forgot to note down the ip adress and port to connect to :(10:46
persiaI don't think that's a big deal: if we bump into bugs like that, we can just unsub.  It's usually fairly easy to identify when someone has it, and it's not that hard to add our tags and unsub.10:47
nigelbyes, so we'll make a policy that we wont take of bugs that we've unsub'd10:48
nigelbso I'd rather have the others in the review queue.  anyway it wont show up in the query unless ready for re-review10:48
persiaHow does "ready for re-review" work?10:49
persiaEspecially for the patch-needswork case?10:49
nigelbwe leave instruction that once you feel the patch needs another review by us, please change the patch-needswork tag to patch tag10:52
persiaOh right.10:55
persiamaco: ^^10:55
nigelbi probably need to work on standard replies10:58
nigelbany other lessons from patch day?11:06
persia10:00 and still 75 bugs.  Over to nigelb!11:07
persiaWe need to do better at determining what to do with Ubuntu-only packages.11:07
persiaWhen there's no real upstream, I'm not sure how much we can help.11:08
persiaMaybe that's just extending the blacklist a lot.11:08
nigelbI'd vote to blacklist packages like eucalyptus11:09
nigelbwe'll just break their workflow11:09
persiaAnd stuff like apport/mountall/casper/ubiquity/etc. as well?11:10
nigelbyep, especially stuff maintained by folks at canonical11:11
nigelbthey are being paid to do it, we're better off helping the packages that noone cares about11:11
persiaI disagree with that.  I don't see how it matters whether it's being maintained by folks at Canonical: I think it matters more whether it's being maintained *in* Ubuntu.11:12
persiaSo, there's folks at Canonical who maintain a bundle of packages and do most of their uploads to Debian, etc.  They might use the package in Ubuntu, but ae normal upsteams in that they don't tend to use distribution-provided packages.11:13
nigelbyeah, well, they tend to sort of coincide11:13
persiaAnd there's package in Ubuntu, like ubuntustudio-controls, that aren't maintained by folks at Canonical, but have no sensible separate upstream.11:13
nigelbwe ignore those is what I'm saying11:14
nigelbthe ones that are always uploaded to ubuntu11:14
persiaSure, but I think we should focus on stuff where upstream maintains it in Ubuntu, rather than anything else.11:14
persiawhereas something like python-lazr.uri would work fine with our regular process.11:15
persia(or maybe that wasn't the best example, but you get the idea)11:15
nigelbits extremely subjective, we'e better off having a wiki page were folks can suggest the ones we can *potentially* ignore and then take a call11:16
persiaThat sounds ideal: let's have a wiki page and invite developers who maintain their packages *in Ubuntu* and don't want us pushing patches upstream to add their packages there.11:17
persiaThat makes it opt-in on a per-developer/per-package basis.11:17
persiaAnd nicely avoids all the social discussions :911:17
persia:) !11:17
nigelbyes, better than them coming screaming at us11:17
nigelbok, so graphs, changes to workflow, standard replies, and page for developers to opt-out11:19
persiaWell, if we have a wiki page, and we announce it to ubuntu-devel@, then if folks come screaming, we can point at the wiki page and the email, and ask them why they don't read their mail :)11:19
nigelbAnything else hit you we're doing wrong?11:23
nigelbDid we advertise too less about patch day? :/11:24
persiaI think we did fairly well.11:24
persia110+ patches reviewed.11:24
nigelbIt would be very intersting to see the patch review mailing list, especially the moderation to see how many people participated11:24
nigelball the bug mails would be stuck in moderation.  I have to ask brian to give me a tar or something of all the mails over last 2 days11:25
persiaI think we also need to work on integration with bugsquad more.11:25
persiaI saw someone in bugsquad suggest subscribing us and adding the patch tag as the way to triage a bug with a patch.11:25
nigelbwoot, yes it is :)11:25
persiaBut I'd rather be more integrated, so that folks upstreaming bugs know about our tags, and we make sure we do the right upstreaming stuff when we forward our patches.11:26
nigelbraise it at the next bug squad meeting?11:26
nigelbits after UDS anyway11:26
persiaSounds like a good plan.11:26
nigelbduring UDS you can say how successful Patch day was ;)11:27
persiaand make sure the upsteaming docs point to our docs about tags when also sharing patches and make sure our docs point at the upstreaming docs as best practice when opening upstream reports, etc.11:27
nigelbour docs point to upstream docs11:27
persiaSure, but let's make sure we *preserve* that as we update our docs :)11:28
nigelbyes, the docs need extensive revision, including how to test11:28
persiaWe're an interesting mix of triagers and developers (and folks who are both), and I'd like to not end up with too much of a separate identity.11:28
persiaPatch review should be something people do, not something people are.11:28
nigelbdid I tell you the statistic I had come up with?11:29
nigelb1800 bugs with patches attached, out if which 1500 will be in our queue if there were no date thingy.  If all the members of ~ubuntu-dev reviewed 10 bugs, we'd be done!11:29
nigelbthere were 153 members of ~ubuntu-dev when I last checked11:30
nigelbif everything comes together, patch review, adopt-an-upstream, bug triage, it would be the most awesomest!11:31
persiaYeah.  It's not hard to review all the patches.  Folks just aren't doing it.  That's why we needed this initiative.11:31
nigelbYep.  I'm deeply saddened by that.11:32
nigelbBut, also happy that in 1 month, we could get going so fast.11:32
nigelbThis cycle, I'd like to explore the possibility of us being the new starting point for development, though I'd advocate uploads to debian than Ubuntu to reduce the delta.11:33
persiaI don't think any model that involves the starting point for development is good.11:33
persiaI'd rather consider us *a* starting point.11:33
nigelbWell, yeah *a* starting point, especially to budding triagers11:34
nigelbPeople who are well versed with bugs and want to start thinking of packaging stuff11:34
persiaAnd I think that work in this team is a close match to a lot of MOTU work, so it makes a good "next step" for bug control folks vaguely wanting to do more development, but without any close association with one of the development teams (e.g. desktop, server, kubuntu, mythbuntu, etc.)11:34
nigelbyep, lost folks like me ;)11:35
persiaRight.  I think we're thinking the same, except for the "a"/"the" bit :)11:35
persiaI don't like to think we're lost, so much as we're more interested in fussing with random stuff than focusing on anything specific.11:35
nigelbyea, I agree with the 'a' too11:35
* persia tends to focus first on anything that is irritating when using Ubuntu installs, and then on various unloved pacakges.11:36
persiaAt least, for me, if someone is caring for a package, they tend to do a good enough job that I'll end up being lazy, rather than fixing it.  If nobody else is watching, then I'm more likely to be annoyed and just fix it.11:37
nigelbDesktop team likes us taking care of patches, so far they're happy that we're pointing out stuff that work11:37
nigelbBut I'd defer mozilla stuff to somene from the team since they take good care of them11:38
nigelbAlso, most of the patches tend to turn up in unmaintained stuff since no one maintaines those pacakgse11:38
nigelbPeople get annoyed and fix the bug.11:39
nigelbAnd we ignore them.11:39
nigelbWe're down to 74, I guess you're reviewing.11:40
persiaI must run: good luck with the end of the day.  I think we made great progress, regardless of the counts, just from the number of folk who came by the channel and read the docs, and had some practice doing it.11:43
nigelbpersia: I'm stepping out for some chores too. :)11:43
nigelbpersia: shall I delcare patch day officially over?11:44
nigelbanother 15 mins to go, but I have to run11:44
persiaIf you have to run, unless you can get someone else, yeah.11:44
persia111 bugs done!11:44
persiaActually more, because we got some new ones (the count went up a couple times)11:44
nigelbPatch Day is officially over.  Thank you Review Leads for your help.  Those who reviewed patches, thank you for your time.11:45
nigelbWe have reviewed 111 bugs in the last 50 hours!11:45
persia49 :)11:45
nigelb48-3/4 ;)11:46
persiaWe hit 61% of our target queue.11:46
nigelbWhich is really really good news11:46
persiaWhich isn't bad for a first time.11:46
nigelbnow, I think we can move that date 1 month back11:46
nigelband plan for regular patch days11:46
nigelbno wait, scratch regular patch days11:46
persiaEvery month?11:46
persiaYeah, let's not have them for a while.11:47
nigelbwe need something to encourage every day review, like 5-a-day11:47
persiaLet's have sporadic patch days on an as-needed basis until we get a feel for how the team works.11:47
persiaand if enough folks want to have a special Patch Day! we can do that.11:47
nigelbyes, when we get dangeoursly high, for now lets just keep things in 2 digits :)11:47
persiaThat makes sense to me, but let's not worry so much about next week, as lots of folks will be paying attention to UDS.11:48
persiaWith luck, we won't get so many patches, but I expect a bit of backlog :)11:48
nigelbpersia: yes, and do tell them about the 10 patch per head thing and the 111 that me manged :)11:48
persiaReally, UDS doesn't work that way: there's no good way to do announcements.11:49
* persia is *not* going to try to schedule a plenary about Patch Day11:49
nigelblol, ok11:49
nigelbjust mention that in the session11:49
persiaBest is just to send mail to the developer and bugsquad mailing lists, saying what was done.11:50
nigelbYes, that's a good idea11:50
persiaThat ends up getting mirrored all over the place11:50
nigelbThe fridge can re-post *that* one11:50
persiaAnyway, really have to go :)11:50
nigelbok, ciao11:50
vishaw.. i did probably only around 5 :(12:21
maco<persia> We need to do better at determining what to do with Ubuntu-only packages. <--- subscribe sponsors?12:22
yofelwould be fine, but only after we convert it into a debdiff12:29
yofelI saw bugs where the sponsors were subscribed, didn't find a debdiff, unsubscribed themselved and subscribed us12:31
macowhich they're not supposed to do12:31
macothe debdiffs-only stuff was supposed to have gone out a year or two ago12:32
yofelhm, I didn't check the dates on the posts12:32
vishmaco: recent bugs too its being done12:33
yofelwell, packaging the patch included reviewing it so it makes sense in some way12:35
maco"check it before you upload" has always been part of the sponsor's job12:35
macoeven if it's a debdiff, they're still supposed to check it out!12:35
vishhrm , if the patch has been forwarded upstream do we unsubscribe te review team?12:36
yofelis says so on the workflow, you're supposed to subscribe yourself and follow the bug after that12:37
vishyofel: Bug #40872 , nigel has subscribed himself, i think we can remove the review team..12:41
ubot3Malone bug 40872 in nautilus "Desktop icons are allowed to overlap" [Unknown,Confirmed] https://launchpad.net/bugs/4087212:41
yofeltrue, work is done12:41
persiaSo, patches for ubuntu-local stuff fals into an annoying gap.12:48
persiaSponsors *aren't* supposed to upload anything that isn't a ready candidate, and *are* supposed to be all sorts of picky to help train nascent developers.12:49
persiaBut we're not even especially focused on getting stuff into Ubuntu.12:49
persiaThe thought was to blacklist the Ubuntu-local stuff from the script that fills the review queue, and then expect whoever maintains the package to process the bugs once in a while.12:50
persia(where "whoever maintains" might be some random Ubuntu developer passing by, depending)12:50
persiaThat said, there's no reason that any of us *can't* turn a patch into an upload candidate in any of several ways, but that it's not the main goal of the team.12:51
yofelhm, about bug 546032 - I did use reportbug a few times already, but never supplied patches, and I don't get how submittodebian works, can I just use reportbug with the patch tag and supply links to the patches in the bug description?12:55
ubot3Malone bug 546032 in tcsh "[PATCH] tcsh-6.14.00-astron $COLUMNS $LINES not set" [Undecided,New] https://launchpad.net/bugs/54603212:55
persiaI usually attach the patch in an email to the BTS.12:56
persiaI'm not sure how reportbug does it.12:56
yofelwell, reportbug is a cli app that creates a mail, but you're right, just sending a mail directly would be easier I guess12:56
persiayofel: reportbug has a -A option that lets you attach something, so you could use that.12:58
yofelah, you're right, thanks12:59
persiaJust be sure to add the patch tag if you sdo.12:59
persiamanpages are wonderful :)12:59
yofelyep, I did check it, too fast it seems :P13:01
=== effie_jayx_ is now known as keffie_jayx
nigelbabumaco: UUD can be taken at UDW14:08
nigelbabuerr UDD14:08
hyperairnigelbabu: so how did patch day go overall?14:11
nigelbabuhyperair: 111 patches reviewed14:12
hyperairthat's a lot14:12
hyperairit's now down to one page \o/14:12
nigelbabuyes, thanks a lot for the tme that the review leads put in14:12
nigelbabuI intend to write a mail, now sleep time14:12
vishnigelbabu: hey ,any reason we you've left the team still subscribed to  Bug #4087214:13
ubot3Malone bug 40872 in nautilus "Desktop icons are allowed to overlap" [Unknown,Confirmed] https://launchpad.net/bugs/4087214:13
nigelbabuvish: Its reviewed, see tags14:16
nigelbabuthe review team will not be unsub'd ever unless they dont want us playing with the bug14:16
nigelbabuI'll change the docs as soon as I'm sane and conscious14:17
vishnigelbabu: yup ,the docs wasnt clear... hence, was wondering if the team needs to be un-sub'd there..14:17
* persia has been unsubbing from everywhere, and wonders why we bother to have sub/unsub at all if we're not unsubbing to drop from the workqueue14:18
* hyperair did the same as persia14:18
nigelbabuvish: I realized the hole in the docs only today.  Its my mistake14:18
nigelbabuJust to make it clear.  Do *not* unsub reviewers unless they dont want us poking in14:18
nigelbabupersia can explain further on that or you can read scroll back14:19
hyperair=\ i don't even remember which bugs i unsubbed reviewers from14:19
nigelbabuwe talked about it 2 hours back14:19
nigelbabuhyperair: dont worry, we can find with tgas :)14:19
hyperairheh okay14:19
nigelbabuI'll do that next week, though I may have to run a huge LP query through api14:20
nigelbabupersia: can follow up with lool about the MIR? I need sleep and rest14:21
persiaI can't explain, because I don't see the point (but I'm sure nigelbabu will explain better later)14:21
persiaWhich MIR?14:21
nigelbabubug 52551214:22
ubot3Malone bug 525512 in libtime-modules-perl "[MIR] libtime-modules-perl" [Undecided,New] https://launchpad.net/bugs/52551214:22
nigelbabubut 525395 is blocked due to this14:22
yofelso instead of usubbing reviewers remove the patch tag? or just add the tags?14:22
* persia doesn't like MIRs are part of not liking components, but isn't prepared to ignore components completely yet14:22
nigelbabuyofel: just change the tag according to situation14:22
persianigelbabu: And the query already hides stuff with the right tags?14:22
nigelbabupersia: yes it does14:23
nigelbabuthe unsub was part of workflow when I didn't know that14:23
nigelbabubrian explained it to me that day, that meeeting we had14:23
persiaOK.  I'm not sure there's any benefit to any of subscription/unsubscription/even having a team in LP if we're not actually using it.14:24
yofelk, so change the tag (and leave only one) and leave team subscribed, right?14:24
nigelbabuyofel: yes14:24
nigelbabupersia: having the team sub'd gives option to others not to have us meddle14:24
yofelI'll try to fix the bugs I still find in my inbox later14:25
hyperairnigelbabu: but other people can't unsub us.14:25
persiaBut explain it to be when you've slept.14:25
persiaI'm sure I'll disagree then too, but we can have a productive debate :)14:25
nigelbabupersia: when there is a bug a and there is a very responsible person taking care of it14:25
vish IMO , unsubscribe if someone takes up the bug..14:25
nigelbabuvish: no14:25
hyperairlaunchpad api again \o/14:25
hyperairnigelbabu: go sleep =p14:25
nigelbabuwhat if that person "gets hit by a bus"14:25
nigelbabuhyperair: good idea14:26
nigelbabuI should14:26
vishnigelbabu: i meant , if forwarded upstream..  , but not very sure the current flow is good though14:26
persianigelbabu: Do.  I'm fully prepared to debate this with you another day.14:26
hyperaircount me in.14:26
yofelme too14:26
nigelbabuvish: we still want it on the list, later, sleep is overtaking me14:26
persianigelbabu: And I think you've been pinging lool sufficiently for 525512: I'll pay attention and say something if necessary whist you sleep, but it may just be a matter of time.14:27
nigelbabupersia: most probably, only if he asks something you feel I wont be able to answer :)14:27
nigelbabupersia: and I dont want to wait for 24 hours to finish a conversation ;)14:27
* nigelbabu goes before someone kicks me out14:27
persiaI think it's 99% a matter of waiting for him to have time, which means it's probably not going to happen until post-UDS.14:28
nigelbabuim just reminding, btw vish had you assigned a bug to me? I vaguely remember a mail14:28
yofelhm, a patch attached and issue already fixed upsteam in a different way would be patch-rejected-upstream?14:28
yofelbug 54218514:29
ubot3Malone bug 542185 in qt4-x11 "holdingnuts GUI badly affected by a bug in Qt 4.6.2" [Medium,Triaged] https://launchpad.net/bugs/54218514:29
* persia looks14:29
vishnigelbabu: the cheese bug?14:29
vishnigelbabu: wasnt that what we discussed the other day?14:29
yofelupstream patch: http://qt.gitorious.org/+kde-developers/qt/kde-qt/commit/1ab5feb6260589f254ed209816cb67dbe9d3e4a514:29
persiaYeah, that'd be patch-rejected-upstream, but it's worth adding a comment that this tag is used to denote the case where a patch doesn't match upstream changes perfectly, so that we know it has to be an Ubuntu-local adjustment, etc.14:31
yofelok, thanks for confirming14:31
persiaI'm not certain that all patch authors would understand this use of "rejected" (which isn't quite right) without explanation.14:31
persiaBut I'm not sure what other tags correctly toss the bug into the right "bucket": specifically the set of stuff that needs a developer to apply in Ubuntu directly.14:32
vishpersia: i see one problem though , once we unsubscribe the team , the person who takes over must fully respond to the bug .. or we need to draw a line where the reviewer work ends.. if the bug patch is redone after 2-3 months what is the procedure then? if the person subscribed has gotten busy and is not looking at bugmail?  who's task is it to check the patch then?14:33
persiavish: I don't think whether the team is subscribed or not makes any difference in that case.14:34
persiaI think we'd do better to define a set of states in our patch management workflow, and then focus on moving patches between these states in various ways.14:35
persiasubscription/unsubscription is one tool that could help with that, because it lets us set a match criteria for LP queries (and in a way that *dosen't* send bugmail).14:35
persiaBut just because the team is subscribed doesn't mean anyone will ever look at the bug.14:36
vishpersia: yup no bug mail , but in that case , having the team subscribed might ensure the bug gets a second look ,... but still the current workflow isnt perfect though , this makes14:36
vishs/this makes/  /14:36
persiado a search sometime on all the bugs assigned to MOTU.  Every single one of those is wrong, and nobody has ever used that query to find work to do.14:36
persiaWhy would it get a second look?  If it doesn't show in the queries on the reviewing guide page, it's not likely to be looked at, regardless of anything else.14:36
vishpersia: if the team is subscribed it could be looked again , but the tags would be misleading.. which is kinda wrong , what we need is the re-tagging to work properly and automatically ,once a new patch is submitted ,the old patch must be auto removed and subscribing the team then would probably work14:38
vishs/old patch/old tag/14:39
persiavish: *why* would it be looked at again?  Who would look at it?14:39
yofelis the mail that the review team gets as a launchpad group sent somewhere or to /dev/null ?14:39
persiaAnd how would the result differ from e.g. a query againt the "patch" tag?14:39
persiayofel: It goes to an LP mailing list that is subscribers-only, so most stuff ends up in a moderation queue, and is dropped summarily.14:39
persiavish: My point is not that the current system is correct.  My point is that there's no inherent value to subscription, unless subscription is used to define a query that is used as a workqueue.14:40
persiaWell, and also that it has no more or less value than any other change we make to bugs to define a workqueue.14:41
yofelah, if there was some way to access it I think it would make sense to leave the team subscribed, but from my point of view I would vote to unsubscribe the team currently14:41
vishpersia: currently the workflow is kinda what anyone can do , there is no need for a specific team :s14:41
persiaWell, unless we use subscription/unsubscription as a way to manage workqueues.  We currently do, in that we only have the team subscribed to a subset of the available bugs, so as to focus on quick response to current patches, rather than too much on older patches.14:42
yofelonly team members can unsubcribe the team, which means that if we don't require that we don't really get to review newcomer work as they have no reason to contact us14:43
persiaI'd say that the vast majority of the patches we reviewed during patch day were submitted in the last couple months, which is good.  If a patch has already sat around for a year, we already look bad, and it's not worth missing new ones to try to clean them up until we're ahead of ourselves with the new ones.14:43
persiayofel: Indeed, which is one of the reasons I *like* having unsubscription be part of the workflow: it encourages a member of the team to review all work by non-members until someone shows they know what they are doing, and can be made a member.14:44
persia(as our current requirements for membership are basically: someone says "Yeah, this person is good at it).14:44
yofelyeah, and about an hour to fulfill the bugsquad requirements14:45
persiaand a week to get approved :)14:47
yofelunless somone doesn't use the desktop bugs team as a loophole yes14:49
yofelas that gives you an indirect membership14:50
persiaThere are lots of loopholes :)  But yeah.14:52
persiaThe point of the requirement is basically that we didn't want to have to explain basic bug triage stuff also in the reviewers docs.14:53
persiaOh, and that most of the tasks end up being very similar to triage tasks, so it's helpful to have that background (in any of the *many* ways in which one can get it).14:54
=== yofel_ is now known as yofel

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!