wgrantHm, we can toss another 10 appservers on chae without much trouble.00:02
lifelessI wouldn't expect that, unless our SQL/total ratio has changed dramatically00:04
wgrantIt hit 50% once in the last week.00:08
lifelessif its seeing less load, thats possible an haproxy config bug00:09
jelmerit's surprising how much people try to "fork" a branch on launchpad by registering a new code import...00:25
wgrantYeah :/00:27
wgrantPeople don't understand that in a DVCS you can just branch.00:27
wgrantBecause on GitHub you can't :/00:27
jelmerwgrant: though we do have a button to allow people to create an *empty* branch, which seems just as pointless00:33
=== wallyworld_ changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: wallyworld | Critical bugs: 253 - 0:[########]:256
=== StevenK changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: wallyworld | Critical bugs: 253 - 0:[#######=]:256
StevenKwallyworld_: I added an alert() to that function and it didn't pop up a message box?01:09
wallyworld_StevenK: did you add it just before the named_post?01:09
StevenKwallyworld_: Just before the if block01:11
StevenK(That is just before the named post)01:11
wallyworld_StevenK: did you make jsbuild and refresh the browser?01:13
StevenKI just ran make01:13
StevenKAnd the alert appears in launchpad.js01:13
wallyworld_make by itself should be ok but make jsbuild is quicker01:13
wallyworld_what ui element did you hit? the red sprite or the link?01:14
StevenKwallyworld_: The link in +subscriptions01:15
wallyworld_StevenK: we may be looking at the wrong file. i know about the subscription portlet.  there's a structural-subscription.js01:16
wallyworld_which may be used for the page you are looking at01:16
wallyworld_line 1205?01:16
wallyworld_StevenK: lp.registry.javascript.structural-subscription.js01:17
StevenKYeah, just found that01:17
wallyworld_that's the only other place i know of in js that unsubscribes someone01:18
StevenKThe do_io does the actual unsub?01:18
StevenKWhat a horrible function name01:18
wallyworld_yes, looks like it01:18
StevenKNo alert either01:22
wallyworld_StevenK: i'll run up a system and have a look.can you paste the url you are looking at01:22
StevenKwallyworld_: It's a local bug01:23
wallyworld_StevenK: but i can got to a bug and append +subscriptions i think01:23
StevenKwallyworld_: So what I did was run make-lp-user stevenk; then jumped into a harness and getUtility(IPersonSet).getByName() and then factory.makeBug(private=True, owner=user)01:24
StevenKIf that makes sense01:24
wallyworld_ok, so you are editing a private bug. there's one of those in the sample data i can look at01:24
* wallyworld_ waits for make to finish01:25
wallyworld_StevenK: i think it make be lp.bugs.lavascript.subscription.js line 21901:30
wallyworld_so there's 3 bits of javascript that are used to unsubscribe01:31
StevenKlavascript sounds good01:31
wallyworld_nope, that's not it either01:35
wallyworld_StevenK: it's the make_action_lin method01:37
wallyworld_line 101201:38
wallyworld_my breakpoint just got hit :-)01:38
StevenKAnd the on click function is internal to make_action_link01:39
wallyworld_at least you know where to start hacking now01:41
StevenKHow do I get at the bug in JS?01:42
wallyworld_StevenK: what do you mean? the link to the bug?01:48
wallyworld_there's a json cache entry LP.cache.bug01:49
wallyworld_from there you have LP.cache.bug.self_link or LP.cache.bug.web_link etc01:49
StevenKLP.cache.bug is undefined01:55
wgrant30 seconds...01:57
StevenKwgrant: ?01:58
wgrantUntil the OOPS report of doom.01:58
wgrantAlthough it seems to be late, unsurprisingly.01:58
StevenKIt's probably choking on 70,000 OOPS02:00
wallyworld_StevenK: there's LP.cache.context and LP.cache.context.bug_link. you can view page source and scroll to the end to see what's in the cache02:02
wgrant40814 DisconnectionError: could not connect to server: Connection refused Is the server running on host "lp-prod-pgbouncer.canonical.com" and accepting TCP/IP connections on port 5433?02:06
wgrant29665 AssertionError: Bug #504291: Store left in a disconnected state.02:07
_mup_Bug #504291: DisconnectionErrors (already disconnected) happening again <lp-foundations> <oops> <Launchpad itself:Fix Released by stub> <Storm:Invalid> < https://launchpad.net/bugs/504291 >02:07
wgrant 270 DisconnectionError: server closed the connection unexpectedly This probably means the server terminated abnormally before or while processing the request.02:07
wgrant  83 Fault: <Fault -1: 'Unexpected Zope exception: DisconnectionError: could not connect to server: Connection refused\\n\\tIs the server running on host "lp-prod-pgbouncer.canonical.com" and accepting\\n\\tTCP/IP connections on port 5433?\\n'>02:07
wgrantAnd that's all of them.02:07
StevenKSo we get to look forward to 60,000 OOPSes every time we do fastdowntime?02:08
lifelesswe need to teach oops-tools to know there was a deploy02:08
lifelessand hide them02:08
wgrantThose assertionerrors shouldn't happen, though.02:08
wgrantThere is a bug somewhere.02:08
wgrantEverything else looks pretty sensible.02:09
wgrantSo we are good to go for tonight.02:09
StevenKTonight? Seriosuly?02:09
wgrant1830: upgrade germanium and cocoplum02:09
wgrant1831: fastdowntime all the patches02:09
StevenKBecause what could possibly go wrong?02:12
StevenKwallyworld_: LP.cache.context doesn't have what I want, how is that populated?02:26
benjiStevenK: IJSONRequestCache(view.request).objects['my_thing'] = my_thing02:28
wallyworld_StevenK: there's 2 ways - automatically with the attrs from the page context, plus one can also instantiate an IJSONRequestCache object02:29
StevenKSo I suspect the bug is LP.cache.context, but I can't see private on there02:29
wallyworld_StevenK: see expose_user_subscriptions_to_js in lp.bugs.browser.structuralsubscription.py02:29
wallyworld_StevenK: if context were bug, then all the attrs would be there02:30
wallyworld_so it's probably something else02:30
StevenKLots of context in those views :-/02:34
StevenKLooks like ... part of a bug02:41
StevenKIt's a task02:42
StevenKSo LP.cache.context.bug_link is the link to the bug02:43
* wgrant stabs Melbourne.02:45
wgrantIt was nice and cloudless and not freezing yesterday, today it is hailing.02:46
wgrantIt was almost getting springish, now back to mid-winter for a while :(02:46
StevenKIt's raining here02:48
StevenKJS sucks04:29
=== almaisan-away is now known as al-maisan
jtvwgrant: it's turning out to be another day of long unexpected phone calls, but here's what we discussed earlier.  Wanna review?  https://code.launchpad.net/~jtv/launchpad/transitional-published/+merge/7472005:58
wgrantjtv: There you go again, making perfectly good classic Soyuz tests *readable*.05:59
jtvI could add some dire but incomprehensible XXX comments in the names of people who have long left the company, if you like.06:00
wgrantStuff like kiko's "this is very funny" gina XXX is good, too.06:01
jtv"XXX kibonzi 2004-03-31: ON NO ACCOUNT should this plearfy the.  If get this wrong, ENTIRE ARXHIVE become problem.  DO NOT CHANGE without asking permission from Celve Alexelo."06:02
wgrantjtv: For dominate.py, do you know of the store.find(SourcePackagePublishingHistory, archive=series.main_archive, distroseries=series, [...]) Storm shorthand?06:02
jtvYes, just don't like it much.06:02
wgrantIndeed, it makes extension of queries difficult.06:02
jtvI like the wider spacing and avoidance of ambiguity in the "==" notation.06:03
wgrantI am normally tempted to use the kwarg notation for stuff like SPPH, but you have taken the probably better alternative of aliasing it.06:03
jtvAlso, I like to think of our database as a relational one.  Which involves so much more than just equality comparisons!06:03
wgrantOtherwise you end up with 5 line conditions.06:03
jtvI don't like doing the aliasing very much, but I still prefer it over the kwargs shorthand somehow.06:04
jtvAlso, this is transitional code.  It will disappear soon, so I'm not too bothered about it anyway.06:04
jtvwgrant: extra points if you can spot everything that's wrong with that XXX I gave earlier.06:05
wgrantI could be cruel and make you fix gina.txt's lint, but perhaps not.06:05
* jtv snaps at wgrant06:05
jtvI know what island you live on.06:05
wgrantBetter still, I'll make you fix it, and then I will petition to have the standard changed to something else, requiring every line to be reindented again.06:06
jtvYou're getting disturbingly good at this.  I may have to call your mommy.06:06
wgrantHey, I'm not the one making scarily plausible XXXs.06:07
jtvPlausible?  2004-03-31?  Nah, we have code reviews to catch mistakes like that.06:08
jtv(Damn, should have written 2004-04-31 obviously)06:08
jtv*That* is what we need: an automated checker for XXX dates in PQM, which will fail our branches because PQM thinks the date is in the future.06:09
* wgrant returns to necromancy.06:12
wgrantSadly, many of these binaries are being rescued from the pool only so they can be immediately reaped :(06:12
jtvwgrant: I take it this is not about my branch?06:15
wgrantIt's recovery from some different mass publication update queries from years ago :)06:16
jtvAh.  And… they're binaries that ought not be reaped?06:17
wgrantThey ought to be reaped.06:17
jtvIs reaped really a regular verb?06:17
wgrantBut they cannot be, because we don't have them in the librarian any more.06:17
wgrantreapt is too close to crept!06:17
jtvAhhh that one.06:17
wgrantSo I must rescue them from the pool back into the librarian, so they can be reaped from the pool, and then the librarian.06:17
wgrantSort of.06:17
jtvGetting the patients all healthy and rosy-cheeked for the executioner, eh?06:18
jtvThanks for the review BTW.06:18
wgrantAh, yes, sorry, got distracted by your XXX abomination.06:18
jtvAbomination even?  Glad you liked it.06:21
jtvwgrant: about the multiple-publications-of-same-version issue…  domination orders by (SPR.version, SPPH.datecreated).  I wonder if it shouldn't be something like (SPR.version, SPR.id, SPPH.datecreated).  The issue affects conventional and new domination, really.06:25
jtvCan there be 2 published SPPHs for the same SPR for the same (Archive, DistroSeries, Pocket, SourcePackageName)?06:25
jtv(And similar for BPPHs etc.)06:25
jtvSorry, for two _different_ SPRs for the same (Archive etc.)06:26
jtvSo two SPRs for the same version of the same package, and each with their own Published SPPH in the same (archive, distroseries, pocket).06:26
jtvIs that at all possible?  If so, what if the SPRs are ordered differently in time than the SPPHs are?06:27
wgrantIt's not legal to have two SPRs in one archive with the same name and version, because they don't fit on disk.06:27
wgrantThat doesn't mean it hasn't happened.06:27
wgrantIt has.06:27
wgrantBut they should never both be published, because they can't both be on disk at once.06:27
jtvSo domination doesn't need to go out of its way to accommodate this situation.06:28
wgrantProbably not. The publisher will crash before it gets into that situation.06:28
jtvWhy are so many words ending on -ation nowadays?  Used to be different when I was your age, I can tell you.06:28
wgrantBecause it will see the publication marked as Pending, try to write it to disk, then blow up when it sees a different file already there.06:28
wgrantSo the conflict never gets marked as Published.06:28
wgrantIt just OOPSes forever until we SQL it out.06:28
jtvGood enough for me.06:29
jtvSo then, would it be fair to say "if a single live version of a package has multiple Published pub records (within the same archive yadda yadda) then the most recent of those pub records dominates the rest of them, and they should become superseded"?06:30
jtvAnd it specializes neatly to the classic domination algo, so that's all dandy.06:31
wgrantSo... I guess you just sort by SPPH.datecreated, and drop a version from the live set once you see it?06:31
wgrantMm, but that's moving the logic into dominatePackage, which is slightly upsetting.06:32
jtvThe details get a bit intimate with the implementation, so step back:06:32
wgrantBut maybe.06:32
jtvdominatePackage goes through a list of SPPHs sorted by descending (SPR.version, SPPH.datecreated).06:32
jtvIt remembers the last live… SPR IIRC that it saw.06:33
jtvIf it encounters a non-live version: supersede by the last-live whatever-it-remembers.06:33
jtvIf it encounters a live version: leave it as it is, but record it as the last live whatever-it-is.06:34
jtvWhat I need to add then, I think, is "is this SPPH the same as for the last live version?  If so, supersede it as well."06:34
=== al-maisan is now known as almaisan-away
jtv(Alternatively I could compare to the last-seen version; same results, assuming the choice of dominant doesn't change, but probably slightly more code)06:36
jtvAnyway, tests first, then look at the implementation details.06:36
nigelbwgrant: Is the font fix okay?06:52
wgrantnigelb: It looks good, but it's not on production yet.06:54
nigelbwgrant: cool, okay :)06:55
wgrantThanks for sorting that out.06:56
nigelbWhen I look at https://bugs.launchpad.dev/debian/+source/mozilla-firefox/+bug/3/+activity06:57
_mup_Bug #3: Custom information for each translation team <feature> <lp-translations> <Launchpad itself:Fix Released> <Ubuntu:Invalid> <mono (Ubuntu):Invalid> < https://launchpad.net/bugs/3 >06:57
nigelboh wait, .dev06:57
_mup_Bug #88545: Abolish the "statusexplanation" database field <lp-bugs> <tech-debt> <Launchpad itself:Triaged> < https://launchpad.net/bugs/88545 >06:58
nigelbI see statusexplanation06:58
nigelbEven though I removed all references to that field in this branch06:58
nigelbIs that normal?06:58
wgrantSounds like you didn't remove all of them.06:58
wgrantnigelb: Oh.07:03
wgrantnigelb: Nevermind, that's fine.07:03
wgrantnigelb: I was misunderstanding.07:04
wgrantnigelb: If you look at the content of the bugactivity table, you'll see that there are changes with whatchanged == 'statusexplanation'.07:04
wgrantnigelb: We could delete them too, but there's no need and little benefit.07:04
wgrantErasing history is bad :)07:04
nigelbwgrant: ah, that's bugactivity table07:06
nigelbI tried to find the table07:06
nigelbmy psql skills are still not 1337 enough :P07:07
wgrant\d bug<TAB><TAB>07:07
wgrantOr not.07:07
wgrantlaunchpad_dev=# \d bug07:07
wgrantDisplay all 228 possibilities? (y or n)07:07
nigelbI did a \dd07:08
nigelband looked in there07:08
nigelbwas that wrong?07:08
wgrant\dd is more than just relations, so it's far bigger than what you want.07:08
wgrant\d shows just tables/views/sequences, so is more interesting.07:08
=== wallyworld_ changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: - | Critical bugs: 253 - 0:[#######=]:256
nigelboh, just missed wallyworld_ :(07:10
nigelbwgrant: Out of panic, I did the ++profile++show page and looked at the sql queries ;)07:10
nigelbhrm, abel isn't in yet.07:11
nigelbI'll have to ask later for my destroy branch07:11
henningewgrant: Hi! ;)07:36
wgrantThis can't be good!07:36
henningewgrant: The process that logs to buildd-manager.log is the buildmaster, i.w. running on build-master host?07:37
wgranthenninge: The process is buildd-manager, running on cesium, also known as builddmaster or occasionally buildmaster.07:37
henningewgrant: and it gets reststarted during a nodowntime rollout, right?07:37
henningeok, just wanted to be sure.07:38
henningeI added extra logging output at the end of a ttbj (elevated it from debug to info) but I don't see anythting.07:38
henningeJust the "starting templates build j..."07:39
henningeso these builds must die before they reach the WAITING state.07:39
henningeI will have to try this out locally or on dogfood.07:40
henningethanks wgrant ;)07:40
adeuringgood morning07:54
=== almaisan-away is now known as al-maisan
jtvhey there mrevell, bigjools08:17
nigelbMorning mrevell / bigjools :)08:25
nigelbadeuring: Hi, are you OCR-ing today?08:26
adeuringnigelb: yes. do you need a reviewß08:26
=== adeuring changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: adeuring | Critical bugs: 253 - 0:[#######=]:256
nigelbadeuring: Yep - https://code.launchpad.net/~nigelbabu/launchpad/destroy-statusexplanation-88545/+merge/7470408:26
adeuringnigelb: ok, I'll look08:27
StevenKnigelb: One thing to note is to ask your reviewer to land it with the --incremental flag -- the branch doesn't fix the linked bug entirely.08:37
nigelbStevenK: ah, ok08:41
StevenKnigelb: The bug is about the existance of the column, you haven't removed it yet. :-)08:42
nigelbStevenK: AHH, right. That gets completely removed in the next step :)08:46
StevenKnigelb: Right!08:46
nigelbStevenK: First time doing such a thing, will remember next time :)08:49
StevenKnigelb: Which is why I'm helping you :-)08:49
nigelbStevenK: Do I get a destroyer badge now? :P08:58
StevenKThe destruction isn't nearly wide-spread enough08:59
StevenKsteven@liquified:~/launchpad/lp-branches/no-more-staticdiff% bzr di -r 13779..13783 | diffstat -s09:01
StevenK 21 files changed, 53 insertions(+), 597 deletions(-)09:01
StevenKNow that is wide-spread destruction.09:01
* nigelb isn't that invasive09:01
StevenKThat was a fun branch. Does that look like StaticDiff or something that supports it? Yes, remove it.09:02
adeuringnigelb: a very nice  branch. Just one question: Shouldn't lib/lp/bugs/stories/bugs/xx-bug-activity.txt be changed too? I get the error "psycopg2.OperationalError: fe_sendauth: no password supplied" when I try to run the test. An unrelated problem, but I can't answer this question myself...09:05
rvbaadeuring: see William's recent email to -dev about this.09:05
nigelbadeuring: Is that the one with the table?09:05
rvbaadeuring: and Hi btw ;)09:05
nigelbadeuring: No, it doesn't need to be fixed. I asked wgrant earlier about this and he said history is a good thing :)09:06
adeuringnigelb, wgrant: ok, I'll look. SImply missed the mail09:06
nigelbThat's actually in bugactivity and not entirely related09:06
nigelbI faced the test trouble too last night :)09:06
adeuringnigelb: ah, sure, so, r=me.09:09
nigelbadeuring: \o/ Can you land it for me as well. And as StevenK said, with the --incremental flag ;)09:11
adeuringnigelb: sure. And your reminder about --incremental may already answer my question: Would you like to work on the remaining usage of statusexplanation too, especially the DB patch?09:13
nigelbadeuring: Yes, I would09:15
nigelbIn fact, I'll probably be starting on it tonight09:15
adeuringnigelb: cool, many many thanks109:16
al-maisanI am getting errors when uploading a source package to a ppa .. is LP down for maintenance..?09:30
jmlal-maisan: no. Otherwise it would say on identi.ca/launchpadstatus and in the topic here.09:31
al-maisanthese are the errors: http://pastebin.ubuntu.com/685841/ -- and they are persistent09:32
al-maisanhmm ..09:32
bigjoolswgrant: looks like poppy didn't reconnect?09:32
cjwatsonjtv: are all new Debian SPPHs permanently PUBLISHED from here on out, or will new ones still be created as PENDING for a while?09:34
jtvcjwatson: all Published.09:34
jtvExcept _maybe_ (I'm not sure about this yet) ones that are copied when creating a new release.09:35
cjwatsonjtv: OK.  The change broke a swathe of ubuntu-dev-tools API scripts that were assuming PENDING, so I'll have to get that changed quickly09:35
jtvcjwatson: ouch!  I'm sorry, I thought this difference had to be inconsequential since PENDING seemed so pointless.09:35
jtvIn the Launchpad codebase we have a helper variable active_publishing_status that consists of PENDING and PUBLISHED.09:36
cjwatsonwe had various getPublishedSources queries that had to be special-cased for Pending before09:36
wgrantjtv: Aren't they still created as PENDING until the next deployment?09:36
wgrantcjwatson: Hmm, getPublishedSources should default to PENDING and PUBLISHED.09:36
wgrantcjwatson: So it should have DTRT.09:36
jtvwgrant: true, but that's not far off.09:36
cjwatsonwgrant: we were overriding the default :-)09:36
wgrantcjwatson: Right, but it shouldn't have been necessary :(09:37
cjwatsonbut hey, that makes it easier, I'll just delete some code.09:37
cjwatsonthat will presumably have the side-effect that now we'll notice Ubuntu uploads before the publisher runs09:37
wgrantbigjools: Argh.09:37
wgrantbigjools: @read_transaction should have handled that...09:37
jtvMeanwhile, Soyuz-capable reviewer needed for the next piece of the Gina-the-Dominatrix puzzle: https://code.launchpad.net/~jtv/launchpad/bug-845326/+merge/7474109:39
wgrantbigjools: Can you arrange for it to be bounced?09:39
wgrantbigjools: And probably a bug filed for maintenance people to look at.09:40
cjwatsonwgrant: I don't see the getPublishedSources default you mention - can you point me to it?09:40
cjwatsonOh, and could somebody attempt to land https://code.launchpad.net/~cjwatson/launchpad/multiarch-translations for me?  Yay for fastdowntime at last. :-)09:41
jtvcjwatson: I'll take it09:41
cjwatsonHm, that's marked as for merging into db-devel09:41
cjwatsonthat should be devel, yes?09:41
stubyer, superceed the proposal and it lets you retarget it.09:42
wgrantcjwatson: Ah, distroseries.getPublishedSources has something like that default, but archive doesn't.09:42
wgrantWow, those are bad APIs.09:42
cjwatsonwgrant: I guess I can just pass status=('Pending', 'Published') then09:42
jtvcjwatson: the multiarch-translations are not attached to a bug?09:43
wgrantcjwatson: Are you using the one on DistroSeries or Archive?09:43
jtvcjwatson: if you're building this on top of the Launchpad codebase, use active_publishing_status.09:43
wgrantcjwatson: If Archive, status=['Pending', 'Published'] should work, yeah.09:43
wgrantAnd if you are using DistroSeries.getPublishedSources, then stop because it shouldn't exist :)09:44
cjwatsonjtv: bug> no, I never got round to filing one.  Is that a problem?09:44
cjwatsonjtv: active_publishing_status> this is an API script09:44
jtvcjwatson: a bit—our Q/A process revolves around the associated bug.09:45
cjwatsonI can file one now then, one moment09:45
jtv(Oh thank you pidgin for telling me _now_ that Colin asked me a question.  Why didn't I just start using Windows so I could get the beta experience all the time?)09:45
jtvbigjools: realistically I expect I need either you or william for that review, simple though it be… https://code.launchpad.net/~cjwatson/launchpad/multiarch-translations09:46
jtvAhem, not that one09:46
jtvI mean https://code.launchpad.net/~jtv/launchpad/bug-845326/+merge/7474109:47
cjwatsonjtv: bug 84547509:49
_mup_Bug #845475: Separate translations out of Packages to reduce download requirements for multiarch clients <Launchpad itself:New> < https://launchpad.net/bugs/845475 >09:49
jtvthanks cjwatson — I'll attach it if necessary09:49
cjwatsonwgrant: I can't pass status=['Pending', 'Published'] through the API.  It only wants one.09:52
rvbawgrant: as part of fixing bug #827608, I'm currently chaning _latestSeriesQuery to return SPPHs instead of SPRs. I know I'll have to fix quite a bit of code after that change but I really think this is doable ... do you see a reason why I should not do that?09:53
_mup_Bug #827608: Sync requester isn't credited with upload <derivation> <Launchpad itself:In Progress by rvb> < https://launchpad.net/bugs/827608 >09:53
jtvcjwatson: annoying… have you tried both tuple and list?  We have some code that gets finicky about that.09:53
cjwatsonjtv: yep, tried both09:53
wgrantcjwatson: Hah. The Python API accepts both, but the interface for lazr.restful is declared to only accept one.09:53
cjwatsonwgrant: oh, and this is Archive, yes.09:53
wgrantcjwatson: That is unpleasant.09:53
jtvcjwatson: did you just resubmit your merge proposal?09:54
wgrantal-maisan: The poppy error is just cosmetic, and we're working on upgrading to a fixed version now. Your upload should still have gone through fine.09:54
jtvand hi al-maisan :)09:55
al-maisanah, I see, thanks wgrant !09:55
al-maisan.. and hello to my favourite fascist reviewer: jtv :)09:56
al-maisanhow are you my friend?09:56
jtvAfter calling me a fascist, that sounds as if you're rhetorically asking in what way I am your friend.  :)09:56
jtvWonderful here though, thanks.  Hope it's the same for you.09:57
cjwatsonjtv: yes, it's https://code.launchpad.net/~cjwatson/launchpad/multiarch-translations/+merge/74743 now09:57
al-maisanjtv: I called you a "fascist reviewer" .. please observe the context ;)09:57
jtval-maisan: True.  And I must have upset the wrong people because I've been convicted to heaps of Soyuz work.09:58
al-maisanjtv: yup .. doing great, thanks!09:58
wgrantal-maisan: (poppy fell victim to our first real test of our new 2-minute-downtime DB patch process -- we're still ironing out the kinks, as you can see)09:58
jtvcjwatson: so it's for devel now, instead of db-devel?  It has no dependency whatsoever on the db patch?09:58
al-maisanjtv: uh-oh .. but you are enjoying soyuz .. admit it :)09:58
wgrantjtv: The DB patch is applied!09:58
al-maisanwgrant: change is always difficult :P09:59
wgrantBut we can apply DB patches daily now, so all is wonderful.09:59
jtval-maisan: http://paste.ubuntu.com/685825/ http://paste.ubuntu.com/685826/09:59
wgrant... apart from poppy.09:59
cjwatsonjtv: Yep, I held off asking for a landing until the DB patch got applied.10:00
jtvwgrant: ah, I wasn't sure about this because it was for db-devel.  Anyway, I'll check the minimum of required boxes & land.10:00
cjwatsonjtv: The new MP is for devel.10:00
wgrantjtv: I think it's all going to be a bit confusing like that for a month or two while people get used to the new hotness :)10:00
cjwatsonjtv: When I initially prepared this branch, I got caught up in some merge target confusion because the changes to fastdowntime were just starting.10:00
jtvYes, spotted it, thanks.  Unfortunately the resubmit also seems to have re-requested reviews.10:00
wgrantAnd while we sort out all the pending MPs.10:00
wgrantcjwatson: I think the process has been redesigned twice since you started the branch.10:01
cjwatsonjtv: I don't think it should actually need those db reviews now10:01
wgrantYou did rather well.10:01
cjwatsonBut yes, it would need a normal review10:01
jtvIt's the same review it went through before, no?10:01
jtvI mean, the code changes had already been approved, right?10:01
jtvIt was an approved MP with pure code changes after all.10:02
cjwatsonThe only change since the code review was a merge from db-devel10:02
jtvOK.  It's going into EC2.10:02
cjwatsonThanks.  Let's see how that goes ...10:02
jtvIt's been working for me today.10:03
jtvUnfortunately the oneiric beta version is causing me some very serious trouble, with minute-long seizures and disappearing emails.  It seems to be related to dbus and X.  The bug for the most immediate symptom seems to be getting attention so let's see what happens.10:04
cjwatsonRestarting compiz occasionally helps.10:05
jtv(The disappearing emails seem to be from TB being too busy to move emails from one folder to another; they probably turn up somewhere in the end)10:05
jtvHow do I restart compiz?  Actually I'm not even sure I'm running it.10:06
jtvI was on unity2d but in oneiric I can't tell the difference any more.  Which is probably a sign that they're being unified, which is good especially given the name.10:06
wgrantjtv: When you hover over a launcher icon, is the tooltip black and transparent?10:07
wgrantArgh, unity 2D's are now transparent too :(10:07
wgrantBut they're a much lighter grey.10:07
wgrantThat's how I distinguish between them.10:08
wgrantBlack = 3D, grey = 2D10:08
jtvIt looks dark-gray and dithered; I can't see anything from the background through it.  Pretty much how it used to look in unity 2d.10:08
cjwatsonpkill -9 compiz (note: will shuffle all your workspaces among windows), or log out and back in.  Although I don't think unity-2d uses compiz; check with 'pgrep compiz'10:08
al-maisanjtv: ah, I see, you are having too much fun .. they should reduce your wages ;)10:08
wgrantAlso, 'unity' now defaults to restarting everything.10:08
wgrantAnd it even works without DISPLAY set, from back in natty when you needed to restart several times a day.10:09
jtvSo I'm still on 2d.  Strange, because I think the new login screen (wonderful, apart perhaps from the delay before the asterisks for my password appear) was telling me I was logging into "ubuntu" as opposed to "ubuntu 2d"10:09
wgrantSo you can just switch to a TTY, log in, and 'unity'10:09
jtval-maisan: heck, they even let me train little fascists now!10:09
al-maisanoh no!10:09
jtvcjwatson: nope, pgrep compiz returns 110:09
jmljtv: https://bugs.launchpad.net/unity-greeter/+bug/83877710:09
_mup_Bug #838777: Really slow shortly after booting <Unity Greeter:New> < https://launchpad.net/bugs/838777 >10:09
jtvthanks jml, as always concise and devastatingly accurate!10:10
jtval-maisan: I just graduated one, and when the other day I got to do some temporary mentoring for another, nobody wanted a review all of a sudden.  :)10:11
wgrantjtv: Oh, you're not rvba's permanent mentor? :(10:11
wgrantI want more clones of you.10:11
jmljtv: np. I've been spending a lot of time filing bugs against oneiric recently. I've even managed to get some of them fixed.10:11
jtv*Entire channel stares at wgrant*10:11
jtvyay for jml10:11
al-maisanjtv: for some inexplicable reason I can relate to that :)10:11
* jtv laughs10:12
nigelbwgrant: More clones of jtv? Are you sure?10:17
cjwatsonjtv,wgrant: bug 845487, FYI10:18
_mup_Bug #845487: Debian source publication checks have broken <ubuntu-dev-tools (Ubuntu):New> < https://launchpad.net/bugs/845487 >10:18
jtvthanks cjwatson10:20
wgrantcjwatson: Note that publications will shortly in the next few days be marked Superseded/Deleted as appropriate.10:20
wgrants/shortly //10:20
jtvnigelb: according to my IRC client it took you 5 minutes and 54 seconds to recover enough from wgrant's statement to formulate that question.10:21
nigelbjtv: I was at lunch :)10:22
rvbanigelb: quick lunch then ;)10:22
jtvcjwatson: to be precise: for packages that gina no longer sees in Debian, the last publication of the highest version will become or remain Published and the rest will become superseded.10:22
cjwatsonThat's fine10:23
jtvFor packages that it _does_ see in Debian,10:23
nigelbjtv: But yeah, took a while to recover :D10:23
cjwatsonI'm pretty sure ubuntu-dev-tools only cares about the most current one in a series10:23
jtvif it only sees a newer version in Debian than in the DB, the publication will become Superseded.10:23
jtvIf there are multiple publications of the same version in the same series/archive/pocket (e.g. because of a component change), the older publications will be superseded.10:24
jtvThe newest publications for remaining versions become/remain Published.10:24
jtvAny publications of versions that are _newer_ than anything that's currently seen in Debian will become Deleted.10:24
jtvThat's the “transitional” stage of Gina domination.10:25
jtvOnce that has run, we'll get more aggressive: if the package is no longer in Debian, any remaining Published publications become Deleted.10:25
=== jelmer is now known as Guest27145
=== Guest27145 is now known as jelmer_
jtvSoyuz-capable reviewer needed!  https://code.launchpad.net/~jtv/launchpad/bug-845326/+merge/7474110:32
=== al-maisan is now known as almaisan-away
jtvCoincidence I'm sure.10:36
jtvcjwatson: I could have a go at bug 845486.  My API knowledge's a bit rusty though.10:39
_mup_Bug #845486: cannot pass sequence to Archive.getPublishedSources through API <Launchpad itself:New> < https://launchpad.net/bugs/845486 >10:39
cjwatsonmine is more so :-)10:42
jtvOur API infrastructure really has a very 1990s notion of polymorphism, doesn't it?10:44
jtvcjwatson: maybe it should accept, as alternatives, a “status” _or_ a sequence of “statuses.”10:45
cjwatsonOh, yes, good point, breaking existing code that passes a single status would be bad!10:46
jtvWe'll have some callsites in Launchpad that need to be converted, but that's unlikely to pose many problems.10:47
jtvIt's still ugly, of course, but I'm not sure we can do better without breaking API compatibility.10:47
cjwatsonjtv: How so?  The existing LP code already accepts either a status or a sequence of statuses; it's only the API declaration that's less flexible10:48
cjwatsonoh, unless you mean as different parameter names10:48
jtvYes, the only proper thing to do would be to convert status=[…] in argument lists to statuses=[…]10:48
jtvWhich is something we've been wanting to do.10:48
jtvEven nicer, arguably, would be to convert all callsites to use statuses.  But given API compatibility I don't think we can retire the "status" parameter anyway.10:49
jmljtv: I think the status-as-list-or-string pattern is used by searchTasks.10:53
jtvadeuring: would you perhaps be available for a pre-implementation discussion as a change from regular OCR duty?10:53
jtvjml: how does the API deal with it there?10:53
jtvThe web-service API I mean?10:53
* jml looks10:53
henningeDo the builders still run on lucid?10:58
jmljtv: it's declared as a list, but apparently passing a string goes through ok. There is then a helper that ensures it's returned sanely.10:59
jmljtv: it's all a bit convoluted.11:00
jtvjml: convoluted sounds bad.11:00
jmljtv: but I think the convolution is mostly searchTasks being weird for other reasons.11:00
jmljtv: rather than to handle this case11:00
jtvhenninge: yes, they run on LTS11:01
henningejtv: thanks11:01
jtvjml: it'd certainly give us an easier fix if this works.  Still a bit misleading though to have a strict type system and then lie about the type.11:02
jmljtv: agreed. although, as an API user, I'd be mightily irritated by having to spell multiple statuses for bugs differently to multiple statuses for source package publishings.11:03
jtvjml: what I hear you say is that maybe the strict type system is more to blame here than the lying.  :-)11:03
jmljtv: well, it's not a strict type system.11:04
jmljtv: it's a lau(d|gh)able attempt to build one in Python.11:04
jtvYou had me there for a moment.11:04
jtvOkay, I'll try to get the List trick to work then.11:05
jtvAlthough in the past I've been bitten by the two ways to test the API: one was fast and followed very dynamic (and for me, convenient) type rules.  The other was slow but more realistic, and applied WADL's type rules.11:06
jmlIBugTaskSearch.importance (or status) has the 'type declaration'. I think BugTaskSearchParams.fromSearchForm does the string->list stuff, I *think*.11:06
* jtv looks11:07
=== matsubara-afk is now known as matsubara
jtvjml: any idea where importance could be passed in through the web UI?  Beacuse IBugTaskSearch.importance is an attribute, but I'm looking for a parameter.11:10
jmljtv: sorry, I don't understand.11:10
jmljtv: I mean, I don't understand why you care. If I were to figure out the answer, it'd be by grepping for fromSearchForm, since that sounds an awful lot like something used in the UI.11:11
jmljtv: If you want to see it used as an API parameter, then look for searchTasks.11:13
nigelbOkay, so I landed a branch that removes the usage of the field that I want to kill.11:16
nigelbNow how do I proceed?11:16
rvbanigelb: Create a branch off db-devel with a db patch in it: https://dev.launchpad.net/PolicyAndProcess/DatabaseSchemaChangesProcess11:19
nigelbexcellent, thanks!11:21
nigelbWow, this process looks detailed.11:31
nigelbI'm going to spend 1 hour trying to read and understand first11:31
bigjoolshaha - I just started documenting Soyuz.11:31
jtvHe's gone mad.11:32
jtvIt had to come to this.11:32
=== bac changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: adeuring, bac | Critical bugs: 253 - 0:[#######=]:256
nigelbHrm, I wonder if the DatabaseSchemaChangesProcess wiki page needs update post the new faster db updates11:36
adeuringjtv: sure. And sorry for the late answer, was in the other room for lunch11:45
jtvadeuring: actually we already had some discussions here, sorry11:50
wgrantnigelb: I believe it's already up to date.11:54
wgrantnigelb: The old process was abolished almost two months ago, because fastdowntime was meant to be done 6 weeks ago :)11:55
StevenKwgrant: Can I put together a patch to drop FKs, then? :-)11:55
wgrantStevenK: I was going to look at that next week, once we've had a clean run with rvb's patch on Monday night.11:57
wgrantSince tonight was not entirely smooth, I don't want to go doing anything quite so big just yet.11:57
StevenKwgrant: I was thinking we do it piecemail, TBH11:58
nigelbwgrant: Oh. Ok.11:58
wgrantStevenK: No, it should be one hit.11:58
wgrantSince we only apply one patch a day.11:58
wgrantAnd there's like 32 tables or something.11:58
StevenKwgrant: Do you have a patch already, or shall I look at writing it?11:59
StevenKI'll make a start on it, anyway11:59
wgrantI have one that's not quite fastdowntime capable.11:59
wgrantI will fix it weekend or Monday.11:59
StevenKIt deals with everything, including staticdiff?12:00
wgrantNo, but it's trivial to extend.12:00
StevenKIt's too slow for FDT, or what?12:00
jmlso, did LP actually have a low downtime db patch rollout?12:09
jmlthis means fastdowntime from now on?12:10
StevenKjml: We can only do fastdowntime, lifeless removed the monthly process from us.12:11
jmlStevenK: well yes, but two days ago you couldn't even do fastdowntime.12:11
bacmrevell: fwiw, i just bought a batch of vouchers and they showed up promptly12:15
mrevellCheers bac. Do you have any idea what might have gone wrong for Vadim?12:16
wgrantjml: Yep, we can do fastdowntime daily now.12:16
wgrantjml: Not completely smooth, but we know how to avoid the problems that show up.12:16
bacmrevell: nope.  all i can think of is the proxy was stuck.12:16
bacor dead12:16
wgrantjml: Got dozens of DB patches for us yet?12:17
mrevellbac, Cheers, well, I'll keep an eye out for similar things in future ... if we end up with a couple it might be easier to work out what's going on. Or maybe not.12:17
jmlwgrant: nope :) you put me off, so now I'm doing the data generation stuff locally first and postponing the "Where does the data live?" question.12:20
wgrantSounds like a good plan.12:21
bacadeuring: are you currently working on any reviews?12:21
adeuringbac: no12:21
wgrantjml: We have 270 tables... I would personally like to discourage adding more that aren't really too directly relevant.12:21
bacadeuring: cool.  i'll claim one of rvba's12:21
adeuringbac: ok12:21
StevenKwgrant: So tempted to land populate-bprc12:22
jmlwgrant: I'm pretty sympathetic to that line of argument.12:22
wgrantjml: I can certainly see the convenience.12:22
wgrantjml: But even LP is discouraged from putting tables in LP's database now.12:22
jmlwgrant: anyway, if I do the scraping & data gathering and just dump to a sqlite db locally for now, that'll help me make progress.12:23
wgrantMaybe we will not be in limbo when you are done :)12:23
jmlThe last time I did limbo it was on rollerskates.12:24
wgrantThat sounds inevitably disastrous.12:25
rvbabac actually, I had to recreate branches from code that was already approved yesterday by jcsackett.12:27
rvbabac: [...]-spph-creator-code is simply the extraction of the code that was previously in [...]-spph-creator12:28
bacrvba: so jcsackett approved a combined branch and you pulled them apart?12:29
rvbabac: exactly.12:30
bacrvba: good to know.  i'll look at them with one eye tied behind my back12:30
rvbabac: I had to refactor that due to the fact that we want to separate code change from db changes completely for FDT.12:31
bacrvba: right12:31
benjihenninge: do you have any answers to my questions on https://bugs.launchpad.net/launchpad/+bug/742662 ?12:33
_mup_Bug #742662: Mixed new line markers causing OOPS while importing translations <bad-commit-13889> <oops> <qa-bad> <rosetta-imports> <Launchpad itself:In Progress by benji> < https://launchpad.net/bugs/742662 >12:33
henningeHi benji!12:34
henningebenji: There is no oops or similar, we came to that conclusion just by looking at the code.12:35
henningebenji: https://code.launchpad.net/~benji/launchpad/bug-742662/+merge/7425412:35
henningebenji: line 302 of the diff12:35
* benji looks.12:35
henningebenji: you add a call to verifyNewlineConsitency to the importer.12:36
henningebenji: that will throw an exception12:36
henningeif the newlines don't match.12:37
henningelet me go to the code12:37
=== almaisan-away is now known as al-maisan
benjihenninge: true, it will throw an exception, which is caught in POTemplate.importFromQueue and POFile.importFromQueue (it wasn't being thrown before, but the catch has been there for a while)12:40
henningebenji: I am sorry, I am just realizing that I was wrong12:41
henningebenji: it *was* being thrown before12:41
benjihenninge: well, it was being thrown, but during imports (only during TTW updates of translations)12:43
benji(TTW = "through the web")12:43
henningebenji: well, there is storeTranslationsInDatabase12:44
henningebenji: which calls sanitize_translations_from_import12:44
henningebenji: so, that was already throwing the exception12:46
henningebenji: I may be missing what we gain from your, code though?12:46
henningeyour code, though12:46
benjihenninge: I don't know that you're missing anything, because now I'm confused12:47
henningebenji: but I am sorry, I did not look far enough to realize that these exceptions are indeed being caught.12:47
henningebenji: the fact that we have such data in the database is old data that was imported before sanitization was working properly.12:48
benjiah!  that answers my next question12:49
benjiin that case I'm inclined to add some tests to the already existing code and then figure out how to kill the pre-existing bad imports12:49
henningebenji: what I find interesing as well is that the bug mentions "importing translations" in the title but refers to web submissions.12:53
henningebut I guess that is just Diogo getting mixed up with the terms.12:54
benjicould be, or he (like I) assumed that the data was recently added12:54
henningebut the OOPSes in the bug are not from the imports12:55
benjitrue, but the reason we get OOPSes is because bad data got in to start with (or rather, we redefined what bad data was after the fact)12:55
henningeyes, so we need to live with that fact.12:56
henningebenji: so, the fix for those OOPSes would be to not check the English string again when submitting translations.12:57
matsubarahenninge, I didn't change the description to include "importing translations". the original bug report only mentioned the mixed newline oops raised.12:57
henningematsubara: hm, the bug does not say it's title or description were ever changed.12:59
henningebenji: ah, I see: the newline style is derived from the English original (it all comes back to me)13:01
matsubarahenninge, isn't it bug 742662? see the change between comments #2 and #313:01
_mup_Bug #742662: Mixed new line markers causing OOPS while importing translations <bad-commit-13889> <oops> <qa-bad> <rosetta-imports> <Launchpad itself:In Progress by benji> < https://launchpad.net/bugs/742662 >13:01
henningebenji: that is when the OOPS gets raised13:01
henningematsubara: sorry /me is blind13:01
henningematsubara: you are right13:01
henningebenji: so the fix is the following:13:02
henningeWhen sanitizing translations from the web ui, the MixedNewlineMarkersError needs to be caught and cause newlines not to be normalized13:04
deryckMorning, all.13:04
henningebecause we don't know what the English original uses.13:05
henningeWhen sanitizing translations on imports, the exception should be treated as before.13:05
henningeFor bonus points it would be great to not abort the import of the file completely but just report an error for this one string, like is done with other errors. But that is really outside the scope of this bug.13:06
henningebenji: is that understandable?13:07
benjithat sounds quite reasonable13:07
henningeHi deryck! ;)13:07
henningebenji: cool ;)13:07
henningebenji: I updated the bug.13:13
benjihenninge: great, thanks!13:14
StevenKCan haz review https://code.launchpad.net/~stevenk/launchpad/kill-malone.bugmessage_owner/+merge/74775 ?13:16
bacStevenK: i'll review it next13:18
henningebac: too slow... ;-)13:18
henningeStevenK: r=me13:18
bacthanks henninge.13:19
henningewhich reminds me ...13:19
StevenKhenninge: Thanks!13:19
henningeDo we have provisions for community reviewers?13:20
henningeAnother week and I'd be one.13:20
jtvI don't think that ever really got off the ground.13:20
jtvOtherwise I would have gotten al-maisan to review my branch today.13:20
StevenKhenninge: You can, but I still think it requires sign-off by a Canonical person13:21
henningeThat's ok, I'll probably be too busy anyway ...13:22
al-maisanhenninge: what is going to happen in a week's time?13:24
henningeal-maisan: I am leaving Canonical on the 16th.13:26
al-maisanah, I see -- good luck with your future endeavours !13:27
henningeal-maisan: thank you!13:28
henningederyck: Audio problems, rebooting.13:31
henningederyck: I just had a great business idea  - "Cake Pieces Online World-wide"13:42
deryckFor Virtual Going Away Parties Everywhere! :)13:42
henningeyou pick your type of cake online and send a number of persons a piece each with a little note.13:43
StevenKbigjools: Can you make bug 396097 a dupe of jtv's gina domination bug?13:43
_mup_Bug #396097: Debian import doesn't remove packages <lp-soyuz> <Launchpad itself:Triaged> < https://launchpad.net/bugs/396097 >13:43
henningederyck: exactly ;-)13:43
nigelb"gina domination" sounds like something that should probaly be not read outo of context13:43
nigelbhenninge: you can sell to Canonical and Mozilla  - two companies with lots of remote teams ;)13:44
bigjoolsStevenK: jtv can even do it!13:44
jtvStevenK: you want it, _you_ do it!13:45
jtvnigelb: see my merge proposal entitled “Gina the Dominatrix”13:45
nigelbjtv: srsly?13:45
nigelbI just googled for it.13:45
StevenKjtv: I don't know the bug number, sorry13:46
nigelbI wonder what google now thinks of me.13:46
StevenKAnd I thought jtv was EOD'd13:46
flacostenigelb: you'll get interesting ad link now ;-)13:46
jtvStevenK: I guess you want bug 83089013:46
_mup_Bug #830890: Gina should delete publications if removed from source archive <derivation> <qa-ok> <Launchpad itself:In Progress by jtv> < https://launchpad.net/bugs/830890 >13:46
nigelbflacoste: Surprisingly, no ads. Noone thought of it!13:46
jtvStevenK: and I'm trying to be EOD because I'm OD'ed on Soyuz.  But I gave Julian one "end of this week" estimate too many and I want this done.13:46
jtvMy main blocker for today: getting a reviewer.13:46
StevenKjtv: So marked13:47
flacostenigelb: there is a business opportunity there!13:47
nigelbflacoste: Haha, I thought so too. We could rick roll them into contributing to gina ;)13:47
jtvflacoste: thanks for pointing out the Google angle.  These are after all the people smart enough to charge their customers for showing you "spam recipe" ads in your spam mailbox.13:47
rvbabac: if you're interested in the follow up branch ;) https://code.launchpad.net/~rvb/launchpad/sync-bug-827608-credit-copy/+merge/7476913:49
bacrvba: ok13:49
rvbabac: thanks.13:49
bacrvba: in your MPs you should list the db branch as a pre-req, no?13:49
rvbabac: well, the way I understand the new fdt process it's not a pre-req technically is it?13:50
bacrvba: good question.  you can't run your test without it...13:50
bacrvba: but in terms of getting a good diff you may be right13:51
nigelbGRAR. Test failure :(13:51
rvbabac: True, I've been advised by lifeless to put the sql patch in my tree, unversioned.13:51
adeuringnigelb: that's what the EC2 tests are for :) The fix shouldn't be difficult13:52
nigelbNo, but I won't be able to get it for another 1 hour at least.13:52
rvbabac: we will see if I'll be able to put it up for review before I EOD but a final branch is coming up. Then it will be the end of the saga :)13:52
nigelbCan I work on the followup branch before this one lands?13:53
StevenKnigelb: You can13:53
nigelbYay, not as depressing then13:54
adeuringnigelb: the follow up branch is the one with the DB patch? If so, sure, continue to work on it. But we should not land it before the other one lands13:54
nigelbadeuring: AH, ok13:55
deryckabentley, codehosting runs for me.  on a freshly up to date devel.13:57
deryckabentley, didn't try that, sorry.  Thought you couldn't get it to run.  Will try now....13:57
abentleyderyck: It ran for me, but I couldn't push to it.13:58
deryckabentley, what's a local push command?  Don't remember, or haven't done it before.14:00
abentleyderyck: bzr push lp://dev/~mark/+junk/foo14:01
bigjoolsis the person picker supposed to time out?14:02
deryckabentley, literally try that?  Or push to some user I create for myself locally?14:02
abentleyderyck: literally mark.  You should have .ssh/config configured for that user.14:03
jmllifeless said that Launchpad was growing a table that had same/similar data to Contents.gz14:03
wgrantjml: Yes.14:03
wgrantjml: It exists, but is not populated.14:03
wgrantabentley: Not by default any more: use utilities/make-lp-user instead.14:03
jmlwgrant: ok. ta. Exposed over API?14:03
jmlmake-lp-user is the best!14:04
abentleyjml: A table of Contents.gz ?14:04
wgrantjml: Not at this stage, but that is part of the rationale for having it in LP.14:04
jmlwgrant: cool. thanks.14:04
jmlabentley: not *that* similar.14:04
deryckabentley, yup, works fine for me.14:04
abentleyderyck: okay, thanks.14:04
jmlalso, getPublishedSources doesn't seem to have a date_published filter14:05
deryckabentley, np.  Sorry it took me so long.14:05
jmlis that by design or simply because no one has wanted one yet?14:05
jml(or am I silly for wanting one?)14:05
wgrantjml: created_since_date?14:05
jmlwgrant: Well, that filters on SPPH.datecreated, afaict14:06
wgrantjml: Do you really care about datepublished?14:06
wgrantThat's usually a mistake.14:06
jmlwgrant: and I don't really know enough to know if I want that or SPPH.datepublished14:06
wgrantWhat is your use case?14:06
jmlwgrant: well, I don't know :)14:06
jmlwgrant: things that have been published recently14:06
wgrantcreated_since_date=whatever, status='Published'14:06
jmlsince I last checked, so I can maintain a local mirror of some of the data in packages.14:07
jmlhappily, that's what I'm doing now.14:07
wgrantIf you want to be really quick, do a separate lookup for Pending.14:07
jmlwhy so?14:07
wgrantThen you can grab the files directly from LP, and have the data before they hit archive.u.c.14:07
jtvcjwatson: I'm afraid I haven't been able to make much headway with the getPublishedSources status arg.14:10
jmlwgrant: cheers.14:10
cjwatsonjtv: we'll probably just end up using status='Published' across the board for a while.14:11
jtvcjwatson: it shouldn't be long before Published is the universal answer.14:11
bacrvba: in the changes for https://code.launchpad.net/~rvb/launchpad/sync-bug-827608-credit-copy/+merge/74769 you've changed the ordering of the results, which will be visible in the on the browser page.  have you verified that doesn't cause confusion?14:11
jmlYou know, I think maybe REST isn't such a good idea.14:11
cjwatsonSome of the callers may want to do an extra query for status='Pending' for the same reason wgrant lists above.14:12
wgrantjml: It's a great idea.14:12
wgrantjml: It just doesn't really work.14:13
jmlwgrant: +114:13
jtvIsn't that what mark said oh, pretty much off the bat years ago?  Me, I still don't know.14:13
bigjoolsanything involving xml sucks14:14
wgrantI am not seeing XML's involvement here...14:15
jtvI'll tell James next time I see him though.14:15
bigjoolsit's a general observation14:15
jtvNot very often these days, but second-to-last time he said "I certainly hope I won't still be using XML in 10 years' time.  That would be kind of depressing."14:16
jcsacketti have just had to rebuild a system, and i'm getting gnomekeyring.IOError when i try to run ec2 test/land. i seem to recall this being a problem once upon a time -- anyone remember the issue?14:16
nigelbHas anyone tried Android development? Its Java and XML.14:16
jmljcsackett: not I, sorry.14:16
jtvnigelb: thanks for pointing that out.  Now I feel even better about my basic old nokia.14:17
nigelbjtv: I have a basic sony errikson :)14:17
jtvjcsackett: I've seen that… IIRC it was some package that you're assumed to have installed.14:17
wgrantjtv: Ah! I am glad it's not just me who doesn't have a smartphone.14:17
jcsackettjtv: hm. so now i must muddle through to find said package. at least it's a direction to investigate, thanks. :-)14:18
jtvwgrant: flexispy.com is one strong argument not to have a smartphone14:18
nigelbwgrant: So, the advantage of smart of being connected all the time. I don't think you stay away from the computer (ZING!)14:18
nigelbToo hasty to make a funny remark14:19
jtvjcsackett: I can dig up some more probably-unhelpful clues from my memory: it was something gnome and kde had their own packages for, and somehow it was assumed that you'd have it pulled in in one of those two ways.14:19
jcsackettjtv: so, i know that there's like the wallet and the manager, but i have the full gnome version of that.14:19
jtvjcsackett: now to find the python package…14:19
jcsackettaaah. it's not just paramiko, then.14:20
jtvbigjools: there's not much at xmlsucks.org14:21
jtvAnd by the way, nigelb: flexispy is also all Java and XML.14:21
jtvdumb is fine by me14:21
nigelbjtv: *stab*14:22
bigjoolsjtv: there used to be more14:22
henningeSo, the launchpad-buildd runs on lucid but it requires bzr-builder >=0.5 which is not package in lucid. How do I solve that?14:33
henningejtv: ^ do you have an idea about that?14:34
henningeI mean it must have been solved for our current builders somehow.14:35
jtvhenninge: one of the bzr people may know.14:35
henningeyeah, I tried in #bzr ;)14:35
jtvor lamont may know.14:36
wgrantjelmer_: ^^14:36
wgrantAre the cat packages available in a PPA somewhere?14:36
wgrantThere may be a PPA owned by ~launchpad or ~bzr for them.14:36
bigjoolsjames has one IIRC14:36
henningewgrant: yes, that's what I was hoping ;-)14:36
wgranthenninge: lp-buildd also works fine on maverick/natty, FWIW.14:37
wgrantAnd dapper and karmic.14:37
henningebut production buildd's run on lucid, right?14:37
jtvOh God I can't take these soap series any more. I think I'll curl up with my kindle.  Have a good weekend, everyone!14:38
wgrantUh. It's complicated.14:38
bigjoolso/ jtv14:38
wgrantI forget what ia64/sparc/powerpc/hppa are running these days.14:38
wgranthenninge: But the virt builders should all be lucid, yes, and I guess that's all you care about.14:38
henningejtv: see you next week14:38
henningewgrant: I guess I do, yes. ;-)14:39
henningebut none of james's ppas has that.14:39
wgrantActually, the virt builders might still be hardy...14:39
wgrantBut anyway, the series doesn't matter too much.14:40
henningewgrant: thanks14:40
wgrantBecause all that's run outside the chroot is bzr-builder, basically.14:40
jelmer_wgrant: sorry, back now14:40
wgrantAnd that is heavily backported14:40
jelmer_henninge: the newer bzr-builder is in the cat archive14:40
henningewhat is the cat archive?14:41
jelmer_henninge: and there is an even yet newer bzr-builder/bzr for the buildds which I have backported here and is due to land on the buildds at some point: ppa:canonical-bazaar/cat-proposed14:41
jelmer_henninge: an internal apt repository used by IS14:41
jelmer_henninge: ppa:bzr/daily should also have a newer bzr-builder for lucid14:46
cjwatsonhenninge: "Canonical Admin Team"14:47
henningejelmer_: I feel stupid but where do I see these ppas in the web ui?14:48
henningeAlso, I don't really want the latest, I just want >=0.5 ;)14:48
jelmer_the canonical-bazaar/cat-proposed PPA is private, the bzr daily PPA is here: http://launchpad.net/~bzr/+archive/daily14:49
henningeoh, +archive14:51
henningejelmer_: I don't see bzr-builder in the ~bzr ppas :(14:55
jelmer_henninge: you're right - sorry. We don't seem to have a recipe for that, I'll see if I can add one14:56
cjwatsonCan anyone explain http://paste.ubuntu.com/685964/ to me?  I don't understand it, and it doesn't seem to have much to do with my branch15:01
cjwatsonShouldn't the "..." match "api.launchpad.dev/beta"?15:01
henningecjwatson: I think that is noise15:03
henningeThis is probably the real culprit:15:03
henninge    - suite_names:15:03
henninge    -     [u'Release', u'Security', u'Updates', u'Proposed', u'Backports']15:03
henninge    ? ^^^15:03
henninge    + suite_names: [u'Release', u'Security', u'Updates', u'Proposed', u'Backports']15:03
henninge    ? ^^^^^^^^^^^^15:03
henningeNo ... involved here.15:04
henningeFailing doctests suck.15:04
cjwatsonThat's odd though, because my branch doesn't touch anything that should change wrapping there AFAICS15:04
cjwatsonI did add a column to distroseries, but why does that change wrapping ...15:05
cjwatson    + include_long_descriptions: True15:05
cjwatsonI bet *that's* the real culprit15:06
henningecjwatson: yes, fooled again ...15:06
henningeDid I mention that "Failing doctests suck?"15:07
henningeSo easy to miss stuff in all the noise.15:07
jmlI wonder why it does that.15:08
jmlI bet it's because diff reporting doesn't interact with the NORMALIZE_WHITESPACE and ELLIPSIS options.15:09
henningeIt always does that to me: One failure and it complains about all ellipsis in the file.15:09
cjwatsonI suppose I'd better try to test that locally before resending to EC215:10
nigelbadeuring: Hi, can you re-land this branch for me? Fixed the test https://code.launchpad.net/~nigelbabu/launchpad/destroy-statusexplanation-88545/+merge/7470415:10
adeuringnigelb: sure15:10
jmlit just passes expected and observed to difflib15:11
jmlwith no processing on expected15:11
jmlHmm. Shouldn't be *too* hard to fix, actually.15:13
jmlAnd by 'fix', I mean, "come up with a workaround that doesn't require patching the stdlib"15:13
jelmer_henninge: there is a daily build in progress now, should hopefully land in ppa:bzr-builder/daily within the hour15:17
adeuringnigelb: tests are running again15:19
nigelbadeuring: thanks, I'm branching db-devel :)15:20
adeuringnigelb: cool!15:20
gary_postersinzui or flacoste, do either of you have a quick answer to this question from https://answers.launchpad.net/launchpad/+question/170633 : If someone is subscribed to a blueprint, should they "receive *all* notifications, including the blueprint information changes like the milestone target" or a limited subset?15:27
sinzuigary_poster, that is the wrong question :(15:28
gary_posterheh, I don't understand15:28
flacostegary_poster: they only receive notifications about changes to the linked page15:28
sinzuigary_poster, blueprint subscriptions are neither15:28
flacosteif the linked wiki supports our "notification protocol"15:28
flacostewhich is basically sending emails on update to notifications@blueprints.launchpad.net15:29
flacosteor something like that15:29
flacostein other words, it probably only works for the Ubuntu wiki :-)15:29
salgadoallenap, around?  (you're one of the people asked to review https://code.launchpad.net/~xaav/loggerhead/export-tarball/+merge/66408 that's probably awake now, so I thought I'd ask you).  the Linaro team would benefit from having a way to get the contents of a branch as a tarball, via the API.  we'd be using it for tiny branches (less than a hundred KB), but people might end up trying to use that on large branches and that may be a pro15:29
salgadoblem.  so, my question is whether or not you think this could be accepted in LP15:29
flacostesalgado: i think ensemble wanted somthing similar15:30
flacostesalgado: there is a bug for that15:30
gary_posterflacoste, sinzui, got it.  Thank you15:30
sinzuigary_poster, The plans to create structural subscriptions never came to fruition, as you recall, they were moved to bug to be only about bugs15:30
gary_postersinzui, ah, right got it15:30
salgadoflacoste, the one I found was not about exposing it over the API, but maybe there is.  /me searches15:30
allenapsalgado: I'll have a look.15:30
salgadoflacoste, allenap, bug 515128 is the one I found15:31
_mup_Bug #515128: Launchpad should make tarballs of branches available for download <lp-code> <Launchpad itself:Triaged> < https://launchpad.net/bugs/515128 >15:31
gary_posterabentley, do you have any suggestions on how to field https://answers.launchpad.net/launchpad/+question/170639 ?15:31
gary_posterabentley, fwiw, I verified and what he describes does seem to be what's in the logs15:32
flacostesalgado: yes, that.s the one15:32
abentleygary_poster: we recommend using http with sourceforge as it seems a bit more reliable.15:33
gary_posterabentley, ah, I probably could have found that somewhere; that rings a bell.  sorry to bother & thanks15:33
abentleygary_poster: no problem.15:33
salgadothanks allenap!15:34
allenapsalgado: It looks like mbp and wallyworld_ are happy with it. wallyworld_, fancy following up on your threat in https://code.launchpad.net/~xaav/loggerhead/export-tarball/+merge/66408? That is, the last comment, where you said you'd land it :)15:37
jelmer_abentley: I thought https worked better actually15:38
salgadoallenap, do you think this could be made available on the API?15:38
allenapsalgado: I mean, I don't have any problem with it. If people start downloading huge branches it /should/ time out and we'll get an OOPS report about it. I think that's okay.15:38
allenapsalgado: Ah, okay, I missed that part of your request.15:39
salgadoallenap, would we really timeout? I thought we timed out only on long-running DB queries15:40
allenapsalgado: I'm not sure that's a good idea. Perhaps it could be streamed into a librarian file and a reference to that returned.15:40
abentleyjelmer_: I've heard the opposite, but I guess that could be wrong.15:40
salgadoallenap, that would have to happen asynchronously, then?15:42
allenapsalgado: It times out on requests too now, afaik. I'm not the right person to talk about that really; I don't know enough about where those time-outs are applied.15:42
flacostesalgado, allenap: only DB queries time outs15:43
salgadoallenap, oh, ok, I'll bug flacoste then.  thanks anyway15:43
flacostei think we had a few other place where we check the timeout15:43
flacostebut not through general python processing15:43
salgadoflacoste, do you remember if what ensemble wants is to get the branch content via the API as well?15:44
flacostesalgado: yes15:44
salgadofor formulas, I assume15:44
flacosteso that they don't have to use bzr15:44
flacosteto retrieve the formula15:44
flacostebut fetch it directly using the API15:44
allenapflacoste: Ah, okay, so if there's a non-db process taking a long time - like exporting a branch tarball - it would never trip the time-out machinery?15:44
=== al-maisan is now known as almaisan-away
flacosteallenap: correct15:44
flacosteallenap: or worse, it would trip it after too long15:45
salgadoflacoste, right. in our case we'd like to avoid bzr for private branches, so that we don't have to send private keys around15:45
flacosteso process for 2 minutes, then do a DB query (for session or whatever other housekeeping) and then timeout15:45
allenapflacoste: Are there any time-outs on the ha-proxy or Apache? I.e. if the response is not complete in X seconds it drops the connection?15:45
flacosteallenap: there is one on apache, but it's much higher15:46
flacostewhich is fine15:46
flacostesince then the client can decide to take its time15:46
flacostebut the expansive server thread as been freed up at that time15:46
flacosteso we don't care15:46
flacosteor much less15:46
allenapflacoste: Yeah, but none between Apache/ha-proxy and the app servers?15:46
flacosteallenap: i don't know off the top of my head, a losa would know, not that it changes anything15:47
flacostesince zope will happily continue processing15:47
flacostewe'll know the connetion was close only when we attempt to write to the connection15:47
salgadoflacoste, so, do you think we could make that available over the API now or would we need some way to prevent it from being used to download large branches?15:47
allenapflacoste: The loggerhead branch that exports tarballs expects to stream, I think, so a time-out on the app server connection would put a bound on that.15:48
flacostesalgado: we had tons of problem with streaming through the app server, we moved that to the librarian15:48
nigelbHi, can I have a patch ID for a db patch?15:48
flacosteallenap: in that case, yes15:48
flacosteis this a loggerhead or launchpad patch?15:48
salgadoflacoste, the existing one is loggerhead15:49
gary_posterabentley, any ideas on https://answers.launchpad.net/launchpad/+question/170658 ?15:49
salgadoflacoste, that may work for ensemble, in fact15:49
salgadobut not for us as we need to get the contents of private branches15:49
allenapflacoste: D'oh, that's not going to work for salgado. /me headdesks.15:50
flacostesalgado: why? loggerhead can be used with private branches15:50
abentleygary_poster: no, at first glance, I'd expect it to be linked.15:50
allenapflacoste: I think it's the proliferation of credentials that's a problem.15:51
flacostehow is it different than with API?15:51
salgadoflacoste, yeah, but then I have to give the SSO credentials to my client and make it do OpenID, no?15:51
salgadowith the API I can use OAuth15:51
gary_posterabentley, ok thanks.15:51
salgadowhich was better summarized by allenap. :)15:52
flacostesalgado: we could teach loggerhead to accept oauth auth15:52
salgadothat could work15:52
flacostesalgado: or a way to get an api script to get an 'auth token' that will work to make loggerhead reqeuest15:52
salgadoflacoste, loggerhead doesn't have access to LP's DB, right?  I was wondering if it'd be possible to use the LP OAuth credentials to authenticate on loggerhead15:53
salgadothat'd be the ideal way, I think15:54
flacostesalgado: right, it doesn't have DB access, but the auth glue code might have15:55
flacostethat's the launchpad_loggerhead thing15:55
salgadobut getting a separate loggerhead auth token via the LP API, although more cumbersome, would still be acceptable, I think15:55
flacostei'm not that familiar with it15:55
salgadook, I'll have a look at that15:55
salgadothanks flacoste and allenap!15:56
allenapsalgado: I think there's some support for general one-time tokens in Launchpad already, so half a solution might already be there.15:57
salgadooh, cool, that's good to know15:58
gary_posterflacoste, response from that blueprints notifications discussion:15:59
gary_poster"""Could you point me to the people behind the Ubuntu wiki changes (notification protocol support). That's probably something that we would like to have on Linaro's wiki (using moinmoin)."""15:59
gary_posterflacoste, should I ask them to file a bug, or is there something quic we can do for them?15:59
flacostegary_poster: https://help.launchpad.net/Blueprint#Subscribing_to_blueprints16:01
gary_posterflacoste, fantastic thanks16:01
flacostei think that page needs update as it's  likely the email is now  notifications@blueprints.launchpad.net16:02
flacostealthough there is still a MX record for specs.launchpad.net16:02
gary_posterflacoste, are you confident enough on that that I shuld change the help wiki?16:03
flacostegary_poster: i'm checking the code :-)16:03
gary_posterah, ok :-)16:03
=== beuno is now known as beuno-lunch
flacostegary_poster: no, it's still configured as specs.launchpad.net16:09
flacosteconfig value: launchpad/specs_domain16:09
gary_posterok cool thanks flacoste16:09
allenapsalgado: What's going to happen next? Are you going to file a bug, or will someone else request this formally? If you're expecting something from me, tell me :)16:09
gary_posterallenap, so unreasonable :-P16:10
salgadoallenap, oh, don't worry about that; I'll investigate and see if we're going to pursue this further or just pass SSH keys around16:11
allenapsalgado: Cool, awesome.16:11
gary_posterabentley, I changed the sourceforge import I mentioned to you before to http, and got a different error: http://launchpadlibrarian.net/79436540/zorba-coders-zorba-sf-trunk.log16:13
gary_posterdoes that mean http is not enabled on sourceforge for this branch, or something else?16:13
nigelbohai, I asked earlier but didn't get a response. Can someone give me a patch number for a db patch?16:18
gary_posternigelb, only stub can AFAIK.  Make a mp against db-devel and ask specifically for his review (Stuart Bishop)16:18
gary_posterat least...that16:19
gary_poster's how it used to work...16:19
nigelbhttps://dev.launchpad.net/PolicyAndProcess/DatabaseSchemaChangesProcess tells that anyone in ~launchad can do the patch ID thing16:19
nigelbI can't make an MP without having a file to make changes in.16:19
gary_posternigelb, ok, hadn't seen that.  I'll do it for you, gimme a sec16:22
gary_posternigelb, I need a comment to describe what the patch does.  Short, like "Add tables for a package symbol database" or "BugMessage.owner NOT NULL"16:25
nigelbgary_poster: "Deleting the statusexplanation column16:27
gary_posterack nigelb, thx16:27
gary_posternigelb, FWIW, the doc I changed had info that seemed like it was geared toward the patch writer.  It is here, if you'd like to take a glance at some advice: http://pastebin.ubuntu.com/685999/ .  Meanwhile, your patch number is 2208-84-116:31
gary_poster(I assumed that your patch would be fastdowntime-compatible, and not require a full old-style system outage; if it did, the last digit would be 0, and you would be asked lots of pointy questions :-) )16:31
=== adeuring changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: bac | Critical bugs: 253 - 0:[#######=]:256
jmlActually, it's more annoying than I thought, because the ellipsis can span multiple lines.16:31
jmlStill, you can get something useful.16:31
nigelbgary_poster: it is fast downtime compatible. I spoke to stub the other day about it16:32
nigelbgary_poster: Thanks btw :)16:32
gary_posterwelcome :-)16:33
=== matsubara is now known as matsubara-lunch
nigelbmtaylor: Congrats on the OpenStack Policy Board elections :)16:50
mtaylornigelb: thanks!16:56
abentleymtaylor: I've actually been hacking around with canonical's private openstack recently, trying to our test suite faster with parallelization.17:10
mtaylorabentley: awesome17:11
mtaylorabentley: how's that been going?17:11
abentleymtaylor: I've run some tests in parallel, but not gotten the whole suite to run yet.  And I just found out about cloud-init, which I'm hoping will make it easier to set up the instances.17:13
=== beuno-lunch is now known as beuno
mtaylorindeed. I haven't used cloud-init myself, but SpamapS tells me it's awesome17:13
abentleymtaylor: if it does what it says on the tin, it'll allow me to parameterize the worker nodes with the current master's IP address.17:16
mtaylorthat's what I hear17:19
nigelbHrm, I don't know how db-devel is supposed to work.17:22
nigelbmake schma asks me to run link-external-sorucecode17:22
nigelbWhen I run it, it says17:23
nigelblink-external-sourcecode: error: Parent branch not specified, and could not be discovered.17:23
abentleynigelb: you need to specify the branch you want to link the sourcecode from, typically your branch of stable or devel.17:23
=== matsubara-lunch is now known as matsubara
jmlSo basically doctest is terrible.17:29
abentleyjml: yes.  Yes, it is.17:37
nigelbkill it with fire? :)17:37
abentleyderyck: So my problem was a stale socket, "/var/run/launchpad_forking_service.sock".  Suggestions on how to deal with this better?17:40
jmlthoughts and suggestions welcome17:40
jmldisagreement will be dealt with according to its desserts.17:40
deryckabentley, the code hosting issue was a stale socket?17:46
abentleyderyck: yes.17:46
deryckabentley, from a crash, I assume, and not left behind from a normal ctrl-c?17:46
nigelbI need some postgres help :/17:46
nigelbtsvectorupdate BEFORE INSERT OR UPDATE ON bugtask FOR EACH ROW EXECUTE PROCEDURE ftiupdate('targetnamecache', 'b', 'statusexplanation', 'c')17:46
nigelbwhat does that do?17:47
nigelbOr rather, how do I found out what it does.17:47
abentleyderyck: i assume so.  Possibly a kill -917:47
cjwatsonCould somebody feed https://code.launchpad.net/~cjwatson/launchpad/multiarch-translations/+merge/74743 to EC2 again?  The doctest that failed last time passes now.17:48
deryckabentley, if it's a kill -9, not sure.  otherwise, so some sort of exit handler to clean up itself?17:49
deryckcjwatson, I'll send it off now for you.17:49
abentleyderyck: I'd like it to either fail noisily and clearly, or remove the stale socket and succeed.17:50
jmlcjwatson: fwiw, I just spent a chunk of time trying to make those errors more decipherable. It's harder than I thought.17:54
abentleyderyck: or perhaps use a random socket.17:56
deryckthat could work17:56
nigelbhrm, when I do \d bugtask in psql, I get the table as public.bugtask18:21
nigelbDoes this mean in my db patch I should do alter table public.bugtask and not just bugtask?18:22
flacostenigelb: nah18:43
flacostepublic is the default namespace18:43
flacostein pgsql18:43
nigelbI need to spend a weekend reading up postgres.18:43
nigelbI wish there were more hours in a day.18:43
nigelbflacoste: Is something wrong with Launchpad emails/notifications?18:46
* nigelb sees delays18:46
flacostenigelb: not that i know of, what makes you say this?18:46
nigelbMaybe its PEBKAC, checking :)18:47
nigelbflacoste: Yep, PEBKAC :D18:53
flacosteglad that's the case :-)18:53
nigelbHrm, really need to be able to batch team additions and deletions.18:54
nigelbOh wait. No. I'll end up doing it.18:54
* deryck is switching work locations, back online in 30 minutes18:58
nigelbI broke something.18:59
nigelbmake newsampledata failed.19:00
sinzuijcsackett, do you have time to mumble?19:34
nigelb\o/ Test bassed.19:44
=== matsubara is now known as matsubara-afk
abentleybac: could you please review https://code.launchpad.net/~abentley/launchpad/simplify-twisted-runner-3/+merge/74868 ?20:04
bacabentley: sure.  let me wrap up CHR and then i'll get to it.20:05
abentleybac: thanks.20:05
=== almaisan-away is now known as al-maisan
bacabentley: done.  thanks.20:08
=== bac changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: - | Critical bugs: 253 - 0:[#######=]:256
nigelbHrm, are we in testfix?20:19
jcsackettsinzui: I'm actually on the road; had a half day today. If you want to ring my cell tho I can talk.20:23
sinzuijcsackett, sorry. I forgot20:23
sinzuihave a nice day20:24
jcsackettNo worries, and thanks. Have a good weekend.20:25
=== al-maisan is now known as almaisan-away
wgrantabentley: That TwistedJobRunner branch solves the massive NoneType response tracebacks we get sometimes?23:38
nigelbwgrant: ohai23:50
wgrantnigelb: Hi.23:51
nigelbwgrant: I have a doubt for a db patch23:51
nigelbI ran into trouble with the fti trigger23:52
wgrantAh, that searches statusexplanation?23:52
nigelbtsvectorupdate BEFORE INSERT OR UPDATE ON bugtask FOR EACH ROW EXECUTE PROCEDURE ftiupdate('targetnamecache', 'b', 'statusexplanation', 'c')23:52
nigelb^^ that one23:52
wgrantSo, fti is the thing most in limbo right now.23:53
wgrantThe script that maintains it (database/schema/fti.py) is not run any more.23:53
wgrantBecause it has to be done during downtime, but can take too long.23:53
wgrantSo you should talk to stub about this.23:53
nigelbokay :)23:53
wgrantWe may have to fix fti.py to just maintain the triggers, without actually rebuilding the data.23:53
nigelbTHe "psql launchpad_ftest_playground -f your-patch.sql" step ran into trouble because of this23:54
nigelbso, not sure how far I'll get23:54
nigelbBut looks like I need to catch stub on Monday.23:54
nigelbso this branches goes onto hold mode :)23:54
nigelbwgrant: Also. Are we in testfix?23:55
wgrantnigelb: We are.23:55
wgrantBut I just got the buildbot slaves cleaned up, and forced the builds.23:56
nigelbCool. That explains that.23:56
wgrantYou've got something that needs landing?23:56
nigelbIts finished tests23:56
nigelbso, I guess your buildbot cleaning should fix it.23:56
wgrantIt won't automatically reland.23:57
wgrantCould you forward me the ec2 mail?23:57

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