/srv/irclogs.ubuntu.com/2012/11/20/#launchpad-dev.txt

StevenKwgrant: http://sourceforge.net/apps/trac/sourceforge/wiki/API => no bugs00:12
wgrantAh00:13
wgrantThere's a bug RSS feed, but that's it00:13
StevenKWhich doesn't help00:14
=== micahg_ is now known as micahg
StevenKwgrant: So I just created two bugwatches pointing to libav, bug 16 had a invalid bug number, and bug 17 had a valid one. After running checkwatches, bug 16 is still Unknown/Unknown but bug 17 is Fix Released/Medium03:28
_mup_Bug #16: "Swedish" and "Swedish (Sweden)" should be the same language <lp-translations> <Launchpad itself:Fix Released by daf> < https://launchpad.net/bugs/16 >03:28
_mup_Bug #17: System error <lp-translations> <Launchpad itself:Fix Released by carlos> < https://launchpad.net/bugs/17 >03:28
_mup_Bug #16: "Swedish" and "Swedish (Sweden)" should be the same language <lp-translations> <Launchpad itself:Fix Released by daf> < https://launchpad.net/bugs/16 >03:28
_mup_Bug #17: System error <lp-translations> <Launchpad itself:Fix Released by carlos> < https://launchpad.net/bugs/17 >03:28
StevenK_mup_: Go away03:28
StevenKwgrant: So I guess bug 634906 is fixed03:28
_mup_Bug #634906: An invalid remote bug ID can cause a checkwatches run to break completely <bugwatch> <lp-bugs> <oops> <Launchpad itself:Triaged> < https://launchpad.net/bugs/634906 >03:28
wgrantStevenK: Excellent.03:33
StevenKSo that's two nailed shut today03:33
StevenKThree if we deploy03:33
=== wgrant changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On-call reviewer: - | Firefighting: - | Critical bugs: ~180
StevenKwgrant: Turns out urlopen that initializeRemoteBugDB calls uses the default timeout value. Perhaps we should set it to 180 and move on?03:41
StevenKIt might be more sinister03:42
wgrantStevenK: Perhaps block the connection locally and see if you can reproduce the behaviour?03:42
StevenKThe urlopen call has a ensure_no_transaction decorator03:43
wgrantSure, we know it has no transaction open03:46
wgrantSince it doesn't get idle-killed03:46
StevenKRight03:46
StevenKI was wondering if that was what was causing the spining03:47
StevenKwgrant: Right, I have forbidden my computer to talk to libav's bugzilla03:52
wgrant-j DROP, I hope?03:53
StevenK Well, duh03:53
StevenKI don't want checkwatches getting port unreachable or wind of it03:53
StevenKIt's still trying03:58
StevenK36 dropped packets03:58
wgrantHmm hmm03:58
wgrantLeave it for half an hour and see what happens :)03:58
StevenKIt hasn't logged anything about libav03:58
StevenKWhich is naughty03:58
wgrantIt usually does03:59
wgrantOh, unless it's hung on some preprobe03:59
StevenKWhich is likely, it's trying to get the version03:59
StevenK2012-11-20 03:59:01 INFO    Updating 1 watches for 1 bugs on http://bugzilla.libav.org03:59
StevenKIt seems it gave up03:59
StevenKNow it's probably POSTing04:00
StevenK2012-11-20 04:01:08 INFO    Error updating http://bugzilla.libav.org/: http://bugzilla.libav.org: <urlopen error [Errno 110] Connection timed out>04:03
StevenKThat's disappointing04:03
StevenKurlopen uses socket._GLOBAL_DEFAULT_TIMEOUT if it isn't set, which is defined as object()04:19
StevenKsocket.create_connection will only call socket.settimeout if the timout isn't that04:19
StevenKIt could be a proxy thing04:23
StevenKsquid just keeps a open connection04:24
StevenKBut I'm grasping at straws04:24
wgrantPossibly04:28
StevenKwgrant: It may explain why neither of us can reproduce it locally04:30
StevenKAnd the squid I just installed gives a 50404:38
StevenKBut that's default config04:38
wgrantIIRC I even tried it from behind squid.internal04:40
wgrantAnd it worked04:40
StevenKStrange04:42
wgrantHave you checked the last couple of hangs?04:42
StevenKI have not04:42
StevenKBut I'll stop scratching my head over this and move onto glaring at Redhat XMLRPC04:43
StevenKwgrant: Ah ha. We send Bugzilla.time as a XMLRPC to redhat's tracker and get back Fault -3200005:58
wgrantStevenK: So the method to get the remote tracker's current time fails?06:00
StevenKYeah06:01
StevenK<?xml version='1.0'?>\n<methodCall>\n<methodName>Bugzilla.time</methodName>\n<params>\n</params>\n</methodCall>\n'06:01
wgrantOh, not even any params? Nice06:01
StevenKIs what we send06:01
wgrantI guess we'll need to file a bug with them :)06:01
StevenKI can't find any docs about Bugzilla.time06:02
wgrantWhat's the full traceback we get from them?06:02
wgrantI assume it works on other bugzillae06:02
wgrantbut theirs is broken06:02
StevenKwgrant: Yeah, it seems fine with others06:03
StevenKLet me hack xmlrpclib06:03
StevenK({'faultCode': -32000, 'faultString': 'Cannot compare a datetime to a regular scalar at /usr/lib64/perl5/vendor_perl/5.8.8/x86_64-linux-thread-multi/DateTime.pm line 1385.\n'},)06:04
StevenKThat's the full stack in xmlrpclib.close06:05
wgranthttps://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&version=4.2 might be your friend06:07
wgrantI suspect06:07
* StevenK stabs bugzilla06:07
StevenKwgrant: http://pastebin.ubuntu.com/1371847/06:09
StevenKThe first run is with the ooo proxy commented out, the second with the redhat one06:09
wgrantyep06:09
StevenKBleh06:10
StevenKCreate an account? :-(06:10
wgrantYeah06:10
wgrantIf only all bugtrackers supported arbitrary third-party OpenID providers :)06:10
StevenKbugzilla-owner@redhat.com is the registered owner, we could just mail them ...06:12
wgrantWell06:12
wgrantYou know we always ask for bugs to be filed06:12
wgrantLet's extend the same courtesy to them!06:12
StevenKwgrant: Bugzilla bug filed, LP bug updated, card created and marked as blocked06:24
wgrantDid you create a bugwatch so we can check its status? :P06:24
StevenKI can't!06:25
StevenKAnd noted that in the LP bug, so bugger off06:26
wgrantHm, why can't you?06:26
StevenKThe watch will never get updated06:28
StevenKThe RedHat bugtracker is disabled06:28
StevenKSo there is little point06:28
wgrantAh, right, I was thinking that if they'd fix it then we'd know06:28
wgrantBut not if it's disabled06:28
StevenKNot turning it back on, either. We can do without an extra 2500 OOPSes a day06:31
adeuringgood morning08:50
stubTrying to bludgeon buildout into running in offline mode is a real pain in the arse09:56
stubSomething is seriously broken when you get download errors when running in offline mode. WTF were you looking there in the first place stupid?09:58
czajkowskistub: you're being a bit harsh on yourself today09:59
=== frankban changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On-call reviewer: frankban | Firefighting: - | Critical bugs: ~180
stubUmm... don't we stop bugs being duplicates of each other?10:26
stuboh, nm10:27
=== almaisan-away is now known as al-maisan
=== rick_h changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On-call reviewer: frankban, rick_h | Firefighting: - | Critical bugs: ~180
smartboyhwHmm yesterday I asked about the release date of CoC not correct on Launchpad QAstaging. Now CoC 2.0 is official, but then the date is still not changed....13:17
czajkowskismartboyhw: date doesnt need to change13:21
czajkowskiit's up to date.13:21
smartboyhwczajkowski, you can't say that the CoC 2.0 is released in 2005 (or can you?)13:22
czajkowskiI can and I did :) it needs to be done that way for reasons  like making sure anyone who has signed previous versions are still valid.13:22
rick_hczajkowski: ah ok. Yea missed the date thing when I pushed13:24
czajkowskirick_h: no tis fine13:24
czajkowskichecked with wgrant and co that it needs to be that way13:24
czajkowskiso tis grand13:24
czajkowskithanks rick_h13:24
rick_hdoes look strange. been many moons since I did CoC stuff (e.g. signed it)13:24
czajkowskirick_h: like the blog post :)13:24
rick_hok, well I'll stop looking at what to fix then.13:25
czajkowskirick_h: :)13:25
czajkowskidont fix what's not broken :)13:25
czajkowskiI can find lots of bugs if you want to fix stuff :p13:26
rick_hheh, I'll just go back to fixing the bug I've been working on13:26
rick_hadeuring: ping, have time to help me out with something?13:34
adeuringrick_h: sure13:34
rick_hhangout ok?13:34
adeuringrick_h: sure13:35
czajkowskirick_h: you see errors like this before https://bugs.launchpad.net/launchpad/+bug/108113114:38
_mup_Bug #1081131: Specifications privacy: error when trying to change sharing settings <Launchpad itself:New> < https://launchpad.net/bugs/1081131 >14:39
rick_hczajkowski: stand up, but will peek in a few14:39
czajkowskinp14:39
rick_hczajkowski: I've not seen that exactly and the bug wanders a bit so not 100% sure. Most things consider owner.14:57
rick_hI've not looked at the sharing ui itself which is seems he got access to bug couldn't change. Maybe deryck or abentley can speak more to that specifically14:58
czajkowskinods14:58
czajkowskisorry for pinging you just saw you online :) wasnt picking on you :p14:58
rick_hthat's ok, I'm more than happy to play flight directing guy and wave you over away :P14:59
czajkowskilol14:59
czajkowskideryck: how is the little one post op ?14:59
czajkowskiall good I hope14:59
abentleyI haven't worked on the sharing UI.  It does sound like it should only be shown as editable to folks with Launchpad.Edit on the appropriate members.15:00
deryckczajkowski, hi.  she's good, thanks! A little sore this morning as you'd expect but she's getting better with each moment.15:09
czajkowskigreat to hear15:09
czajkowskiI had mine out as an adult and was back to eating as normal 3 days later.15:10
czajkowskibut bf had his out there last year, took him 2 weeks to recover15:10
czajkowskieveryone recovers differently15:10
deryckshe's eating pretty good now herself, albeit soft stuff like mac-n-cheese and mashed potatoes.15:10
czajkowskigood going15:11
deryckshe was starving.  but my little would consume the doors of the kitchen if we'd let her.15:11
czajkowskideryck: while you're here you might be able to shed some light on https://bugs.launchpad.net/launchpad/+bug/108113115:11
_mup_Bug #1081131: Specifications privacy: error when trying to change sharing settings <Launchpad itself:New> < https://launchpad.net/bugs/1081131 >15:11
deryckczajkowski, yes, was just looking at that.  I agree with abentley's assessment above.  we need to ensure we don't show the edit icons to the wrong folks.15:12
deryckczajkowski, but it does sound like we already block the action, per the bug's description/15:12
deryckczajkowski, I've traiged it now.15:13
derycktriaged it even15:13
czajkowskigrand job thanks folks15:14
=== matsubara is now known as matsubara-lunch
derycknew title to make it clearer, too.  bug 108113115:15
_mup_Bug #1081131: +sharing should not show edit icons if you don't have launchpad.Edit <private-projects> <Launchpad itself:Triaged> < https://launchpad.net/bugs/1081131 >15:15
rick_hderyck: or abentley either of you have a sec to review since I'm OCR? I'm particularly want to make sure I'm saying that the ProjectGroup part is correct. https://code.launchpad.net/~rharding/launchpad/filter_more_products/+merge/13516415:40
abentleyrick_h: I can have a look.15:40
rick_habentley: thanks15:40
abentleyrick_h: You are right about ProjectGroups.15:48
rick_habentley: ok cool. The use in top_projects had me thinking I was missing a connection that I would allow leakage through15:49
abentleyrick_h: Where possible, I've been updating APIs to accept a user parameter, rather than using ILaunchBag.user.  Have you looked at that option?15:49
rick_habentley: no, I didn't look. I'll chase it up to browser and see if I can add the user15:50
rick_hcargo culting usage fml heh15:50
abentleyrick_h: If you could, that would be great.15:50
rick_habentley: rgr, makes sense now that you say it15:51
=== matsubara-lunch is now known as matsubara
abentleyrick_h: Could you please review https://code.launchpad.net/~abentley/launchpad/proprietary-karma/+merge/135188 ?16:13
derycksinzui, I'm about to head out to lunch, but would love to catch some voice time with you after that if you have time.16:14
rick_habentley: sure thing16:14
abentleyrick_h: Thanks.16:14
=== al-maisan is now known as almaisan-away
=== deryck is now known as deryck[lunch]
rick_habentley: r=me, I bow before your storm-fu turning the manual sql into storm16:31
rick_hthough ugh at reading it16:31
abentleyrick_h: was a multi-step process, and I had to check with curtis about the meaning of the original.  Do you think I should try to clean up the storm version more?16:31
rick_habentley: I think it's just more I can read sql ok. So no, I think it's ok16:33
rick_has is16:33
abentleyrick_h: Okay.  Thanks for the review.16:34
=== yofel_ is now known as yofel
=== Ursinha-afk is now known as Ursinha
=== frankban changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On-call reviewer: rick_h | Firefighting: - | Critical bugs: ~180
rick_habentley: I'm looking at trying to update the usage to include the user in the call but hitting a TAL wall: https://pastebin.canonical.com/78762/17:40
rick_hany idea or example where the method in the TAL gets a parameter passed in?17:40
rick_hI'm bzr grep'ing through but not coming up with an example I can see17:40
sinzuirick_h, We don't want python in tal. It is difficult to test17:44
sinzuirick_h, add a helper to the view to call the context with the user17:44
rick_hsinzui: yea, I'm thinking of punting on the suggest for my branch tbh. It's just going to be a giant pita to make user a param to the changes17:44
sinzuirick_h, or ...17:44
* sinzui looks for cheat code17:45
rick_hsinzui: because doing that I hook the portlet to the view17:45
rick_hwhich I guess is ok since it's the one use but that seems dirty as well17:46
jcsackettrick_h: is this so you can get user in that method to do the privacy filter?17:47
sinzuirick_h, we have several cases where the mode must have a user, but the callsite will not pass it. The code will get the current request to find the user. maybe17:47
sinzuirequest = get_current_browser_request()17:48
sinzuiuser = request.user17:48
jcsackettyou can also get the user from the launchbag, can't you?17:48
sinzuioh, it is person = IPerson(request.principal)17:48
rick_hjcsackett: yea, that's what I did but the suggestion was to avoid it17:48
jcsackettrick_h: ah.17:48
rick_hso trying to figure out the best way to replace the launchbag with direct calls because hte usage is in a portlet .pt17:48
sinzuiyeah, the launchbag is used too much17:49
rick_hI think sinzui is right. The best thing is to add a helper on the view, but then it's ugly because the portlet is making the view api change17:49
=== deryck[lunch] is now known as deryck
rick_habentley: ok, comment added. Updated one of the 3 change points.18:29
=== rick_h changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On-call reviewer: - | Firefighting: - | Critical bugs: ~180
=== gary_poster|away is now known as gary_poster
jcsackettderyck: are you around, and can you review https://code.launchpad.net/~jcsackett/launchpad/404-project-milestones-privacy/+merge/13525221:55
deryckjcsackett, yes and yes :)21:55
jcsackettderyck: awesome. thanks!21:55
deryckjcsackett, on call now but will look shortly21:55
jcsackettderyck: works for me.22:01
sinzuiwgrant, StevenK: https://pastebin.canonical.com/78751/22:31
wgranthttp://launchpadlibrarian.net/122750088/qxTB0SY4SJMb6haWptIok9Gyind.txt22:34
sinzuiStevenK, https://bugs.launchpad.net/launchpad/+bug/66530722:41
_mup_Bug #665307: cronscripts/expire-bugtasks.py fails trying to put backtrace in librarian <lp-foundations> <Launchpad itself:Triaged> < https://launchpad.net/bugs/665307 >22:41
StevenKwebops: The NDT r16286 deployment is done, can you move it to Past Deployments?22:45
sinzuiwgrant, StevenK: https://bugs.launchpad.net/launchpad/+bug/68744622:45
_mup_Bug #687446: process-mail dies on malformed email addresses <email> <lp-foundations> <oops> <Launchpad itself:Triaged> < https://launchpad.net/bugs/687446 >22:45
deryckjcsackett, r=me22:47
=== matsubara is now known as matsubara-afk
jcsackettderyck: Awesome, thanks.22:52
deryckjcsackett, np.  sorry it took me so long.  had back to back calls.22:52
deryckgood night, everyone.22:55
slankStevenK: I had trouble finding your ping again ... that NDT is moved in productionstatus22:57
StevenKBAH, wrong window after all, too.22:58
StevenKslank: Thanks22:58

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