=== almaisan-away is now known as al-maisan
=== al-maisan is now known as almaisan-away
StevenKBleh, it doesn't OOPS00:15
StevenKIt gives a traceback directly00:15
StevenK2012-11-20 02:54:33 INFO    Error updating http://bugzilla.libav.org/: Failed to parse XML description for http://bugzilla.libav.org bugs [u'221', u'222', u'298']: mismatched tag: line 101, column 400:28
StevenKwgrant: So I'd expect a cronscript that tosses an exception to print out the OOPS number and exit, which doesn't happen. :-(00:50
wgrantStevenK: Well, have you changed the code to make it do that?00:51
StevenKI don't know how :-(00:51
StevenKI thought I had to add the OopsHandler in scripts.logger, but LaunchpadCronScript adds it anyway00:52
StevenKThere is a log_unhandled_exceptions_func function, but I'm not sure if that interacts with the handler, or what goes on00:52
wgrantWell, find out how the current handler works00:54
wgrantThat should tell you why the OOPS handler doesn't00:54
StevenKwgrant: I'm not sure if OopsHandler is even used by anything that isn't a test01:09
StevenKlogger.exception adds an ERROR, which should get tripped by the OopsHandler to give an oops01:31
StevenKwgrant: I've been staring at this too long. Can you glance at http://pastebin.ubuntu.com/1373869/ and see if you can find something I'm missing?01:32
wgrantWhat doesn't work?01:39
StevenKThe raise Exception in the midst of expire-bugtasks prints out a full traceback01:40
wgrantStevenK: OopsHandler in its current state is designed to log an OOPS for any message greater than a warning01:42
wgrantYou might want a separate one for exceptions01:42
wgrantHm, though it looks like it's meant to handle it01:42
StevenKLike I say, been staring at it too long01:44
wgrantStevenK: Hm, it works fine for me01:46
wgrantI get OOPSes out of it properly01:46
StevenKwgrant: Oh?01:46
StevenKCan I see the output?01:46
wgrantThere's no special output, because you need a custom formatter for that01:47
wgrantBut an OOPS is logged01:47
StevenKSo I need to grow a OopsFormatter01:48
wgrantProbably, yes01:48
cjwatsonwgrant: Hmm, if bug 817358 is specifically about lots of bugs, shouldn't it have been fixed by the introduction of ProcessAcceptedBugsJob?01:48
_mup_Bug #817358: Copying packages with lots of associated bugs can cause timeout <bugs> <package-copies> <timeout> <Launchpad itself:Triaged> < https://launchpad.net/bugs/817358 >01:48
StevenKwgrant: Maybe we just make LaunchpadFormatter do it01:48
wgrantcjwatson: Copies01:49
wgrantcjwatson: Not accepts01:49
cjwatson(I realise syncSource is always going to be somewhat terrible, but in this specific instance ...)01:49
cjwatsonwgrant: Yes, so?01:49
cjwatsonwgrant: The implementation goes through the same bits01:49
wgrantOh, I guess an in-app sync would use the same code...01:49
wgrantFair point01:49
cjwatsonTa.  Not that it affects the critical count01:50
=== wgrant changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On-call reviewer: - | Firefighting: - | Critical bugs: ~170
StevenKHmmmm, now if the handler has created an oops, how the heck do I get my hands on it01:55
StevenK... Why does the formatter get called first :-(03:02
wgrantStevenK: You may have to turn it all into a formatter rather than a handler, possibly03:07
StevenKLooking into a filter03:08
StevenKMight help03:08
StevenKI hope03:08
StevenK 5102 steven    21   1 1569m 1.5g 3312 R  89.1 19.3  11:43.05 bzr04:14
StevenKwgrant: https://code.launchpad.net/~stevenk/launchpad/destroy-librarianformatter/+merge/13531104:33
wgrantStevenK: Why do you hate freedom^Wtracebacks?04:33
wgrantIt would seem prudent to show both the traceback and the OOPS ID04:34
StevenKI can remove that bit of LaunchpadFormatter if you wish04:34
wgrantThat would be nice04:34
wgrantOtherwise if the OOPS goes nowhere then we have no idea what went wrong04:34
StevenKHealing: 389 lines of code04:35
StevenKwgrant: http://pastebin.ubuntu.com/1374074/04:37
wgrantAnd corresponding test changes, presumably04:37
wgrantBut yeah, that sounds good04:37
StevenKI've tossed it at ec2 test, so we'll see what the fallout is.04:37
wgrantI mean you added two tests for the OOPS ID04:38
wgrantThey are presumably now broken04:38
StevenKI did?04:39
wgrantThose were just in the description04:39
StevenKYes, that was a demo04:39
StevenKwgrant: Hmm, does my change kill your milliseconds logging?04:48
wgrantRemarkable timing04:49
wgrantThat logging is not mine, but stub's04:49
wgrantBut you still use LaunchpadFormatter, don't you?04:49
wgrantWhich should preserve it04:49
StevenKwgrant: But it was created differently based on the milliseconds argument04:50
StevenKLibrarianFormatter was a subclass of LaunchpadFormatter04:50
wgrantYou're right, yeah04:50
wgrantYou'll need to revert that bit04:50
stubLibrarianFormatter should die05:35
StevenKstub: I killed it05:35
stubYay. I would have but was scared of dealing with the test fallout05:35
StevenKSo far only one failing test05:36
stubBug #64110305:37
_mup_Bug #641103: LibrarianFormatter should die <lp-foundations> <Launchpad itself:Triaged> < https://launchpad.net/bugs/641103 >05:37
StevenKstub: You'll note there is an MP attached ...05:37
stubI could have sworn we supported multiple --log-file arguments05:58
* stub wonders why he didn't05:58
wgrantHuh, don't we?05:59
wgrantI'd always assumed that just worked05:59
wgrantBut I guess I've only ever used the default stdout -q/-v options plus a --log-file05:59
stubno, just noticed in the MP and had to double check. It would be trivial to support it06:02
nigelbbigjools: hahaha, gavinator07:04
bigjoolsthe meme commences07:07
lifelessoh no07:18
lifelessyou didn't07:18
adeuringgood morning08:21
czajkowskibigjools: you're a tad evil!10:04
czajkowskiat least this memme isn't filling up planet ubuntu ..yet!10:05
rick_hmorning party people11:46
mgzmorning rick_h11:47
=== matsubara-afk is now known as matsubara
=== broder is now known as broder_
jcsackettmorning all.13:53
czajkowskiello jcsackett13:53
=== allenap` is now known as allenap
adeuringabentley: https://code.launchpad.net/~adeuring/launchpad/bug-1056881/+merge/13537414:37
abentleyadeuring: ack14:37
abentleyadeuring: If a spec is made PROPRIETARY, the AccessArtifactGrant.grantor will be the person who made it proprietary, and so they will be able to unsubscribe anyone.15:25
adeuringabentley: yes15:26
adeuringabentley: well, provided that they had to grant an AAG15:27
abentleyadeuring: Right.  They won't be able to unsubscribe anyone who had an APG.15:27
adeuringabentley: is anything wrong with this logic?15:28
abentleyWell, it's not a great rationale for granting the ability to unsubscribe people.  Though of course I understand the limitation in the schema that makes this hard.15:31
adeuringabentley: right, but there also this logic: Basically, people with lp.Edit permission can also grant access to other people, and the same permission is needed to change the information type. And revoking grnats need again the same permission15:33
=== beuno_ is now known as beuno
abentleyadeuring: I'm in the check-in hangout, so I'll get back to this after.15:40
adeuringabentley: gah, forgot that completely..15:40
=== matsubara is now known as matsubara-lunch
jcsackettabentley: are you reviewing today? if so, can you look at https://code.launchpad.net/~jcsackett/launchpad/filter-in-getRequestTargets/+merge/13545716:00
=== abentley changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On-call reviewer: abentley | Firefighting: - | Critical bugs: ~170
jcsackettabentley: well, that answers my first question. :-P16:01
abentleyjcsackett: Yes, I'll look at it after I'm done with abel's.16:01
jcsackettabentley: awesome, thanks.16:01
jcsackettderyck: i was just looking over our cards, and i think there are some comments on bug 1043902 that make a strong case for dropping it. has there already been discussion incorporating that info?16:02
_mup_Bug #1043902: magic InformationType numbers in access grant related DB procedures <information-type> <tech-debt> <Launchpad itself:Triaged> < https://launchpad.net/bugs/1043902 >16:02
abentleyadeuring: This solution is an improvement, but ensure a bug is filed about the lack of a subscribed_by field in SpecificationSubscription, and reference it in the security adapter and bug comments.16:04
abentleys/bug comments/test comments/16:05
adeuringabentley: ok16:05
abentleyadeuring: You can address the too-long enum names by assigning the enum to a local variable.  That is what I do.16:07
adeuringabentley: I know...16:08
abentleyadeuring: Other than that, r=me.16:11
adeuringabentley: thanks16:11
deryckjcsackett, yes, I think that can be dropped, but would love to hear for sure from adeuring. Just in case there's some bit of what he's worried about that I don't understand.16:12
jcsackettderyck: fair.16:12
deryckadeuring, can we drop bug 1043902?16:12
_mup_Bug #1043902: magic InformationType numbers in access grant related DB procedures <information-type> <tech-debt> <Launchpad itself:Triaged> < https://launchpad.net/bugs/1043902 >16:12
adeuringjcsackett, deryck: Its not that important for me. My main reason for writing this bug was that I like to have a kind of additional check for things secuirty related.. FOr xample, if the meaning of an integer value is changed in Python code but not in stored procedures, this can have security implications. OTOH, LP devs tend not to be insane enough to this ;)16:14
adeuringso, its fine for me to drop this card16:15
abentleyjcsackett: in  getRequestTargets, having the user default to none means that calling code can accidentall omit it.  I think it should be a non-optional parameter, even though this means changing parameter order.16:16
jcsackettabentley: there's a small number of call sites, so that works for me. same comlaint for list_product_request_filters?16:17
abentleyjcsackett: Same complaint for list_product_request_targets16:18
jcsackettright, that's the one.16:18
jcsackettsmall set of sites there, so no worries.16:19
abentleyAside from that, this looks good.  r=me.16:19
jcsackettthanks. i'll make those changes.16:28
deryckadeuring, ok, thanks.  sorry was on call.16:38
deryckjcsackett, do you mind dropping both the bug and the card then?16:38
jcsackettderyck: on it.16:38
=== yofel_ is now known as yofel
=== matsubara-lunch is now known as matsubara
=== Ursinha is now known as Ursinha-afk
=== deryck is now known as deryck[lunch]
rick_habentley: review if you have the time. https://code.launchpad.net/~rharding/launchpad/translatables/+merge/135492 after talking with deryck went with the less impact approach18:20
=== Ursinha-afk is now known as Ursinha
=== deryck[lunch] is now known as deryck
abentleyrick_h: Hmm.  Perhaps we don't need to do this, since AIUI products cannot be proprietary if they have translation enabled.18:45
abentleyrick_h: That's not actually implemented yet, but https://bugs.launchpad.net/launchpad/+bug/1079785/comments/318:46
_mup_Bug #1079785: public artifacts are permitted on confidential projects <private-projects> <Launchpad itself:In Progress by abentley> < https://launchpad.net/bugs/1079785 >18:46
rick_habentley: ah, gotcha18:51
rick_hcompletely didn't think of that part18:51
rick_habentley: so I read that as we'd like to get to deleting things that would fall into here, but we don't currently so I'm going to go with let's make sure we don't leak for the moment?18:58
abentleyrick_h: I'm not thrilled with that approach, because we're not guaranteeing privacy now, and it makes more work for later when we have to disable the test and remove the code.19:04
rick_habentley: I guess if we have cards for handling the delete/clean up and the whole transitino process I'd be ok with burying this under that work.19:05
rick_hbut I don't want to leave a hole out there without something on the board that says it won't be a hole19:06
abentleyrick_h: Sure, let's do that.19:06
rick_habentley: so will yuor current branch then do all that is mentioned in that comment?19:06
abentleyrick_h: Probably.  I can certainly commit to doing the translation bit.19:08
rick_hOk, will put the branch in WIP and move the card back into the final with a note on it that it can be wiped once that's true19:08
rick_hand I suppose the other branch I landed yesterday can probably backed as well then19:08
abentleyrick_h: Right.  Sorry I didn't catch it then.19:09
rick_hoh well, I'll move that card back into final and alter it to a back-out work then.19:10
abentleyrick_h: I've added a misc card for fixing any existing data.19:12
jcsackettabentley: can you review https://code.launchpad.net/~jcsackett/launchpad/blueprints-in-ui-not-specification/+merge/13550319:51
abentleyjcsackett: Sure, but give me a few minutes.19:51
jcsackettabentley: no rush.19:51
abentleyjcsackett: r=me20:02
jcsackettabentley: thanks!20:06
=== abentley changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On-call reviewer: - | Firefighting: - | Critical bugs: ~170
sinzuiStevenK, wgrant, I am in a meeting. I may be 30 minutes late.22:01
=== gary_poster is now known as gary_poster|away

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