/srv/irclogs.ubuntu.com/2010/04/05/#ubuntu-reviews.txt

mpontillothanks all. I read the workflow on the wiki; one question: when you add the 'patch-needswork' (or any other patch-*) tag, do you remove the 'patch' tag at that point or keep it? Looking in launchpad, I see people doing it both ways01:51
mpontillo... also, I'm trying to understand which patches should vs. should not be accepted into Ubuntu. I guess after release, if it's a higher risk item (sync from Debian for a new release) it would go into -backports, and if it's a lower-risk but higher priority (i.e. crash fix) it would go into -updates ...02:09
mpontillo... but before release, I'm assuming the general strategy is that serious things that would be -updates make it in, but things that would go into -backports we wait and sync from Debian02:09
nigelbmpontillo, generally things that go into -updates and -backports need to go into a development version first02:47
nigelbnow, the best thing would be to forward the patches upstream02:47
nigelbinitially when the guide was written we planned to remove the patch tag, but that is messing up with the script that subscribes ubuntu-reviewers to bugs with patches02:48
mpontillothanks nigelb; sounds like the 'patch' tag should be left on through the entire process (until the the patch has been rejected or integrated into some release). I can clarify the text on the wiki if you like02:50
nigelbI know... I wrote it :D02:51
nigelbso, now you just leave the patch tag in there02:52
nigelbpersia, how much backlogged into mails are you?07:47
persia9 days07:47
nigelbouch, ok.  I'll not mail07:47
nigelbwell, updates include some pretty graphs (see topic)07:47
nigelbpatch day moved up to wednesday may 5th07:48
nigelband I still need to convince brian to make the script understand 'patch*' tags07:48
nigelbpersia, did you see ^09:28
persiaI did.09:29
nigelbI could use some help with the convincing09:30
persiaHave you tried putting it on the QA team meeting agenda?09:30
nigelbem, no09:30
persiaIt's at a really inconvenient time-of-day, but maybe one of us might be awake for it sometime in the next couple weeks.09:31
persiaIf not, perhaps someone else would be willing to present, based on some wiki page clearly stating the position.09:31
nigelbI asked bryce a few days back09:34
nigelbhe said he'd talk to brian... but really... it would be nice to have the 3 or 4 of us09:34
nigelbI wish I could be there at uds :(09:35
nigelbjust needs the few active people to come together and sit down and talk09:35
nigelbabudid I miss something?09:43
nigelbabupersia, qa team meeting is okay time for me... it generally isn't around 3 hours right?09:45
=== \vish is now known as vish
persianigelb: It usually lasts about an hour.  If it's an OK time for you, then please raise it.10:57
persianigelb: If it can't be sorted pre-UDS, I'm definitely up for trying to catch the folks involved there.10:57
nigelbpersia, we can try to talk to bdmurrary once before that10:58
persiaAt least once :)10:58
nigelbwell, if I can get you, bryce, bdmurray, and jcastro together for 20 minutes it would be extremely productive10:58
nigelb(these would be the people who are interested in getting the review queue going)10:59
nigelbalso, MOTU should ideally promote review queue as stepping stone to packaging training10:59
persiaI think we have to demonstrate that the review queue works before we can convince MOTU it's the right place to send prospective folk.11:00
persiaAnd if you don't try to include me in that list, it might be earlier, because the rest of the folks are close timezone-wise.11:00
nigelbby getting the review queue down to zero?11:00
nigelbyeah... well.. as many as humanely possible11:01
persiaWell, either down to a non-frightening number, or demonstrate sufficient progress that there is a sane expectation that it will be there soon.11:01
nigelbwell.... I for one think we should be focusing on the subscribed bugs11:01
nigelbthat is a definite way to get things moving11:01
nigelbbut just one person working on things isn't going to create magic overnight :(11:02
persiaIt's not just one person though, is it?11:02
nigelbwell <1011:02
persiaThat's a number I'll believe.11:02
persiaMOTU started with about 3 folks: it's just a matter of docs and publicity.11:03
nigelbOh!11:03
nigelbIn that case, I guess we still have hope11:03
persiaAbsolutely.  My expectation is that the reviewers team will reach a state of being a first-class participant in the basic workflows sometime around November.11:04
nigelbdid you sign up for being a contact on patch day?11:05
persiaDoesn't mean that we don't have a lot to do now, but it always takes a while to get from a few interested folks to somewhere else: basically we have to demonstrate that we do enough to be worth consulting when other folks are doing stuff.11:05
nigelbyes, i.e., as established reviewers11:05
persiaI didn't yet.  I'll take a bunch of the slots that are left open at the end: I'm flexible and in an unpopular timezone.11:05
persiaRight.11:05
* vish always wondered what timezone persia was in :)11:06
nigelbvish, jp11:06
vishah..11:06
nigelbthat's why he's always around when we're around11:06
vishhehe , i was thinking he was at the other end , alaska or somewhere ;)11:07
nigelbpersia, I wanted an established reviewers team member to start off patch day11:07
persiaI'm only 3:30 offset from there most of the time.11:08
persianigelb: Your local time at day-start is 15:30, right?11:08
persiaIs that bad for you?11:08
nigelbyep.. I'll be at work11:09
persiaMaybe nhandler or bdmurray can start.  If not, I will.11:10
nigelbI'll try to catch bdmurray11:10
nigelbwill you be up around the time he's waking up?11:10
nigelbtoday i.e.11:10
persiaMaybe.  The other side of the day works better for that combination, but is bad for you :)11:10
nigelbset a time...11:11
nigelbI'll go to bed early if need me11:11
nigelbbe11:11
persiaI think he's usually around something like 15:00 to 28:00 or so.  I'm usually around 0:00 to 16:00 or so.11:12
nigelbUTC?11:13
persiaYeah.11:13
nigelbso 23:00 UTC sounds fine?11:13
* nigelb gulps ... 3:30 am11:14
nigelbstill.... it would be very productive I guess11:14
persiaI'm not sure it would be that productive.  I'm not that diurnal, so maybe better to schedule something for a day I'm around later.11:14
persiaTuesday or Thursdays are often good candidates.11:14
nigelbTuesday is meeting horror day11:15
nigelbeveryone is bound to be busy11:15
persiaWell, there's no QA meeting, and that's what keeps me around until 17:00 or 18:00 most weeks.11:15
nigelbso tomorrow at 23:00 seems better?11:16
persiaWhy 23:00?11:16
nigelb17:00?11:17
persiatomorrow at 16:00 ought be fine.11:17
persiaOr 17:00.11:17
persiaBut I don't know that I'll be up that late tonight.11:17
nigelbthat time is fine with me too11:17
nigelbso tomorrow it is11:17
persiaThought so :)11:17
nigelbinformation that needs to be added to reviewers team wiki?11:20
nigelbhttps://wiki.ubuntu.com/ReviewersTeam/KnowledgeBase11:20
nigelbI can't think of any11:20
persiaThere's work needing doing to better integrate stuff (especially the simple patch/debdiff stuff), and it needs a pointer to the workflow, but yeah, otherwise seems complete.11:26
persiaPersonally, I'd rather see it organised in terms of activities, but that will come with time and practice doing stuff.11:26
nigelbI was thinking of having a how to generate patches from svn and git but that seemed too much11:26
persiaFor us, yeah.11:27
persiaI think it's worth having that documentation in general though.11:27
persiaBut I think it's more valuable for MOTU docs.11:27
nigelbit would be appropriate for us to have it as we're the people upstreaming patches11:27
persiaGood point.11:28
persiaAlthough I think that the docs would be slightly different for the different audiences.11:28
nigelbyes. definitely.11:28
persiaTHe base stuff is the same, but the HOWTO differs between case A: someone has a patch and wants to apply to upstream, and generate something to send for review vs. case B: someone knows a fix is upstream and wants to cherrypick to generate a debdiff.11:29
nigelbexactly11:30
nigelbwhich means the wording might get a lile tricky11:31
persiaOr maybe it involves three documents.  One is "How to use the tool", and two are example workflows using the tools.11:32
persiaThen the dev pages *and* the reviewers pages can reference it.11:32
nigelbor one "how to tool" and we'll link it up differently in 2 pages11:32
persiawith inclusion.  That works.11:33
=== etali1 is now known as etali
nigelbyep.  inclusion is what I thought too11:34
nhandlerpersia: I can only do a 15:30 on a Sunday sadly12:10
persianhandler: No worries.12:12
nigelbI'm holding off on creating page on git and svn how-to13:03
nigelbeach upstream might have different policies and I need to go through the bigger ones at least13:04
nhandlernigelb: Instead, why not try and link to (or ask upstream to create) those guides ?13:52
nhandlerThey will be more accurate and better maintained, and more people can benefit from them13:52
nigelbwhen I submitted code upstream, generally, i had to look for generic guides and teak them13:52
nigelblike look for a generic guide to submitting patches in git and then looking at upstream style guides13:55
=== yofel_ is now known as yofel
nhandlernigelb: Which is why you should ask the upstream that uses git to update their guides to include enough basic info about git for beginners to contribute22:19
nhandlernigelb: Also, what is Debian/QADelta ? Is it meant to list the reasons for diverging from Debian?22:20

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