/srv/irclogs.ubuntu.com/2011/06/21/#launchpad-dev.txt

sinzuiwgrant, stevenK mumble>00:03
wgrantStill no builders :/00:17
LPCIBotProject windmill-devel build #253: STILL FAILING in 1 hr 7 min: https://lpci.wedontsleep.org/job/windmill-devel/253/00:20
jcsackettsinzui, wgrant, stevenK, wallyworld: i have to run. have a good one!00:32
lifelessis getUtility(IStoreSelector).get(MAIN_STORE, DEFAULT_FLAVOR) totally equivalent to IMasterStore(class) ?00:42
wgrantlifeless: No.00:42
wgrantMASTER_FLAVOR00:42
lifelessso when should we use IMasterStore?00:42
StevenKI'd like to kill getUtility(IStoreSelector)00:42
lifelessI'd like to kill all of it00:43
wgrantlifeless: IStoreSelector and I(Master|Slave)?Store are alternatives.00:43
lifelesswe have had performance bugs talking to both stores00:43
lifelesswgrant: sure; but when should we use which one?00:43
LPCIBotProject devel build #820: STILL FAILING in 5 hr 33 min: https://lpci.wedontsleep.org/job/devel/820/00:44
wgrantlifeless: There is no difference.00:44
wgrantlifeless: Assuming you request the same flavour.00:44
lifelessIs thre an IDefaultStore then ?00:45
StevenKNo, IStore00:45
LPCIBotProject parallel-test build #52: STILL FAILING in 1 hr 16 min: https://lpci.wedontsleep.org/job/parallel-test/52/00:45
wgrantRight, DEFAULT_FLAVOR is IStore00:45
lifelessright00:45
lifelessso put another way, we probably need to do IStore everywhere and nuke the slave/master use outside of the policy setup00:46
wgrantlifeless: Possibly.00:47
lifelesswhy wouldn't we?00:47
wgrantlifeless: Because in the past we have sometimes explicitly executed expensive queries on the slave.00:48
lifelessin terms of db bloat, long transactions (> a few seconds) are bad on any replica00:49
wgrantLoad, not length.00:49
lifelessyes, I know thats what you wrote00:50
lifelessI'm picking it apart00:50
lifelessin terms of memory footprint, queries that are large will spill to disk00:50
lifelessin terms of cpu overhead, we've got more headroom on the master than on our slaves.00:51
lifelessif its a script, I want them talking to appservers anyway.00:51
lifelessI could see a case for parallel execution of complex queries, but our current stack isn't the right place for that anyhow00:52
wgrantStevenK: http://paste.ubuntu.com/630105/00:59
LPCIBotProject windmill-db-devel build #407: STILL FAILING in 1 hr 5 min: https://lpci.wedontsleep.org/job/windmill-db-devel/407/01:25
StevenKwgrant: Mind having another look at https://code.launchpad.net/~stevenk/launchpad/db-add-bprc/+merge/64783 ?02:13
wallyworldsinzui: ping02:19
wgrantStevenK: Reviewed.02:27
StevenKwgrant: You mean the long description of IBPP?02:29
wgrantIts docstring.02:29
StevenKYes, as in the 3 line bit of the docstring?02:29
wgrantYes.02:30
* StevenK fixes that by removing it.02:31
StevenKwgrant: with dest_file as tempfile.NamedTemporaryFile(): ?02:38
StevenKSigh, my knowledge of with sucks02:43
wgrantStevenK: Other way around, but yeah.02:55
lifeless-> buy a webcam for dublin03:44
jtvmorning friends04:00
wgrantlifeless: qas is more than four months old. Shall we restore?06:09
wallyworldwgrant: have you tried to ec2 anything lately? i'm getting loads of what appear to be unrelated (to my changes) errors06:23
wgrantwallyworld: Not for a while. What's up?06:24
wallyworldwgrant: lots of stuff, from ImportError: cannot import name PackageBuild in soyez/model/archive.py to bug subscription errors, and much more06:25
wgrantwallyworld: This isn't your superconflictfun branch, is it?06:26
wallyworldwgrant: no, i shelved the latter pipes and landed something based on earlier work06:26
wallyworldthe diff in the mp looks as i expect06:27
wgrantwallyworld: What if you merge devel? No crisscrosses?06:27
wallyworldnot last time i tried which was yerterday. i'll try again. i just wanted to check first to see if it was "just me"06:27
wgrantbuildbot is not crying. So probably just you.06:28
wallyworld:-(06:28
wgrantThe subscription stuff crisscrossed and had to be reverted when it was landed.06:28
wgrantI'd merge devel into your branch and then check the rancestor diff.06:29
wallyworlddoesn't really explain the PackageBuild import error and some others06:29
wallyworldsince none of my changes went near there06:29
wgrantExactly.06:29
wgrantCrisscross.06:29
wgrantParticularly if you merged any of the Bugs branches at any stage.06:30
wgrantOr anything even slightly related to them.06:30
wallyworldi've only merged devel06:30
wgrantEver?06:30
wallyworldso far as i can recall06:31
wallyworldjust merged in devel, no aparent problems06:31
wallyworldbut then again, there weren't that many new changes from yesterday06:32
wgrantwallyworld, StevenK: Do you see any reason to not restrict the email address search to (email = ? OR email LIKE ? || '@%%')?06:59
wgrantI doubt searching for partial email addresses is common, except for just specifying a local part.06:59
wgrant(and we get heaps of false prefix matches :/)06:59
wallyworldwgrant: i think that's a goof idea to implement07:03
wallyworldwgrant: bug 79884707:03
_mup_Bug #798847: Person picker search for partial IRC nickname doesn't return intended result <disclosure> <exploratory-testing> <person-picker> <Launchpad itself:Triaged> < https://launchpad.net/bugs/798847 >07:03
wallyworldwgrant: do we really want to search on partial irc nicks07:03
wgrantwallyworld: I saw that and considered won'tfixing it.07:03
wgrantIt's a barrel of worms.07:03
wallyworldi agree07:03
wgrantCan of worms.07:03
wgrantWhatever the idiom is.07:04
wallyworldbarrel of monkeys too07:04
wallyworldespecially since the example was 'jools' and the user was searching for 'bigjools'07:04
wallyworldnormally one would enter a prefix, not the last characters in a name07:05
wgrantThat really requires tokenising IRC nicks further.07:05
wgrantAnd uh, no.07:05
wallyworldyou mean you search for partial suffixes?07:05
lifelesswgrant: sure07:06
wgrantwallyworld: No. But to implement it would either mean matching any substring, or we'd have to guess at tokenising it without whitespace.07:07
wgrantSo... no.07:08
wgrantKill it.07:08
wallyworldwgrant: yes, that's why i said i agree we shouldn't do it :-)07:08
* wgrant won'tfixes.07:08
wgrantOr perhaps I should wait for sinzui.07:09
wallyworldwgrant: i was going to ask him about that and also the picker batch widget bug but he had already EOD07:11
wallyworldi should say the pagination control bug 79877207:11
_mup_Bug #798772: Pagination in person picker should appear after the results, not before <disclosure> <exploratory-testing> <person-picker> <Launchpad itself:Triaged> < https://launchpad.net/bugs/798772 >07:11
wgrantwallyworld: We probably want to completely rework that.07:12
wgrantwallyworld: Remove the page numbers and stuff.07:12
wallyworldthat is done in lazr-js so are we saying we want to change the paginator location for *all* pickers or just personpicker07:12
wgrantwallyworld: Since it makes lifeless war on batchnav very hard.07:12
wgrantSo if we're going to move it to the bottom and break from lazr-js, we should really change it.07:12
wallyworldwell, one step at a time perhaps07:13
wallyworlddon't want too much scope creep07:13
wallyworldi'm wondering how that bug got tagged disclosure anyways07:13
wgrantWhich?07:14
wallyworldthe paginator one07:14
wgrantIt was discovered during exploratory testing of the new shiny.07:14
wallyworldabove ^^^07:14
wgrantSo it's our problem unless we decide it isn't.07:14
wallyworldsure, but it's been there forever and is not related to our work07:14
wallyworldat all07:14
wgrantOur work was to make the pickers not suck.07:14
wgrantSearch was part of that.07:15
wgrantDisplaying more relevant info was another.07:15
wallyworldhmmm. not sure how having the paginator at the top makes the pickers suck07:15
wallyworldi like it there07:15
wallyworldless mouse movement07:15
wgrantIndeed.07:15
wgrantBut some disagree.07:15
wallyworldwell then they are wrong :-P07:16
wgrantIndeed!07:16
lifelessFWIW, paginator can be above or below IMO; page #'s however... poor idea07:16
lifelesswhich number will your result be on?07:16
wgrantlifeless: Poor and pointless.07:16
lifelessits like a lottery07:16
wgrantIf you07:16
wgrantIf you have to click past the first page for a given person more than once, we have failed.07:17
wallyworldthe only thing i see them adding any value for is to indicate how many results there are, but that can be done other ways07:17
StevenKWOW.07:17
StevenKWhen did we hit bug 800000?07:17
wgrantwallyworld: But it's limited now, so it's only useful for very small numbers.07:17
* StevenK prods _mup_ 07:17
wgrantHuh, so we did.07:17
wgrantJust noticed I filed bug #80004307:17
_mup_Bug #800043: IRC nickname search in person picker is case-sensitive <disclosure> <person-picker> <Launchpad itself:In Progress by wgrant> < https://launchpad.net/bugs/800043 >07:17
wgrant800000 must be private.07:17
wgrantYeah.07:17
lifelesswallyworld: showing the result set size is generally uninteresting07:18
wallyworldshould we have a 1000000 party next month?07:18
wgrantNot even I can see it.07:18
lifelesswallyworld: there are a very few specific cases when its anything other than geek chic07:18
StevenKbug 79999907:18
wallyworldlifeless: i think it adds value but that's just me07:18
lifelesswallyworld: consider the phone book; do you care how many booths there are in it ?07:18
wallyworldyes!07:18
lifelesswallyworld: and how many freds ?07:18
wgrantIt gives a hint as to the selectivity of your search.07:18
wgrantWhether you're likely to find the person in the next few pages.07:19
lifelessyes, the magnitude matters07:19
wallyworldwhen searching, if there are too many results, it gives you a cue to be more specific, especially if you want to browse the results07:19
lifelessfew/10s/100s/many07:19
wallyworldwgrant: +1. what i was trying to say07:19
lifelessthe precise count doesn't matter most of the time.07:19
lifelessand magnitude estimation isn't one of those times.07:20
wallyworldagree with that too07:20
wallyworldwe should do it as a slider :-)07:20
wallyworldas you move  the slider, the results pan past07:20
wallyworldthat would be way cool imnsho07:21
lifelessvertical slider would be nice07:21
wallyworldyes07:21
lifelesseven just vertical next/prev markers07:21
lifelessrather than horizontal for things that are laid out vertically.07:21
lifelessthat always gets me.07:21
wallyworldand also doesn't really suit right-left locales either07:22
lifelessquestion07:24
lifelessare anonymous users None, or something else?07:24
lifelessin terms of LaunchpadView's self.user07:25
wallyworldi guess None but am not 100% sure07:25
wallyworldi recall some test code doing a self.user == None check07:25
wgrantNone07:26
wgrantBah.08:03
wgrantThe email address prefix is useful :(08:03
wgrantFor eg. "jon" won't catch jcsackett without it.08:03
wgrantBecause his displayname is crazy.08:04
=== jamesh_ is now known as jamesh
rvbawgrant: Hi! I'm sorry to bother you again with this but could you have a look at the multi parent branch? (https://code.launchpad.net/~rvb/launchpad/init-series-bug-789091-devel/+merge/63676)08:33
wgrantrvba: And what about sources?08:36
wgrantAnd are you regretting trying to do this yet? :P08:36
rvbaWell, not yet :)08:37
wgrantAh, good, Soyuz has got your soul already.08:37
rvbawgrant: So I suppose the same trick should be used for the sources then ...08:38
wgrantI guess so :/08:38
wgrantAlso this is probably going to be very slow.08:39
wgrantBut maybe not.08:39
rvbaI'll estimate the overhead of this.08:39
lifelesswgrant: jcsackett working is a bit of a special case08:44
wgrantlifeless: It is, but it helps for some other cases too.08:44
wgrantBecause name stemming is hard.08:44
lifelesswgrant: folk that don't make their metadata match the identity they tell their peers are going to be pretty hard to find08:44
wgrantlifeless: Thanks.08:53
adeuringgood morning08:56
rvbawgrant: Fix pushed (tests added: test_binaryfile_already_in_archive/test_sourcefile_already_in_archive).09:18
lifelessgrar09:32
lifelessserial conflicts09:32
lifeless*hates* on 4 hour test runs09:32
StevenKThen fix it, dear Eliza?09:34
lifelessStevenK: its in the pipeline09:35
* StevenK is only trolling.09:35
lifelessI know09:36
=== gmb changed the topic of #launchpad-dev to: Performance Tuesday | https://dev.launchpad.net/ | On call reviewer: gmb | Critical bugs: 205 - 0:[######=_]:256
maxbMorning all. How is the deployment report looking today?09:55
jmlgood morning Launchpadders10:21
jmlI was getting productive email stuff done and thought, "why am I so productive"10:21
jmland then I realized my IRC client was off10:21
wgrantmaxb: Somewhat dire.10:27
wgrantStill blocking on lots of reasonably nasty jtv QA in the middle of the 12000 line branch.10:28
wgrantAnd Red squad seems to not be here today.10:28
wgrant(your rev is after the 12000 line branch, its reversion, and its relanding, but there is another QA fix after that, and before that is jtv's stuff)10:29
rvbawgrant: Julian is ill. Jtv has trouble with irc. The rest of us is here ;)10:40
wgrantrvba: Do you know if he's working on his QA?10:41
rvbawgrant: jtv?10:42
wgrantrvba: Yes.10:42
rvbawgrant: I don't know ... He can be reached by email.10:42
rvbawgrant: I've fixed the problem we talked about earlier this morning (https://code.launchpad.net/~rvb/launchpad/init-series-bug-789091-devel/+merge/63676)10:43
wgrantrvba: I think we need to chat with Julian about this.10:44
rvbawgrant: ok.10:44
wgrantHmmm.10:45
wgrantAll domestic flights in south-eastern .au cancelled tomorrow. And all internationals delayed probably until Friday.10:45
wgrantThis does not bode well.10:46
StevenKIndeed10:49
jmlwgrant: Chile volcano?10:49
huwshimijml: yeah10:49
jmlhuwshimi: you managed to escape though.10:50
huwshimijml: There have been delays for the last week or so10:50
nigelbWait, a volcano in Chile is affecting .au? o.O10:50
huwshimijml: Yeah, just10:50
wgrantnigelb: The ash is everywhere.10:50
wgrantWell.10:50
wgrantMostly above us now.10:50
lifelessnigelb: ash going west10:50
wgrantBut still everywhere.10:50
nigelbAhh. I forgot about the earth being round temporarirly.10:50
jmlnigelb: yeah. all that hippy crap about us being interconnected beings on one planet turns out to be true10:51
nigelbjml: lol10:51
nigelbwgrant: Are you going to miss the sprint or rally or whatever its calle dnow.10:51
nigelb*called now10:51
wgrantnigelb: Hopefully not!10:51
nigelb:)10:52
wgrantI think it's still a Thunderdome.10:52
wgrantBut I'm not sure.10:52
wgrantMaybe just an Epic.10:52
wgrantWhich is colocated with the Platform Rally.10:52
nigelbFun!10:52
nigelbjml: On reading the blog post about your depature, I first feared you were leaving Canonical :)10:53
jmlnigelb: nope10:54
jmlnigelb: very glad to be staying10:54
nigelbjml: :)10:55
cjwatsonLP's loss is our gain :-)10:55
nigelbheh10:55
lifelessjelmer: hi11:00
lifelessjelmer: did you redefine the meaning of persisted enums in your code import patch?11:01
jelmerlifeless: hi11:03
jelmerlifeless: These were prematurely added constants according to mwhudson, never used in real life.11:04
lifelessk11:05
lifelessargh, the deploy queue just got riskier still11:22
lifelessjelmer: how would you feel about rolling back that code import autoupgrade thing?11:22
jmljust came across https://dev.launchpad.net/LaunchpadHackingFAQ11:23
jmlthere's a blast from the past.11:23
lifelessjelmer: we've a few high risk things already in the queue, and with yours added we may be unable to deploy for days11:23
lifelessjelmer: if you're -really- confident in it, its fine.11:25
lifelessjelmer: I'm also curious why you didn't just fix the scanner to handle the new backup directory scheme bzr uses ( I think there is a code hosting bug on that)11:26
lifelesss/just//11:26
wgrantlifeless: s/days/even more days/11:26
wgrantBut I think jelmer's fix looks good, FWIW.11:27
jml*sigh*11:28
jmlhttps://dev.launchpad.net/Hacking is a fork of LaunchpadHackingFAQ11:29
* jml sets about merging the two and deleting the latter11:35
wgrantcjwatson: Nothing of note this week that will prevent a couple of minutes' downtime for cocoplum services, while we update it for your germinate change?11:41
lifelessallenap: how is rabbit fixture looking ?11:46
allenaplifeless: I landed it on Saturday :) No failure so far (that I'm aware of) \o/11:47
lifeless\o/11:47
lifelessallenap: did you file a bug upstream?11:47
cjwatsonwgrant: nope, please go ahead11:47
allenapallenap: No. I wonder if I should file it on Erlang or RabbitMQ.11:48
lifelessallenap: rabbit for starters11:48
lifelessthey offer programs after all :)11:48
allenaplifeless: Okay.11:49
lifelessallenap: also, do you want to land my layer patch now the substrate is solid ?11:49
allenaplifeless: Yeah, good idea.11:49
jmldo we have a standard wiki macro for "obsolete page"?11:53
nigelbjml: remember jorge's lightning talk? ;)11:54
nigelbDelete! Delete! Delete!11:54
jmlnigelb: yeah, that was the Ubuntu wiki11:55
jmland I have deleted some stuff already11:55
nigelbI didn't know there was help on tal stuff till now.11:55
nigelbI went and read zope documentation for that and didn't understand much.11:55
StevenKTal is quite ... opaque.12:00
nigelbStevenK: Interesting choice of words.12:01
=== benji_ is now known as benji
jelmerlifeless, sorry, I missed that12:30
jelmerlifeless, bzr's scheme hasn't changed, but it has one method to backup an existing and one for renaming a directory out of the way12:30
jelmerthey have different naming schemees. the second is only used by "bzr join" at the moment.12:31
lifelessjelmer: so how did we end up with the latter scheme named directories during the incident?12:52
jelmerlifeless: I called that method manually12:57
jelmerI don't think bzr should have two name schemes for backed up .bzr directories12:58
lifelessyou're going to change join ?13:06
jelmerlifeless: I filed a bug about it13:11
deryckHi, abentley and adeuring14:24
abentleyderyck: hi.14:25
deryckabentley, adeuring -- so it's very noisy in this place.  Can we do an IRC standup?14:25
abentleyderyck: sure.14:26
derycklet's wait on adeuring to ack before we start.14:26
abentleyderyck: we could make a temporary channel, or use #launchpad-meeting, if you like.14:27
adeuringderyck: abentley: I'm here14:27
deryckabentley, sure, let's move to #launchpad-meeting to avoid too much noise here.14:28
deryckadeuring, hi.  changing channels ^^14:28
jmljelmer: you'll be in Dublin, right?14:33
jelmerjml:Yep14:33
adeuringderyck: could you please run this SQL statement on staging: http://paste.ubuntu.com/630399/ ?14:40
deryckadeuring, sure.... grabbing it now14:41
abentleyadeuring: why "select 1"?  Doesn't that just return "1" always?14:44
danilosmatsubara, hi, is there a way to easily search for any OOPSes beginning with "IntegrityError: duplicate key value violates unique constraint \"tm__potmsgset" across all of our systems?14:44
adeuringabentley: I have in idea yet. I guess that the idea is to see if the is any record matching the WEHER clause14:45
matsubaradanilos, I can query the oops db using SQL. there's no UI to do such search in oops-tools14:45
adeurings/in idea/no idea/14:46
adeuringabentley:  I even don't know yet where the query comes from in Python ;)14:46
abentleyadeuring: I think SELECT 1 will always return 1.14:47
danilosmatsubara, can you please do that for the above exception text? after "tm__potmsgset" there could be different things14:47
adeuringabentley: except if there are no result rows14:47
danilosmatsubara, if LIKE "blah%" is too slow, you can look for actual constraints from https://bugs.launchpad.net/launchpad/+bug/603530/comments/514:49
_mup_Bug #603530: Still violating tm__potmsgset__potemplate__language__no_variant__diverged__impo <lp-translations> <oops> <Launchpad itself:Triaged> < https://launchpad.net/bugs/603530 >14:49
matsubaradanilos, thanks. LIKE syntax is not returning me any results14:50
abentleyadeuring: Ah, I see.14:50
deryckadeuring, here you are.  with two runs, to warm the cache.  https://pastebin.canonical.com/48802/14:51
adeuringderyck: thanks!14:52
derycknp14:52
adeuringabentley: ^^^ So, now I'll stare for a while at the result (practically, I'll print it out so I can make notes with a pencil)14:53
danilosmatsubara, cool, I think we can call that bug closed, so we just solved one critical bug :) yaay ;)14:53
matsubaradanilos, not so fast14:53
matsubaradanilos, https://devpad.canonical.com/~matsubara/danilo-integrity-error.txt14:54
danilosmatsubara, oh, :(14:54
danilosmatsubara, these are all still from last year, before this code was rolled out14:54
danilosok, now there's a few new ones14:55
matsubaradanilos, reload. the file was incomplete. it should show you 972 rows14:55
danilosmatsubara, yeah, the final ones seem to be from March 1st14:57
matsubaradanilos, hmm perhaps I should order that by date to make things easier14:58
danilosmatsubara, yeah14:58
danilosmatsubara, that'd be helpful :)14:58
matsubaradanilos, ok, refresh and it'll be ordered14:59
danilosmatsubara, thanks; considering they were happening every day multiple times until March 1st, I'll still call the bug fix released if it hasn't ever happened since15:03
matsubaradanilos, sounds good15:03
danilosmatsubara, also, I can't get to any of those OOPSes myself15:03
danilosmatsubara, i.e. does https://lp-oops.canonical.com/oops.py/?oopsid=1886M2307 work for you?15:04
matsubaradanilos, yeah, those are not referenced in the LP db, so the oops purge deleted them15:04
danilosmatsubara, ah, ok, thanks15:04
sinzuideryck, can you run GTK_DEBUG=ALL ./bin/test -vv --layer=YUI 2>1 >  _test.log and send/pastebin the log for me15:06
derycksinzui, definitely.  Thanks for continuing to work on this!15:07
deryckwas going to ping you too :)15:07
adeuringabentley: There is a line "sort method: external merge" in the "explain analyze" output: That for the "UNION" statement, I assume. One of the two subqueries needs 13 seconds, and the other a few milliseconds. So, the first thing that could be changed is: Run the queries separately. that would save 7 seconds. Not enough, but it could be a start. Next thing I'll do is to figure out where the query is issued in Python.15:09
derycksinzui, unfortunately, not much to see:  http://pastebin.ubuntu.com/630410/15:14
sinzuioh, I get huge amounts for data15:14
sinzuimaybe my stderr redirect is mangled15:15
abentleyadeuring: if it's just running an existence check, I wonder whether adding LIMIT 1 to the subselects would help.15:15
adeuringabentley: right, that could help. Lets' try it!15:15
derycksinzui, I did 2>&1 > file, which I thought was the correct form.  but same output.15:17
derycksinzui, maybe I need debug version of a package?15:17
adeuringderyck: could you please try another query: http://paste.ubuntu.com/630412/ ?15:17
deryckadeuring, sure.15:17
sinzuideryck, I fear that I can see the needed data because I am using development vrsions of gtk libvs15:18
sinzuideryck, I think your libs are compiled with debug off15:18
derycksinzui, yeah.15:18
derycksinzui, I don't mind compiling if you want, but I fear that might also fix the issue :)15:18
sinzuiI get 2000 lines of output15:18
huwshimideryck: Do you want to reschedule our call today? Sounds like things are a bit crazy where you are.15:21
deryckhuwshimi, it's still top of next hour, right?15:21
huwshimideryck: Yeah15:22
deryckhuwshimi, I'll be fine for that.15:22
huwshimideryck: OK sure, as long as it's easy enough for you15:22
deryckhuwshimi, oh yeah, np.  I'll tell everyone else to go away in 40 minutes. :)15:22
sinzuideryck, you would need libgtk2*-dbg and libwebkitgtk-3*-dbg and I do not know if they replace your current libs15:23
huwshimideryck: haha15:23
derycksinzui, ok, let me apt-get install15:23
deryckadeuring, https://pastebin.canonical.com/48809/15:25
adeuringderyck: thanks!15:26
derycknp!15:26
derycksinzui, I need to switch back to my main office.  be back online in 10.  and I'll finish this install and try again then.15:27
adeuringabentley: ^^^ so, some progress, but the sorting for the first subquery needs 4 seconds, even in the second run. I'll look now where the query is issued and will try to understand why it is executed.15:29
derycksinzui, I'm back working on this.  dbg package for libwebkitgtk is taking forever to download.15:41
sinzui:(15:43
sinzuiI wish one of my computers would show the segfault15:44
sinzuithough my daughters are happy I stopped taking their computers to test this15:44
derycksinzui, so nothing new in the log even after the dbg packages15:46
sinzui:(15:46
nigelbjcastro: woah, good one!15:50
nigelbargh15:50
derycksinzui, can you paste the command you're using to get the log once more?15:56
sinzuiNew yucky attempt: strace -Ff -tt ./bin/test -vv --layer=YIU 2>&1 | tee _strace.log15:57
derycknow there's some noise :)15:58
sinzuideryck, old with noise: GTK_DEBUG=ALL ./bin/test -vv --layer=YUI 2>&1 | tee  _test.log15:58
adeuringabentley: The "bad query" is executed during Snapshot(ubuntu) in lp.app.browser.launchpadform.LaunchpadEditFormView.updateContextFromData(). So, a "regular property", don't know yet which one it is.15:59
sinzuiI am familiar with the gtk output. strace will challenge my archeology skills.15:59
huwshimideryck: Ready when you are. How do you want to do this?16:00
deryckhuwshimi, 3 minutes and I'll meet you in mumble, cool?16:00
sinzuiokay16:01
huwshimideryck: Yeah that's fine16:01
derycksinzui, heh, the paste is straining my browser to return control to me. :)16:02
* deryck tries chromium16:02
derycksinzui, I can't get a pastebin to load it, so I'm pushing it to my home on devpad as yui-strace.log.  15% now. :)16:05
sinzuiokay16:05
huwshimideryck: Do you want to just let me know when you're done. There's no hurry16:05
deryckhuwshimi, joining mumble now16:06
huwshimideryck: Sure16:06
deryckhuwshimi, meet me in Orange 1 o 1.16:06
huwshimideryck: no problems16:06
adeuringabentley: tracing through Snapshot.__init__() with pdb and watching the postgersql statement log, it seems that Distribution.has_published_sources issues the bad query.16:10
gmbadeuring: Ping for when you've got a second.16:14
adeuringgmb: yes?16:14
gmbadeuring: There's a question on answers.lp.n about deleting HWDB submissions; I have no idea about it. Could you take a look if you ahve the time: https://answers.launchpad.net/launchpad/+question/16197216:14
gmb?16:14
adeuringgmb: sure16:15
gmbThanks16:15
flacostedanilos: what's happening with the qa of bug 772754?16:16
_mup_Bug #772754: After better-bug-notification changes, list of bug subscribers is confusing <qa-needstesting> <story-better-bug-notification> <Launchpad itself:Fix Committed by gary> < https://launchpad.net/bugs/772754 >16:16
flacostervba, allenap: can one of you assess if we can deploy revision 13259, please?16:17
danilosflacoste, same as yesterday, it's qa-ok if r13258 is included in the deployment, but revisions in between are still in qa-needstesting; I've emailed jtv asking him to QA his 3 non-trivial revisions16:17
gary_posterflacoste, danilos, jtv replies saying that bigjools needed to do the qa16:17
flacostedanilos: the only needstesting that i see before 13258 are yours actually16:18
gary_posterbut he is out16:18
rvbabigjools is not here today :(16:18
danilosflacoste, maybe they've done it in the last 30 mins or so16:18
flacostervba: i know, that's why i'm asking you :-)16:18
flacosteor allenap16:18
* rvba looking16:18
flacostethat's related to what you guys have been working on16:18
flacostedanilos: what do we need to do to make the report happy, for you revision, do you know?16:19
danilosflacoste, they are all for 772754, and I've marked it as qa-ok now that it's all clear up to 1325816:19
danilosflacoste, wait for it to update now :)16:19
flacostedanilos: awesome!16:19
flacostethanks16:19
danilosyw16:19
timrcjcsackett, hello, you there?16:20
rvbaflacoste: just to make sure, you're talking about http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/13259 ?16:23
flacostervba: yes16:24
rvbaLooks pretty safe to me, it's a simple change in a message displayed on the difference page.16:25
flacostejcsackett: i think you made a mistake in your commit message16:25
rvbaIf DF is up to date I can even qa it.16:25
* rvba checks.16:25
flacostejcsackett: revision 13251 looks like it's still blocked, but I think your 13260 commit actually reverts it16:26
flacostejcsackett: but it was tagged rollback-1352116:26
flacostejcsackett: if that's the case, maybe changing the tag will make the qatagger script happy16:26
flacostedanilos: i think deployment is still blocked on jcsackett and rvba as we cannot deploy without the rollback of revision 1325116:33
rvbaflacoste: I marked bug 798222 (fixed by 13259) as qa-ok because the fix is a one-liner fixing a simple 'display' bug.16:33
_mup_Bug #798222: +localpackagediffs should explicitly say "no diff available" <derivation> <qa-ok> <Launchpad itself:Fix Committed by julian-edwards> < https://launchpad.net/bugs/798222 >16:33
rvbaflacoste: is that ok with you?16:34
flacostervba: fine withe me, thanks16:34
pooliegmb: can you pilot barry's patch to bug 797281?16:37
_mup_Bug #797281: LP API broken in oneiric with python-httplib2 0.7.0-1 <oneiric> <lazr.restfulclient:In Progress> <python-httplib2 (Ubuntu):Fix Released by barry> <python-httplib2 (Ubuntu Oneiric):Fix Released by barry> < https://launchpad.net/bugs/797281 >16:37
gmbpoolie: Sure thing.16:37
poolieta16:40
LPCIBotProject windmill-devel build #258: STILL FAILING in 41 min: https://lpci.wedontsleep.org/job/windmill-devel/258/16:48
jcsackettflacoste: sorry, didn't get msg updated when you pinged me.16:55
flacostepoolie, gmb: i'm actually on it16:55
jcsackett13260 is indeed a rollback of 13251.16:55
adeuringderyck: could you try another query: http://paste.ubuntu.com/630443/ (probably not very much progress, but anyway...)16:56
deryckadeuring, sure.16:56
derycksinzui, my strace is up on devpad now, btw.16:56
gmbflacoste: Oh. Bugger, I've just merged that and pushed it since it was r=.16:56
flacostegmb: fine by me :-)16:57
flacostegmb: technically, we should run the tests manually16:57
flacostesince there is no automated PQM test run there16:57
flacostejust to make sure16:57
gmbflacoste: yes, that thought occurred to me after I'd pushed it. I'll do that now.16:57
flacostebut i'm sure barry did it16:57
flacoste:-)16:57
sinzuideryck, thank. I pulled it a while ago16:57
derycksinzui, ok, cool, thanks for the help!16:58
deryckadeuring, looks fast to me.  :)  https://pastebin.canonical.com/48812/16:58
jcsackettflacoste: i'm not sure what the tag issue is; looking at our qaprocess stuff in the wiki it's tagged appropriately. a bad commit tag for the bad revision, and all qa tags removed. thoughts on what it should be?16:59
flacostejcsackett: the message was [rollback=13521]16:59
adeuringderyck: uh, yes, thanks! I assumed that we would have an ORDER BY which would slow down the query, but this is good eoungh, I think17:00
deryckadeuring, excellent.17:00
adeuringderyck: now I just need to duble-check that it was the right query :)17:00
jcsackettflacoste: that was the bad revno. qaprocess says rollback=badrevno, unless i'm being dense?17:01
flacosteUrsinha: how can we unblock the qatagger?17:01
deryckadeuring, so you need me to run it again?17:01
adeuringderyck: no, I just want to double-check if I did not make a copy_paste error or some other nonsense17:01
flacostejcsackett: deployment-stable.html shows "[r=jcsackett][bug=707234][rollback=13521] " has the commit message17:01
flacostewhich is not the correct revision17:01
flacostebut the tags are correct17:02
jcsackettflacoste: aaah, crap, yes i see.17:02
deryckadeuring, ah, ok.17:02
flacosteanyway, we should be good to deploy up to 1326517:04
flacosteso i'm going to ask for that17:04
flacosteafter lunch17:04
jcsackettsounds good.17:05
timrcjcsackett, do you think you'll be able to attempt to ec2-land my changes again :)?17:14
abentleysinzui: I also get the segfault on my natty vm, now that I've properly updated it.17:15
sinzuiabentley, does the vm install anything special? Is it natty + rocketfuel?17:16
abentleysinzui: Not special.  An X86-64 Ubuntu desktop.  I can provide a package list if useful.17:17
abentleysinzui: it does have libgtk2.0 on it.  Shall I pull that?17:17
sinzuioh17:17
sinzuideryck, are you 64bit?17:18
sinzuiabentley, a package list would rock.17:18
derycksinzui, indeed I am.17:19
derycksinzui, I should have mentioned that.  it's important.  sorry.17:19
jcsacketttimrc: sure, paste me the link, and i'll see to it in a moment. :-)17:22
timrcjcsackett, I pushed a new branch 4 days ago that fixed those test failures.. should I merge in trunk before attempting this again.17:23
timrc?17:23
timrc(it's been 4 days)17:23
timrchttps://code.launchpad.net/~timrchavez/launchpad/set_ppa_private_from_api_724740-4/+merge/6484017:23
timrcjcsackett^17:23
=== salgado is now known as salgado-lunch
* jml pulls stable, first time in a while17:32
=== matsubara is now known as matsubara-lunch
adeuringgmb: fancy a review of a small MP: https://code.launchpad.net/~adeuring/launchpad/bug-799785/+merge/65373 ?17:34
gmbadeuring: Sure17:34
LPCIBotProject windmill-devel build #259: STILL FAILING in 40 min: https://lpci.wedontsleep.org/job/windmill-devel/259/17:41
=== gmb changed the topic of #launchpad-dev to: Performance Tuesday | https://dev.launchpad.net/ | On call reviewer: - | Critical bugs: 205 - 0:[######=_]:256
gmbadeuring: Approved17:42
adeuringgmb: thanks!17:42
Ursinhaflacoste: sorry, I was lunching and forgot to change my nick17:54
Ursinhalet me read the backlog17:55
jcsacketttimrc: you should basically always update from trunk before trying to land.17:59
Ursinhaflacoste: if the problem is that the commit msg has the wrong number, quick workaround is remove the tag from the bug; proper solution is to fix bug 65962918:02
_mup_Bug #659629: qa metadata for commits is write-once and cannot be updated <qa-tagger:Triaged> < https://launchpad.net/bugs/659629 >18:02
LPCIBotProject db-devel build #652: STILL FAILING in 6 hr 10 min: https://lpci.wedontsleep.org/job/db-devel/652/18:17
flacosteUrsinha: you mean remove the bad-commit-13521 tag?18:23
flacosteerr, bad-commit-1325118:23
Ursinhayes18:23
Ursinhaif we know that is ok to land, I believe adding qa-ok would do the trick18:23
LPCIBotProject parallel-test build #56: STILL FAILING in 46 min: https://lpci.wedontsleep.org/job/parallel-test/56/18:28
LPCIBotLaunchpad Patch Queue Manager: [r=lifeless][bug=797820, 800043] In ValidPersonOrTeamVocabulary,18:28
LPCIBotmove non-exact non-FTI matches back down into the FTI range. Also18:28
LPCIBotmatch IRC nicknames case-insensitively.18:28
jmlg'night all18:32
Ursinhanight jml18:34
=== salgado-lunch is now known as salgado
m4n1shIsn't a wait for 18 hours  a bit too much for PPA build18:49
m4n1shhttps://launchpad.net/~zeitgeist/+archive/ppa/+build/258283718:49
LPCIBotProject windmill-db-devel build #412: STILL FAILING in 1 hr 5 min: https://lpci.wedontsleep.org/job/windmill-db-devel/412/18:51
maxbUndesirable, sure. The available resources just aren't there18:51
LPCIBotProject devel build #823: STILL FAILING in 6 hr 5 min: https://lpci.wedontsleep.org/job/devel/823/18:57
m4n1shsad18:59
abentleym4n1sh: It appears we lost the borrowed builders on June 17 and the queue's been a little nuts since then.19:05
m4n1shabentley: borrowed builders?19:06
abentleym4n1sh: A bunch of our builders are actually borrowed from another department.  Hardware enablement?  Can't recall.19:07
m4n1shhmm19:07
m4n1shnever mind, can wait19:07
abentleysinzui: So much stuff depends on libgtk2.0-0...19:31
sinzuiabentley, yeah19:31
sinzuiabentley, I really suspect 64bit is the issue. Maybe deryck can confirm he is using 64bit19:35
derycksinzui, I am19:35
* deryck thought he confirmed that this morning19:35
sinzuideryck, sorry, I was being an idiot19:37
sinzuiNow I need to get a 634bit install19:37
derycksinzui, no, no worries.  Just couldn't completely recall myself. :)19:37
flacostem4n1sh, abentley: yes, about half of the builders are taken for hardware enablement at critical points in the Ubuntu release cycle, it usually lasts a few days (though sometimes they forget to put them back in the main pool)20:05
timrcjcsackett, okay, merged trunk 39 minutes ago... (sorry I've been pulled 50 different directions)20:18
timrcjcsackett, https://code.launchpad.net/~timrchavez/launchpad/set_ppa_private_from_api_724740-4/+merge/6484020:18
jcsacketttimrc: i saw when you did, i've sent it out.20:18
timrcjcsackett, great, thanks :) I will begin praying now20:19
jcsackettsinzui: have some time to chat?20:50
sinzuijcsackett, you request is timely. yes20:50
bacflacoste: i'm trying to do a collection assignment in the API/lplib but am getting NoBoundRepresentationError.  am trying archive.enabled_restricted_families = [arm] -- any hints?20:56
bacwhere arm is a proper IProcessorFamily object20:56
flacostebac: we don't support that20:57
flacostebac: that's why i suggested exporting the mutator as an operation20:57
flacostesetEnabledRestrictedFamilies(families)20:57
bacflacoste: i must've missed that suggestion20:58
flacosteor add/removeEnabledRestrictedFamily()20:58
flacostei probably wasn't clear20:58
flacosteso get -> through the collection20:58
bacflacoste: or it flew over with all the other stuff we discussed20:58
flacosteset -> through operations20:58
flacosteone or many20:58
flacosteadd/Remove/set20:59
flacostei don't have a strong opinion20:59
flacosteset is probably fine20:59
bacflacoste: thanks!20:59
flacostesince they can get the collection, turn that into an array20:59
flacostelist20:59
flacosteand then call set with the changes20:59
lifelessmoin21:37
jcsackettsinzui: have a moment?21:57
=== salgado is now known as salgado-afk
=== matsubara is now known as matsubara-afk
lifelesspop quiz23:11
=== lifeless changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: - | Critical bugs: 205 - 0:[######=_]:256
lifelesswhats a great micro-framework for python webapps, present (and usable) in lucid ?23:12
lifelessgary_poster: hi; are there docs on setting up a new buildout-as-we-use-it for a new project?23:12
cody-somervillelifeless, django? :P23:14
lifelesscody-somerville: *micro*23:15
cody-somervillelifeless, python-flask advertises its self as a micro framework but isn't available in lucid. However, in addition to python-django I did find the following in lucid: python-quixote, python-pylons, python-cherrypy3, python-turbogears2, and python-webpy23:21
LPCIBotProject parallel-test build #57: STILL FAILING in 1 hr 17 min: https://lpci.wedontsleep.org/job/parallel-test/57/23:29
lifelesscody-somerville: yeah, none of those are microframeworks23:31
lifelessexcept webpy kindof23:31
lifelessI think I'll give web.py a shot23:33
lifelessbut first, lxc23:33
LPCIBotProject windmill-devel build #260: STILL FAILING in 1 hr 6 min: https://lpci.wedontsleep.org/job/windmill-devel/260/23:33
lifeless*hate* sample data.23:48
lifelesscorrupt conjoined master row in the sample data.23:49
wallyworld_which table?23:51
lifelessbugtask23:52
lifeless bug 323:52
_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 >23:52
wallyworld_very hard to get rid of sample data given how many tests use it23:52
lifelessshows 'status tracked in sarge'23:52
lifelesswhich means that the sarge row is a conjoined master23:52
lifelessbut the sarge row is assigned to a milestone, and the master row is not.23:53
wallyworld_i assume that would never be allowed to happen in practice since our business logic would enforce the required ata integrity rules23:54
lifelessthis makes bugsummary fail the distro portlet test because the sample data isn't consistent - the old code /may/ have queried series tasks23:54
lifelessI'm looking into the old code now :)23:54
lifelessright23:56
lifelessthe prior code was counting milestones on either distro or distroseries23:56

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