/srv/irclogs.ubuntu.com/2011/04/19/#launchpad-dev.txt

jcsackettsinzui: seems i was disconnected from my irc bouncer earlier. you want me to join the 7pm standup to catch up?00:00
sinzuiIf you want time00:00
jcsackettsinzui: i'm at my computer now anyway. :-)00:01
wgrantsinzui: Can we increase the WIP limit on Development/Coding, given the squad is now larger?00:29
hassan1990hi there, I need some help about creating a branch based on another person branch on launchpad using bazaar.00:38
spivhassan1990: #launchpad is a better channel.  The quick answer is "bzr branch lp:~another-person/proj/branch lp:~/proj/branch"00:41
=== Ursinha-afk is now known as Ursinha
=== Ursinha is now known as Ursinha-afk
LPCIBotProject windmill build #189: STILL FAILING in 1 hr 11 min: https://lpci.wedontsleep.org/job/windmill/189/03:26
poolielifeless/whoever, what do you think about bug 745801?04:39
_mup_Bug #745801: system-based authorization doesn't store useful credentials in gnome-keyring <amd64> <apport-bug> <natty> <launchpadlib :Triaged> <python-launchpadlib (Ubuntu):Triaged> < https://launchpad.net/bugs/745801 >04:39
pooliehm, that's a lame question04:40
pooliea better one is, if i proposed just disabling keyring integration, would you merge it?04:40
poolieit seems highly flaky in launchpadlib in natty04:40
lifelessmy problem here is I don't know who the stakeholders are.04:41
lifeless*I* was certainly happy with what we had before.04:41
lifelessgary or flacoste or jml may know more.04:41
lifelessprior to the reorg changes to dependent libraries don't seem to have been as visible to jml as I'd like them to be04:42
poolie_i_ would like this to progress because04:42
pooliewell, it breaks me nearly every time i run feed-pqm04:42
poolieand secondly because it hurts vorlon and jamesw04:43
pooliei realize it's already critical04:43
pooliemaybe i should mail -stakeholders?04:43
pooliei realize too there are a lot of critical bugs04:43
lifelessso04:45
lifelessI'm easy04:45
lifelessuhm04:45
lifelessgaining traction on this04:45
lifelessprobably need to talk to skaet (rm of ubuntu, its -very- late to be rolling an API break back)04:45
poolieoh04:45
poolieyou might think i'm talking about a different bug04:46
pooliethis is not the api break one04:46
pooliei think that horse has bolted04:46
lifelessalso need to talk to the stakeholders that wanted it in the first place (because if this is a feature goal for ubuntu, its a Big Deal to disable it)04:46
pooliethis is that lplib tries to use gnome keyring but gets it wrong04:46
lifelessI don't know if they are directly on -stakeholders, but I would expect -stakeholders to know who they are04:47
poolieof course we could just fix it04:47
lifelessthat would be good too04:48
pooliei don't want to spend days on it04:48
pooliewe don't know exactly what the problem is and so that makes it a bit hard to estimaet04:48
lifelessjames seems to have a pretty solid direction towards isolating it04:49
poolieshould i do this, or do you think an lp dev will get to it?04:49
lifelessour bias is oldest-first of criticals04:49
lifelessI think if you do it it will happen a lot faster than otherwise04:50
pooliek thanks, i'll take a break then have a quick shot at it04:51
poolieta04:52
LPCIBotProject windmill build #190: STILL FAILING in 1 hr 2 min: https://lpci.wedontsleep.org/job/windmill/190/05:15
wgrantlifeless: https://code.launchpad.net/~wgrant/launchpad/unbreak-api-timeouts/+merge/5822605:27
lifelesshttps://bugs.launchpad.net/launchpad/+bug/745799/comments/705:29
_mup_Bug #745799: DistroSeries:+queue Timeout accepting packages <timeout> <Launchpad itself:Triaged> < https://launchpad.net/bugs/745799 >05:29
wgrantlifeless: Yup.05:29
wgrantlifeless: I glanced at it over the weekend and saw it looked like strucsubs.05:30
wgrantYay.05:30
lifelesswgrant: :( - you could have commented !05:30
wgrantIt was a glance, not a useful analysis like that.05:31
wgrantIt wasn't in oops-tools at that point, so I couldn't get much useful out of it.05:31
wgrantBeyond "oh look, lots of structsub queries. what a surprise"05:32
lifeless:)05:32
jtvThis is our mission, right here.  To fix this: http://theoatmeal.com/blog/fix_computer05:33
wgrantJava‽ Now that's just low.05:35
=== almaisan-away is now known as al-maisan
wgrantlifeless: Thanks.05:35
huwshimiWhy icing for the css + junk folder? What's the history there?05:42
lifelesshuwshimi: context?05:43
wgrantThe CSS and JS is just icing on top of the HTML.05:43
jtvwgrant: congratulations, you spotted what was wrong with the Linux picture!  And not just Java, but Java _after you already know C++_05:43
lifelesshuwshimi: are you asking why its named that ?05:43
huwshimilifeless: yeah05:43
huwshimiwgrant: but the javascript isn't really there, and the images are up a level.05:44
wgranthuwshimi: It all ends up served from that. it was mostly there initially, but got split up (eg. for sprites)05:44
wgrantOr to go in /@@05:45
lifelesshuwshimi: other projects call it static05:45
lifelesshuwshimi: the key point is we compile it once and write to the front end servers so the appservers never see it05:45
huwshimiwgrant: incidentally there is javascript up a level too, and it has css in it05:45
wgrantYay05:45
wgrantThe tree is a mess :(05:45
huwshimiwgrant: yeah that's what I'd like to solve05:46
lifelesswgrant: any idea how '   1 /    0  https://bugs.launchpad.net/openoffice/+bug/1/comments/325' ?05:57
_mup_Bug #1: Microsoft has a majority market share <iso-testing> <ubuntu> <Clubdistro:Confirmed> <Computer Science Ubuntu:Invalid by compscibuntu-bugs> <EasyPeasy Overview:Invalid by ramvi> <GNOME Screensaver:Won't Fix> <Ichthux:Invalid by raphink> <JAK LINUX:Invalid> <The Linux OS Project:In Progress> <metacity:In Progress> <OpenOffice:In Progress by lh-maviya> <Tabuntu:Invalid by tinarussell> <Tivion:Invalid by shakaran> <Tv-Player:New> <Ubuntu:I05:58
wgrantlifeless: Must be a lock, surely. Do you have the ID?06:02
lifelessnope06:07
wgrantCL22006:07
lifelessthats from the oops report06:07
wgrant00155-15181@SQL-launchpad-main-master SELECT BugMessage.bug, BugMessage.bugwatch, BugMessage.id, BugMessage.index, BugMessage.message, BugMessage.remote_comment_id, Message.datecreated, Message.id, Message.owner, Message.parent, Message.raw, Message.rfc822msgid, Message.subject, Message.visible, MessageChunk.blob, MessageChunk.content, MessageChunk.id, MessageChunk.message, MessageChunk.sequence FROM BugMessage, Message, MessageChunk WHERE ...06:07
wgrant... Message.id = MessageChunk.message AND BugMessage.message = Message.id AND BugMessage.bug = 1 ORDER BY BugMessage.index, MessageChunk.sequence06:07
jtvStevenK, wgrant: the generate-contents script wants overrides.  Is there an easy way to produce those for a test distro I create on the fly?06:31
spmjtv: that translate deleteion script thingy on LPS - all done, worked a champ. was done in ~5-10 mins.06:39
jtvspm: wow, I thought it had fossilized06:39
jtvWas that the one about Puerto Rican Spanish translations or something along those lines?06:40
wgrantjtv: They're created by FTPArchiveHandler... so not really.06:40
jtvwgrant: any way I can convincingly fake them?  I just need enough to convince apt-ftparchive that Contents files ought to be written.06:40
jtvspm: ISTR we had a follow-up request for that waiting in the wings.  I'm looking it up.06:41
wgrantjtv: See if you can ask FTPArchiveHandler to, otherwise just write some fake ones. They're really trivial.06:41
wgrantCould probably even be empty.06:41
spmjtv: np06:41
wgrantI don't think Contents generation is likely to use them much...06:41
jtvGah!  No Unity, I want a new browser window in my current workspace, not to switch to another workspace where I had a page open.  I thought we were supposed to focus on documents instead of applications now.06:42
jtvwgrant: well I'm not getting Contents files and apt-ftparchive is spewing out lots of messages about not being able to open overrides files.06:43
wgrantjtv: You're so behind the times. The world has given up on WMs... everything has to use tabs now.06:43
wgrantjtv: touch them and see what happens, I s'pose.06:44
jtvThat's fine with me.  But why does the tab have to be in a different workspace just because I happen to have a page open in the same app somewhere else?06:44
jtvwgrant: so an empty file would be a valid overrides file?06:44
wgrantjtv: Yes.06:44
wgrantit just has no overrides in it.06:44
jtvGrr where did my compose key go?  I had it working earlier.06:44
jtvThanks.06:44
jtvspm: found it.  Expect a follow-up request for a script run soon.06:49
spmnp06:49
jtv(And thanks for running the request, of course)06:49
jtvUnity, you're not telling me there is no way to open a second gvim window from the GUI?06:51
pooliejtv: :!gvim&07:12
=== al-maisan is now known as almaisan-away
lifelesswgrant: hey, quick question07:43
lifelesswgrant: do you know if anything in downtime queries bugs with a status filter [of the form ...or status = .. or status=07:44
lifelesswgrant: *and* looks for INCOMPLETE bugs07:44
wgrantlifeless: I doubt it.07:49
wgrantlifeless: Soyuz doesn't.07:49
wgrantlifeless: And I hope Codehosting doesn't.07:49
wgrantAnd the librarian certainly doesn't.07:50
lifelesscool07:53
lifelessin which case, I am nearly done.07:53
lifelessgrrr07:57
lifeless  File "/home/robertc/launchpad/lp-sourcedeps/eggs/storm-0.18.0.99_lpwithnodatetime_r392-py2.6-linux-x86_64.egg/storm/properties.py", line 67, in __set__07:58
lifeless    obj_info.variables[column].set(value)07:58
lifeless  File "/home/robertc/launchpad/lp-branches/working/lib/canonical/database/enumcol.py", line 39, in parse_set07:58
lifeless    self._enum.name, value.enum.name))07:58
lifelessTypeError: DBItem from wrong type, 'BugTaskStatus' != 'BugTaskStatusSearch'07:58
wgrantYay07:58
lifeless*hate*07:58
lifelessI bet this is going to be painful08:02
wgrantYes.08:03
lifelessTypeError: Comparisons of Items are only valid with other Items08:04
lifeless>>> from lp.bugs.interfaces.bugtask import BugTaskStatus, BugTaskStatusSearch08:07
lifeless>>> BugTaskStatus.INCOMPLETE == BugTaskStatusSearch.INCOMPLETE08:07
lifelessnot freaking helpful.08:07
lifelessI'm quite literally amazed that the existing subclass scheme is working * at all *.08:08
lifelessoh joy, its in a different project.08:10
lifelessahh08:20
lifelessuse_template(BugTaskStatus, exclude=('UNKNOWN'))08:20
lifelessrather than subclassing.08:20
lifelessheaddesk headdesk headdesk08:20
* spm hands lifeless an asprin08:21
lifelessthumper: hey, whats up with bug 154556 ?08:22
_mup_Bug #154556: Bug searches should be case-insensitive with respect to status values <feeds> <infrastructure> <lp-bugs> <lp-foundations> <oops> <Launchpad itself:Triaged> <lazr.enum:In Progress by thumper> < https://launchpad.net/bugs/154556 >08:22
thumperlifeless: stalled, boot me tomorrow08:42
lifelessthumper: willdo08:43
wgrantwallyworld: Hi.08:56
rvbalifeless: Hi Rob, I saw this morning that you "Fix Committed => Fix Released" a few of "my" bugs ... should I be responsible to do that (i.e. followup my bugs all the way to the release)?09:01
wgrantrvba: Generally he or I will close them (whichever of us a requested the rollout)09:03
rvbawgrant: all right then, thx.09:03
pooliehi rvba09:04
rvbapoolie: hi09:04
=== thekorn_ is now known as thekorn
StevenKrvba: If you notice your bugs getting Fix Released, you can update the Kanban board by moving the cards to Done-Done09:04
rvbaStevenK: will do.09:05
adeuringgood morning09:06
lifelessrvba: whomever asks for a rollout will followup on the rollout and ->In Progress or -> Fix Released or (don't touch) as appropriate.09:06
lifelessrvba: you're welcome to ask for rollouts and do this yourself :)09:07
rvbalifeless: well, all I did so far is behind a feature flag (hum...) so I confess I never encountered the need to do a rollout ... but I'll keep it in mind. Thanks Rob.09:08
lifelessrvba: when you have beta users I imagine it will start to matter on a personal basis09:09
rvbalifeless: right.09:09
lifelessrvba: but there is also the team matter of having the deploy queue short, which we can all help with.09:10
wgrantYes, once ScottK is on your back :P09:10
pooliervba i was just thinking of you today when i saw bug mail from you09:10
pooliei hope you're actually enjoying lp09:10
pooliesince i recommended it to you09:10
wgrantpoolie seems to be most efficient at quietly recruiting people.09:11
StevenKHehe09:11
rvbapoolie: so far so good ... and thanks a lot for the recommendation ... I'll owe you a beer for that in Dublin :)09:11
pooliei hope eventually lp's ui will be as beautifully formatted as his resumé09:11
pooliegood :)09:11
rvbapoolie: :)09:12
bigjoolspoolie: huh, had no idea it came from you :)09:13
=== almaisan-away is now known as al-maisan
mrevellMorning!09:31
pooliehi mrevell09:33
lifelessI pity the reviewer for this09:35
LPCIBotProject windmill build #191: STILL FAILING in 1 hr 0 min: https://lpci.wedontsleep.org/job/windmill/191/09:42
bigjoolsumm is the timeout on staging really ~5 seconds?!09:46
wgrantbigjools: It's 11s/09:49
wgranthttps://staging.launchpad.net/+feature-rules09:49
bigjoolswell, the page is timing out after 509:50
wgrantWhich?09:50
wgrantYou mean +localpackagediffs, which is OOPSing instead?09:50
bigjoolshmmm about 8-9 now09:50
lifelessargggh09:53
lifelessheat updating uses OFFSET to get bugs to process.09:53
stubjtv: Your TOT connection bouncing?10:49
jtvstub: no, no, not at all10:49
jtvThis is my 3BB connection bouncing.10:49
stubMine is bouncing :-(10:49
jtvThe problem could be at CAT10:50
jtv"International connections?  And they do nothing for minutes at a time?  GC them!"10:51
jtvIt's been particularly crappy for me since the beginning of Songkran.10:51
jtvI tunnel one of my connections through ssh with protocol keepalives just to stop that from happening.10:52
stubFine here until today apart from by ADSL connection not being able to authenticate just before Songkran10:52
jtvWater everywhere & no staff…10:54
jtvstub: is it your DSL connection that's bouncing?  Or something at a higher level such as TCP?10:54
jtvI had some router trouble myself; maybe it was just the heat.10:55
adeuringlifeless: still around?10:55
lifeless\o/ lp:~lifeless/launchpad/bug-75946711:01
lifelessadeuring: a little11:01
stubjtv: Upstream. ADSL link was fine.11:04
adeuringlifeless: can you have a look at my comment on https://code.launchpad.net/~adeuring/launchpad/api-query-permissions-on-object/+merge/58136 ?11:06
bigjoolsjtv, is someone riding a space hopper over your connection?11:16
lifelessany reviewers around ?11:17
lifelesshttps://code.launchpad.net/~lifeless/launchpad/bug-759467/+merge/5826211:17
lifelessadeuring: I've replied11:21
adeuringlifeless: thanks11:22
lifelessadeuring: as long as we don't penalise *every* read, I'm happy with most anything :)11:22
adeuringok ;)11:22
lifelesswallyworld: is https://bugs.launchpad.net/launchpad/+bug/761494 landable ?11:38
_mup_Bug #761494: picker doesn't save selected value into associated textfield <regression> <ui> <Launchpad itself:In Progress by wallyworld> < https://launchpad.net/bugs/761494 >11:38
wallyworldlifeless: being ui reviews as we speak. hope to sort it out tonight11:52
lifelesswallyworld: it needs UI review?!11:54
wallyworldlifeless: someone said it needed one on account of the javascript11:54
lifelessmeh11:54
lifelessui reviewer set is one person at the moment11:54
lifelesswe have many more folk fluent in js11:55
wallyworldlifeless: i sort of meant js when i said ui11:55
lifelessyah, curtis - the one ui reviewer - has proposed disbanding ui reviews11:55
lifelesswallyworld: ok, I've also read it now11:58
lifelessI haven't cross referenced YUI11:58
lifelessbut this is a major issue ; please land11:59
lifelessskip ec211:59
lifeless22:59 < lifeless> wallyworld: ok, I've also read it now12:00
lifeless22:59 < lifeless> I haven't cross referenced YUI12:00
lifeless22:59 < lifeless> but this is a major issue ; please land12:00
lifeless23:00 < lifeless> skip ec212:00
wallyworld__lifeless: ack12:01
lifelessI think we should consider cherrypicking it in fact, as it seems to be broken across everything12:01
lifelesss/cherrypicking/cowboying12:01
wallyworld__lifeless: yeah :-( if only windmill tests were not disabled, would have picked it up during ec212:02
wallyworld__lifeless: not having windmill leaves a huge hole in our testing safety net12:02
deryckMorning, all.12:03
lifelesswallyworld__: ack.12:03
lifelessmrevell: you have mail12:14
mrevellThanks lifeless12:14
lifelessmrevell: if you want to chat about it, I'm around for another 20 with the first 5 doing house stuff12:14
mrevelllifeless, I'll reply to the email, don't worry, it's not an urgent thing. I take your point and mostly agree with you, but I'll reply fully in the email.12:16
wallyworld__lifeless: fyi ec2 land has succeeded12:17
=== wallyworld__ is now known as wallyworld
lifelessmrevell: cool12:23
lifelesswallyworld: ec2 land? or bzr lp-land ?12:24
lifelesswallyworld: the former seems pointless with windmill disabled12:24
lifelessand will take 8 hours before its on qastaging12:24
wallyworldlifeless: ec2 land. bollocks. should have done lp-land12:24
wallyworldlifeless: i'll still do a lp-land now?12:25
lifelesswallyworld: yeah12:25
wallyworldlifeless: done and done12:28
lifeless\o/12:28
=== mrevell is now known as mrevell-luncheon
lifelessnight all12:50
=== al-maisan is now known as almaisan-away
=== almaisan-away is now known as al-maisan
=== mrevell-luncheon is now known as mrevell
LPCIBotProject windmill build #192: STILL FAILING in 1 hr 2 min: https://lpci.wedontsleep.org/job/windmill/192/13:59
LPCIBotProject devel build #653: FAILURE in 5 hr 0 min: https://lpci.wedontsleep.org/job/devel/653/14:02
* StevenK glares at Jenkins14:03
StevenKAh, no, that racy test *again*.14:03
=== salgado is now known as salgado-afk
deryckadeuring, henninge -- https://dev.launchpad.net/MaintenanceRotationSchedule14:06
lifeless*yawn* really sleep time14:07
jkakarG'night lifeless. :)14:16
=== Ursinha-afk is now known as Ursinha
jcsackettcan i get a review for https://code.launchpad.net/~jcsackett/launchpad/add-security-audit-utility/+merge/58199 from someone?15:14
=== al-maisa` is now known as almaisan-away
=== almaisan-away is now known as al-maisan
wgranthttps://code.launchpad.net/~wgrant/launchpad/quickly-drop-shipit/+merge/58312 would like a semi-urgent review.15:20
benjiwgrant: I'll do it. and then jcsackett's15:21
jcsackettbenji: thanks!15:21
wgrantbenji: Thanks.15:21
benjiwgrant: done15:21
wgrantThanks.15:21
=== salgado-afk is now known as salgado
benjijcsackett: I'm done with https://code.launchpad.net/~jcsackett/launchpad/add-security-audit-utility/+merge/58199.15:42
=== al-maisan is now known as almaisan-away
jcsackettthanks, benji.15:43
rvbaHi henninge, I'd like to ask you something about your change in security.py (forwardCheckAuthenticated).15:45
henningervba: sure15:46
rvbaI've got a test failing with "ValueError: ('Undefined permission id', <Distribution 'Distribution883444' (distribution883444)>)" (full stacktrace http://paste.ubuntu.com/596071/)15:46
rvba./bin/test -cvv distroseriesdifference test_package_diff_request_link15:46
rvbaI think the call (in forwardCheckAuthenticated) to check_permission_is_registered(permission, obj) should be check_permission_is_registered(obj, permission)15:46
rvbahttp://paste.ubuntu.com/596075/15:46
rvbabut maybe I'm wrong ... because if I'm right your fix would not have passed ec2 :)15:47
rvbahenninge: what am I missing?15:47
henningervba: I'd be surprised if that was the case because the test suite did not notice it.15:48
henningervba: looking15:48
henningervba: oops15:49
henningervba: so the adapters I changed are not being exercised by the test suite ...15:49
henningervba: is that a new adapter?15:50
rvbahenninge: from a few days ago15:50
bigjoolshe got pwned15:51
rvbaarg15:51
rvbahenninge: "./bin/test -cvv distroseriesdifference test_package_diff_request_link" uses the very adapter you mention in your email: EditDistroSeriesDifference15:51
henningethat is very strange15:52
rvbaindeed15:52
henningervba: I had failing tests but I re-ran them all locally and they passed.15:52
henningethat one was not amont them, though.15:52
henningeI guess that whole ec2 run was broken ;(15:53
henningervba: anyway, thanks. I'll submit a fix quickly.15:53
rvbahenninge: great thanks :)15:53
rvbahenninge: it would be nice to understand how this appended though ...15:54
henningervba: let me see if I still have that mail. It was very large so I might have deleted it.15:55
* rvba tests test_package_diff_request_link on a fresh branch15:56
* rvba confirms the test is failing on a fresh checkout of devel16:00
bigjoolsdid someone with microscopes for eyeballs set the latest font size?16:08
benjithere's some sort of Moore's law of web font sizes, they halve every 18 months16:10
bigjoolsha16:11
timrcjust set your resolution 800x600, problem solved16:12
bigjoolsor ctrl-mouseup!16:12
henningervba: I still have the test output. The failure did not happen on that run.16:14
henningehttp://paste.ubuntu.com/596085/16:14
rvbahum ...16:15
rvbathe mystery persists16:16
jelmerderyck: HI16:20
deryckHi jelmer16:20
jelmerderyck: Sorry, I mean: hi16:20
jelmerderyck: just saw your bug report; are you using the natty daily ppa?16:20
* deryck doesn't mind shouting hello :-)16:20
deryckjelmer, yes, I think so.  just added it via add-apt-repository on natty.16:21
* deryck looks16:21
deryckjelmer, yeah, http://ppa.launchpad.net/bzr/daily/ubuntu natty main16:21
jelmerderyck: hmm, odd16:21
jelmerderyck: thanks16:21
deryckjelmer, np16:22
=== deryck is now known as deryck[lunch]
cody-somervillebigjools, press ctrl + 0 to restore default zoom16:28
bigjoolscody-somerville: then it turns into something the size of a sparrow's arse16:28
cody-somervillebigjools, Firefox or Chromium?16:29
bigjoolsboth16:29
cody-somervilleWhat font size do you have set in Edit > Preferences, Content16:30
bigjoolsbut them I am 40, so my eyes ain't what they used to be :)16:30
cody-somerville?16:30
cody-somervillebigjools, lol16:30
bigjoolsno default font size16:31
sinzuijcsackett: mumble?16:50
=== deryck[lunch] is now known as deryck
allenapIs qastaging deploying or is it broken?17:55
mrevellNight!18:13
=== Ursinha is now known as Ursinha-lunch
LPCIBotProject devel build #654: STILL FAILING in 5 hr 0 min: https://lpci.wedontsleep.org/job/devel/654/19:03
benjiif anyone feels like reviewing a 600 line lint branch, this is your lucky day: https://code.launchpad.net/~benji/launchpad/lint/+merge/5837119:25
flacosteabentley: i suggest you send an email about bug #766337 to launchpad-dev19:26
_mup_Bug #766337: Should be possible to reload JSON cache <Launchpad itself:Triaged> < https://launchpad.net/bugs/766337 >19:26
flacostegiven JS-heavy work done in feature squad, it would be likely that one of the feature squad could do it as part of their work19:26
flacosteit's an elegant solution to a common problem19:26
abentleyflacoste: Okay.19:26
sinzuiallenap: I am looking at the qastaging issue. The error looks like the the lp tree was updated, but qastaging did not get the lp-production-config update19:33
jcsackettsinzui: do you know a good place for me to go digging into some of the specifics of how our exceptions are turned into OOPses? or who might be best to bug to figure that out?19:40
jcsacketti've realized that what i knew about for cron scripts tied to logging errors--if an exception occurs that isn't caught and logged, the logging oopshandler obviously isn't going to come into play. :-/19:40
sinzuijcsackett: A quick mumble will sort this out19:41
jcsackettsinzui: works for m.19:42
jcsacketts/m./me.19:42
=== Ursinha-lunch is now known as Ursinha
lifelesscan has review? https://code.launchpad.net/~lifeless/launchpad/bug-759467/+merge/5826221:00
allenaplifeless: I'll do it.21:51
allenapleedsliam: Hello chap :)21:52
lifelessevening allenap21:53
allenapEvening lifeless :)21:53
timellohi! Is there any launchpad method in the api that can do a search in packages like https://launchpad.net/ubuntu/+search does?22:02
lifelessProbably not, that search returns many different types of results and our api is (sadly) a bit difficult around heterogynous interfaces22:03
timello:/22:03
lifelesshttps://launchpad.net/+apidoc/devel.html is our development api doc22:03
lifelessreplace devel with 1.0 to see the stable version22:04
timellolifeless, yeah, I've being looking at this reference22:04
lifelesssinzui: hi22:10
lifelessbug 766561 - could that be related to our team notify change, or perhaps its more likely to be in gary_poster's baliwick ?22:11
_mup_Bug #766561: person in team A which is a subteam of team B not getting bug email for bugs team B is subscribed to <Launchpad itself:New> < https://launchpad.net/bugs/766561 >22:11
gary_posterit doesn't sound like us, fwiw22:11
lifelessgary_poster: hi; btw - bug 745799 - that is you :)22:12
_mup_Bug #745799: DistroSeries:+queue Timeout accepting packages (bug structural subscriptions) <timeout> <Launchpad itself:Triaged> < https://launchpad.net/bugs/745799 >22:12
sinzuiI changed merge notifications. Are teams not being notified about merges?22:12
lifelesssinzui: ah right, I thought we changed a helper function in there.22:12
sinzuiI did, but nothing but merge ever used it22:12
sinzuiIt will be months or years before we make everything queue messages22:12
lifelessgary_poster: so a couple/few weeks ago this team stopped getting mail22:13
lifelessgary_poster: they are utterly confused as to why22:13
gary_posterlifeless, I have the rest of the week to get our feature ready for review, and then two weeks for bug fixes after that.  Could you help me prioritize these within that context?22:14
gary_poster(no frustration there, just rrying to be clear with my situation)22:15
lifelessgary_poster: yup. I think we need to determine if this is real (e.g. are we -not- sending brad & others in that team email)22:15
lifelessif we are sending the mail, its at his end, phew, move along.22:15
gary_posterlifeless, IOW, drop everything?22:15
lifelessif we aren't, its a regression and I think its reasonable to say a maintenance squad should look at it22:16
gary_posteroh ok, drop everything to investigate?22:16
lifelessbut if it turns out to be related to your feature work, I think its probably better at that point to hand it over to you as you'll know whats changed recently.22:16
gary_postersure, I'm completely fine with that22:16
lifelessgary_poster: someone should; as its near your EOD I will either look myself or coordinate it22:17
gary_posterbut I would prefer to not have it affect my time this week22:17
lifelessI think the next step is log trawling :)22:17
gary_posterheh22:17
gary_posterI'm looking at the timeout too22:18
gary_posterI assume any oops is a reasonable starting point22:18
lifelessgary_poster: yeah; I've analyzed one in detail for repeated queries; the strucsub ones summed to 2.2 seconds22:22
lifelessfrom ~20 separate queries. I suspect its a per-bug potato programming issue that has been exacerbated by the [slight] increase in cost of determining structural subscription22:23
lifelessgary_poster: I'22:24
lifelessI don't think you should drop anything to work on the timeout22:24
lifelessit was a preexisting problem22:24
lifelessthe mail-not sending however may well need dogpiling.22:24
lifelessI'm looking in logs myself now.22:24
gary_posterlifeless, ok.  if we do have to work on reducing the structural subscription queries themselves, which I would understand in the abstract, it may require some...significant thinking.  If this is some page-specific thing exacerbated by some potato programming there specifically, I'd be much happier.22:26
gary_posterlifeless, re bug 76651, are you looking for 759176 in logs22:28
gary_poster?22:28
_mup_Bug #76651: Comment on change is lost when the bugtask change form returns with errors. <lp-bugs> <Launchpad itself:Invalid> < https://launchpad.net/bugs/76651 >22:28
lifelessgary_poster: well, what we'll need is a way to do email notifications for N bugs all at once22:28
lifelessgary_poster: +queue accepts uploads that may close some number of bugs22:29
gary_posterlifeless, I see22:29
lifelessgary_poster: so we need to be able to process a 'fix released' status change for N bugs in the space of one webapp transaction (+ the actual accept logic itself)22:30
lifelessgary_poster: not actually spool the mail - the deferred sending etc is still used AIUI22:30
gary_postergotcha22:30
gary_postersure22:30
lifelessone way would be to handoff the post to a script22:30
lifelessor we could batch better in the webapp, etc etc etc22:31
gary_postersure22:31
allenaplifeless: r=me.22:31
allenapGood night all.22:31
lifelessallenap: thanks! gnight22:31
lifelessgary_poster: yes I'm grepping22:32
lifelessnow I've unwedged my carob link22:32
lifelessbrads name is not there22:32
gary_posterlifeless to build a "process structural subscriptions for batches of bugs" would require a new set of functions, as you might guess...and the logic is generally very bug specific so that will be interesting.  I'm sure we can increase the possible N within a given timeframe, but it would be work specific to that page22:34
gary_posterand depending on how big N needs to be, asynchronous will be necessary eventually22:35
lifelessgary_poster: we have feature requests for batch actions on bugs, so perhaps not all that specific. Anyhow, I think we agree that this is an exacerbation not a direct regression; I'm fine with it being looked at holistically with performance rather than zomging it in your squad22:36
gary_postercool thanks22:37
gary_posterI'm particularly happy to look at it once we are on bug rotation :-)22:37
gary_posterso as far as the other bug goes22:37
gary_posteryour log search suggests that the bug is in fact in LP mail sending22:38
lifelessyup22:38
lifelesswe're choosing not to send to him22:38
lifelessI'm checking teamparticipation22:38
gary_posterah cool thanks22:38
lifelesshttps://launchpad.net/~brad-figg/+participation has 'the dell team'22:39
gary_posterlifeless, for scheduling, could I schedule investigation of that bug for Thursday, or does our QoS need to be higher for this?22:41
lifelessI think we should treat this as a showstopper22:41
lifelessits pretty key functionality22:41
gary_posterwell, I'm reasonably confident we have tests for the basic functionality22:41
gary_posterI would certainly hope so anyway :-/22:41
gary_posterthat's not a feature we would have developed, of course22:42
gary_posterso I suspect that this is an edge case22:42
gary_poster"private bug" is our current red flag there22:42
gary_posterso, would this be acceptable:22:42
gary_poster1) Verify ASAP that we have tests that show that the basic story of that bug is working.  If not, add test and fix.22:43
gary_poster2) If #1 did not reveal a problem, on a more scheduled timetable, investigate this particular edge case.22:43
gary_poster[stop]22:43
gary_posterPerhaps 1.5, investigate if "private bug" is the kicker22:45
gary_postersince that's obvious ATM22:45
lifelessI hate that ''https://launchpad.net/api/1.0/~dell-team'' in a web browser is ~ useless22:49
lifelessgary_poster: teamparticipation shows brad-figg as a member of dell-team22:51
lifelessin prod22:51
lifelessis there some way to get e.g. https://launchpad.net/api/1.0/~dell-team/+sub-teams to render the json ?22:53
gary_posterlifeless, tomorrow morning, I'll have someone on the squad investigate the bug.  My goal will be to fix or show in a LP test that basic team subscription sends emails as expected; and to fix or show that basic team subscription sends emails as expected for private bugs.  If neither of those are a problem, I will postpone further work unil next week unless you tell me otherwise (like, say, now ;-) ).22:56
gary_posterrender the json...I think I have tried horrible webclient hacks in the past but I have no recipe atm22:57
lifelessgary_poster: its team-team-person22:59
lifelessgary_poster: I would wager we only really test team-person22:59
gary_posterlifeless, yeah, sorry, understood.  only really test team-person: maybe so.  If so, we'll find it quickly23:00
gary_posterlifeless, I'm running away in minute or so btw.23:01
lifelessok, ciao23:03
lifelessI will continue coordination23:03
gary_posterthanks23:10
lifelesssuspiciously, the send-bug-mail script optimisation is in the suspect range.23:15
sinzuiwallyworld: ping23:28
wallyworldsinzui: hi23:28
sinzuiwallyworld: I will not be available for the stand up in 30min. My daughter demands that I pick her up from soccer.23:29
wallyworldsinzui: been there :-)23:29
sinzuiwallyworld: Can you convey my apologies.23:30
lifelessdo teams provide IPerson?23:30
wallyworldsinzui: will do. we could have it now?23:30
lifelessif so we have some rather confused code23:30
sinzuiwallyworld: if member so no mind23:31
wallyworldsinzui: ack23:31
sinzuilifeless: teams are IPerson with the addition of ITeam23:31
sinzuilifeless: They are IPerson until we execute __init__ in Person then they get the extra interface23:32
lifelessyeah, I thought so23:32
lifelessthanks23:32
lifelessI'm looking into this notification bug23:32
lifelessno joy so far23:33
=== salgado is now known as salgado-afk
lifelessit should end up in _get_recipients_for_team which is unchanged since before the issue23:37
lifelessat least, the webapp generation should do that23:39
lifeless i suppose its possible the mail sending backend is doing something special23:39
maxbAny obvious reasons why a dev instance of launchpad would be rendering links to https://launchpad.dev/+icing/revNone/.... (which of course 404) ?23:44
lifelessrun make23:45
lifelessIIRC its a make rule to update the revno23:45

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