cjwatsonupgrading mawson for some QA00:37
cjwatsonactually, maybe I can do this on qas - never mind, won't *hurt* to have dogfood more up to date00:39
StevenKcjwatson: Doing QA after midnight?00:41
cjwatsonif I'm awake anyway00:42
wgrantcjwatson: Can you tell me when you're done, please?00:44
cjwatsonwgrant: dogfood upgrade finished - sorry if I got in your way00:48
wgrantcjwatson: Not significantly :)00:51
StevenK% grep -m 1 def lib/lp/code/tests/test_branch_webservice.py02:13
StevenK    def test_createMergeProposal_fails_if_reviewers_and_review_types_are_different_sizes(self):02:13
* StevenK eyerolls02:13
lifelessmwhudson: I havdn't. Thanks.02:14
mwhudsonlifeless: i found it very interesting02:17
wallyworld_lifeless: your branch in bb now. let's hope nothing goes wrong in the next 5 hours :-)02:32
cody-somervillemwhudson, Are you still working on that project group milestone bug fix?02:33
mwhudsoncody-somerville: not really02:34
mwhudsoncody-somerville: i'm certainly not doing it on work time02:34
lifelessthakns wallyworld_02:35
* cjwatson gets out of the way of lifeless' deployment pipeline steamroller.03:02
wallyworld_wgrant: if you get a chance later, could you run this on df https://pastebin.canonical.com/6714 ... it's a standard person vocab query but with an extra left join from person to account03:26
wgrantwallyworld_: That looks like it's missing a digit.03:27
wgrantAlthough that is a very old person vocab query.03:27
wgrantWhich probably won't work on the current schema.03:27
wgrantIs that just a coincidence?03:27
wallyworld_that's what a standard test runs03:27
wgrantwallyworld_: Your paste URL is from 200803:28
wallyworld_i just logged sql when running a test03:28
wgrantIt's a Launchpad pillar search query03:28
wallyworld_ffs, sorry https://pastebin.canonical.com/67141/03:28
wgrantIt's a remarkable coincidence.03:28
wgrant Total runtime: 127.058 ms03:29
wallyworld_seems ok i guess for a vocab query? i'll paste the version without the account left join03:29
wgrantNice move with the CTE03:29
wgrantIt's an effective planner barrier.03:29
wgrantSo it almost forces a good plan.03:30
wgrantAnyway, I'm wandering off for a while -- be back in 30 or so.03:30
wallyworld_the cte was already there03:31
wgrantwallyworld_: Within 2ms of the same time03:31
wallyworld_wgrant: excellent, thanks03:32
wgrantThe account filtering is done afterwards on an average of about 102 people, so it's trivial.03:32
wgrantReally gone now.03:32
StevenKwallyworld_: https://code.launchpad.net/~stevenk/launchpad/branch-information_type-factory/+merge/10809404:01
wallyworld_StevenK: info type defaults to public, right? ie if left unspecified in factory call?04:04
StevenKwallyworld_: No, it defaults to None, which means don't override it04:09
StevenKIn the old code private=False ended up being an no-op04:10
StevenKwallyworld_: If it defaulted to PUBLIC, If you said makeBranch(product=private_product) the branch would get created as USERDATA and then overridden to PUBLIC.04:12
lifelessmwhudson: thanks for the link04:41
lifeless"The subtle beauty of a consistent system is that the invariants tend to hold even when the designer does not know what they are."04:44
mwhudsonlifeless: i thought you'd like it04:50
mwhudsonthe commutative datatypes sound interesting too04:50
* StevenK stabs update-cache05:04
StevenKwallyworld_, wgrant: https://code.launchpad.net/~stevenk/launchpad/no-makebugtask-private/+merge/10809605:06
wgrantStevenK: There was really only one callsite?05:09
StevenKOnly one callsite that did makeBugTask(private=True), yeah05:10
=== jtv1 is now known as jtv
* wallyworld_ just went out a bought a new Galaxy S3 \\o/05:51
StevenKwallyworld_: And sold the new laptop to pay for it?05:51
wallyworld_nah, that's what credit cards are for :-)05:51
StevenKOh, so your plan is to pay for it eventually05:52
wallyworld_but i have to get a micro sim :-(05:52
wallyworld_yeah , sometime by 202505:52
wallyworld_but i am really bad at waiting05:52
StevenKwallyworld_: Naughty. I pay my credit card off in full every month.05:52
wallyworld_i try to, but i spend too much money i don't have05:52
StevenKWell, that's the root cause, isn't it? Don't spend money you don't have. :-)05:53
wallyworld_but it's new and shiny and i wanted it and i couldn't wait05:54
StevenKDidn't your mother ever teach that "I want never gets" ?05:54
wallyworld_nope :-)05:55
* wallyworld_ decides whether to get a box cutter to try and make the old sim card fit05:55
=== Ursinha` is now known as Ursinha
=== Ursinha is now known as Guest90314
adeuringgood morning07:05
lifelesswallyworld_: thanks for your assistance this morning07:11
lifelessnoodles775: https://dev.launchpad.net/Running/LXC07:11
lifelessnoodles775: may help with your development environment07:11
wallyworld_lifeless: no problem, i did very little. i hope everything works well.only 1 hr till bb finishes :-)07:15
mwhudson"New code import: nyancat/trunk"08:40
* jml googles nyancat08:56
mwhudsonjml: don't do it!08:57
gmbmgz, You said something yesterday about bzr using a separate pipe for subunit output. Where can I look to find out how you actually go about doing that in a safe, sane, clean and above all quick-to-implement way (note that none of the above values have actually to be true).08:58
jmlmwhudson: it's a repetitive, high-pitched cat-cake rainbow thing08:58
mwhudsonjml: yes08:58
mwhudsonjml: congratulations on not knowing about this particularly useless internet meme until now08:59
jam1sinzui: when you're around, I'm having a funny bug wrt launchpad answers. I set myself as an answers contact, and I see myself on https://answers.launchpad.net/launchpad09:00
jam1however, I *don't* see myself on any given question (like: https://answers.launchpad.net/launchpad/+question/198964)09:00
jmlgmb: why not insist on a just, fair, true and loyal implementation?09:00
jam1though francis said he could see me on the list09:00
jmlmwhudson: thanks :)09:00
jam1and... I'm not getting emails for new requests, I do seem to get posts to old ones09:00
=== jam1 is now known as jam
gmbjml, I will accept fair, true and loyal, as long as I can have peace, respect for all, free love and a hard-boiled egg.09:00
jmlgmb: I'm afraid I rather insist on having my eggs soft-boiled, old chap.09:01
gmbWell, no-one's perfect.09:01
jmlwhat difference is there between LaunchpadZopelessLayer and ZopelessDatabaseLayer that might affect mail?09:03
wgrantNot much.09:04
wgrantUnless it uses the librarian...09:04
jmlI guess this must, somehow.09:05
jmlAs when I switched it to use ZDL, the failures I got were about mail not being sent.09:05
wgrantI don't think we change the mail config in LaunchpadLayer...09:06
wgrantIf anything, I'd expect it to send less mail.09:06
jmlanyway, I guess I'll push that forward to another patch09:07
jmljames's thing09:07
wgrantjames's thing?09:07
jmlwgrant: https://code.launchpad.net/~james-w/launchpad/drop-filecmp/+merge/10802109:08
czajkowskijam: want to mumble and I can talk you through it09:08
jamczajkowski: I'm on now09:10
jamczajkowski: I only hear static when you talk09:12
jamwe can just do IRC if you prefer09:13
jamalso, I'd like to document it somewhere, so someone else in the future has access to it09:13
jmlanyone already have a simple script to check current loc score for a branch?09:27
jmlotherwise I'll hack one up with diffstat, cut & dc09:27
gmbjml, If you find one, let me know. I've got a slack-time card for it.09:27
gmbHaven't got round to it yet.09:27
gmb*find or make09:27
cjwatsonhttp://paste.ubuntu.com/1016103/ (~/bin/diffstat-loc-delta here) postprocesses diffstat to a single number09:28
cjwatson(yeah, it's perl, bite me)09:28
cjwatsonor you could probably repurpose bits of loc-contributions in lp-dev-utils09:28
jmlcjwatson: heh09:28
cjwatsonwhich is vastly more overengineered09:28
jmlcjwatson: at least it's strict perl09:29
cjwatsonloc-contributions gets kind of confused if you run it in a non-devel branch and (I think) attributes everything after the branch point to the last rev it has from devel and shows the delta for that commit, which in the general case has nothing to do with the author it's supposed to be examining.09:32
cjwatsonProbably not desperately hard to fix ...09:32
bigjoolsbzr diff -r submit:|diffstat ?09:32
StevenKbigjools: I think jml was after one number09:33
cjwatsonright, so with my script, bzr di -rsubmit: | diffstat-loc-delta09:33
bigjoolsah ok, didn't catch on to that09:33
cjwatson(or -rancestor: for a bit less noise from bzr)09:34
bigjoolsancestor needs more typing :)09:34
cjwatsonbigjools: thanks for doing *PPH.requestDeletion() API exporting way back when, BTW - I only just cottoned onto it being there and wrote a client for it09:35
jmljml@lpdev:~/src/launchpad$ bzr damage09:35
jmlUsing submit branch file:///home/jml/.repos/launchpad/devel/09:35
jmlDamage: 67 lines of code09:35
bigjoolscjwatson: \o/09:35
cjwatsonjml: cute09:35
bigjoolsjml: haha!09:35
bigjoolsso it's net difference09:36
gmbmgz, nm, I've found the relevant code.09:38
* jml searches for a team with suitably broad write permissions.09:38
cjwatsonjml: http://paste.ubuntu.com/1016109/ ? :-)09:39
jmlcjwatson: :D09:39
jmlthat's on lp:bzr-damage if anyone wants it. ~canonical has commit access. based on lp:difftodo, which I use all the time.09:42
jmlgmb: ^09:42
gmbjml, Awesome, as they say, sauce. Ta.09:42
gmb"they," of course, being the "everyone" in "everyone knows."09:43
* bigjools is off, g'night folks09:45
StevenKwallyworld_: Yours is red because r15335 needstesting09:46
StevenKAnd the QA machinery knows r15325 is a bad rev09:46
wallyworld_StevenK: but those 2 revs aren't related09:47
StevenKwallyworld_: Your rollback is r15336, so everything between r15325 and it has to be deployable for r15325 to be.09:47
wallyworld_ah, ok then09:48
wallyworld_StevenK: btw, my scissors and the micro sim experiment worked :-)09:49
StevenKOh, twitch09:49
mgzgmb: sorry for delay, trying to hack through to getting code commited while it's all in my head09:57
mgzwill reply in a sec09:57
stubShould all branches to fix a bug land with [incr], or only the second+, or just the first, or what?09:59
wgrantstub: Anything that doesn't fix the bug, normally.09:59
wgrantstub: It just prevents qa-tagger and me from closing the bug.10:00
mgzgmb: okay, I'm through the thicket, if you've got any questions still about subunit, bug me10:33
gmbmgz, At the moment, my questions are all very much of the nature of "that's really straightforward! Now, how the sweet hell do I get zope.testing to do that without breaking the entire universe?"10:33
gmbBut no fear, I'll bug you as necessary.10:33
mgzright, that'll be the tricky bit :)10:34
jmlfwiw, I did zope.testing's subunit support initially10:44
jmlso I might be able to help a little10:44
mgzjml: the basic desure is to keep the subunit stream off stdout when running tests in more than one process10:46
mgzit's a bit to easy to break the world if a test has something that ends up printing to a standard stream10:52
jmlisn't that just subunit's passthrough stuff being borked?11:01
mgzit doesn't help that the format is super-fragile, but TestProtocolClient(testresult.TestResult): """A TestResult whic11:08
mgz...bad paste11:08
mgzanyway, you can pass a dedicated stream, which is then less liable to random breakage11:09
jmlhttps://code.launchpad.net/~jml/launchpad/generate-ppa-logging/+merge/108040 needs review12:13
jmlunfortunately I've made it huge in an attempt to clean up12:13
jmlwgrant: if you're around... ^12:16
rick_h_jml: jcsackett should be around in the next hour/two and is OCR. I think wgrant is pretty EOD/night at this point.12:17
wgrantjml: I can't review that tonight, sorry. I'll be more than happy to do it tomorrow if nobody else does it.12:20
jmlwgrant: np. thanks.12:21
=== Guest90314 is now known as Ursinha
cjwatsonHmm, members of ubuntu-archive who aren't in ubuntu-core-dev can't run auto-sync13:33
* cjwatson ponders whether that's correct13:34
cjwatsonI can see how it arises, obviously; rather impedes automation though13:34
=== jcsackett changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: jcsackett | Firefighting: - | Critical bugs: 3.47*10^2
=== sagaci_ is now known as sagaci
jmljcsackett: hi14:51
jcsackettjml: heya.14:52
jmljcsackett: would you please take a look at https://code.launchpad.net/~jml/launchpad/generate-ppa-logging/+merge/108040 ?14:52
jcsackettjml: happily.14:52
jmljcsackett: it's kind of long. sorry.14:52
jmljcsackett: I went far & wide to try to make it line-count neutral. as it is it's still 30 lines in debt.14:52
jmljcsackett: oh, you should also note that an earlier version of said patch is currently cowboyed, running on germanium14:53
jmljcsackett: thanks.14:53
jcsackettjml: got delayed on looking at your MP; i'm a little ways through, but i've noticed that jelmer reviewed your code about 19 hrs ago and approved. did you explicitly want a second review, or just hadn't noticed it had a vote already?15:29
jmljcsackett: second review. jelmer approved an earlier, shorter. version for cowboying but not for landing.15:30
jcsackettjml: aaah. okay, i understand now.15:30
jcsackettjml: one quick note on the LoC thing; you can actually run a tool from lp-dev-utils that tells the total line debt you have. in this case, you're at -42, so the extra 30 still leaves you ahead of the game.15:31
jmljcsackett: oh, interesting. This is from a separate arc of work though, so I would personally like to count it separately.15:33
jcsackettjml: dig.15:33
* jml think of 'bzr damage' hacks.15:33
jmljcsackett: fwiw, the assertContentEqual change is busted. I'm reverting that locally.15:35
cjwatsonjcsackett: When you get a chance, I could use a review of https://code.launchpad.net/~cjwatson/launchpad/packageset-score-rename-support/+merge/107981 - fixing this is going to take three successive deployments, so I'd like to get it moving15:36
jcsackettcjwatson: sure thing. i'll look as i finish with jml's.15:37
jcsackettjml: r=me on the MP. if you address the comments issues and set the commit msg, i can send that out to land for you.16:22
jcsackettcjwatson: taking a look at yours now.16:22
jmljcsackett: thanks. will do.16:23
jmljcsackett: one thing though, when I said 'numbers are arbitrary', I meant the 3, 45 & 16 I was using for testing, not the constants for how many seconds are in a day or microseconds in a second :)16:24
jmljcsackett: I can change either or both of those if you want, although I don't think either change will do much to enhance readability16:24
jcsackettjml: my comments are poorly placed. i am referring only to the parts about dropping this stuff when we drop py 2.616:25
jmljcsackett: ah right. can do. I assume I don't have to file a bug for those XXX comments then?16:26
jcsackettjml: i see no need for that, and there are plenty of XXX comments without a bug. just name and date are fine. :-)16:27
jmljcsackett: cool. thanks. worth changing https://dev.launchpad.net/PolicyandProcess/XXXPolicy then, I reckon.16:29
jcsackettjml: probably so, yes.16:29
jcsackettcjwatson: r=me.16:30
cjwatsonjcsackett: Great, thanks!  Could you "Status: Approve" it and I'll land it16:31
jcsackettcjwatson: oh right; you can land but not mark approved. strange corner case to be in. :-P16:32
jcsackettcjwatson: done.16:32
* jml <3 wikis16:33
cjwatsonLovely.  Off to EC2.16:33
jmljcsackett: I'm in the same boat actually.16:35
jmljcsackett: changes pushed.16:35
jcsackettjml: ok, then i've flipped the final toggle and you may land your branch. :-)16:36
cjwatsonjml: Good to know it's not just me.16:36
frankbanjcsackett: could you please review https://code.launchpad.net/~frankban/launchpad/bug-996720/+merge/108196 ?16:40
jcsackettfrankban: already looking at it. :-)16:41
frankbanthanks jcsackett :-)16:41
jcsackettfrankban: r=me. good fix.16:45
frankbangreat, thank you16:45
=== salgado is now known as salgado-lunch
cjwatsonWhat happens to OOPSes generated during tests?17:02
cjwatsonActually I guess I don't know if it's an OOPS.  I'm getting an Unauthorized with a rather inscrutable response body, and would like to try to figure out what's trying to fetch the attribute ("items", so not very specific) that's the proximate cause of the Unauthorized.17:02
cjwatson... I've fixed my immediate problem by staring closely at print debugging.  I'm still interested if there are more powerful tools I'm missing out on.17:10
jmlcjwatson: that's almost certainly caused by zope's security proxy17:23
jmlcjwatson: afaict, oopses ought to be captured by default within tests.17:23
cjwatsonFor once it actually wasn't the security proxy17:24
cjwatsonWell, maybe it was, not sure.  I was returning a list of lists of dicts by accident when the interface said list of dicts.17:24
=== deryck is now known as deryck[lunch]
jmlcjwatson: it'll be the proxy raising that error.17:31
cjwatsonPerhaps my question would be better phrased as "how do I actually get to see that OOPS?".17:31
cjwatsonI was getting stuff of the form http://paste.ubuntu.com/1016677/ .17:32
=== deryck[lunch] is now known as deryck
sinzuijcsackett, do you have time to review https://code.launchpad.net/~sinzui/launchpad/queue-project-email/+merge/10823520:25
jcsackettsinzui: sure.20:27
jcsackettsinzui: r=me.20:43
sinzuithank you.20:43
=== salgado is now known as salgado-afk
=== jcsackett changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: - | Firefighting: - | Critical bugs: 3.47*10^2
bdmurrayI thought I'd look at doing some launchpad dev and I'm having trouble running it at the moment22:17
bdmurrayCouldn't find a distribution for 'lazr.jobrunner=0.6'22:17
czajkowskibdmurray: have you seen the latest write up http://blog.launchpad.net/general/contributing-to-launchpad22:22
bdmurrayczajkowski: yes and one thing that page doesn't address is actually running launchpad so you can see what it looks like and interact with it via a browser22:26
bdmurrayor how to keep your launchpad development system up to date22:27
cjwatsonbdmurray: did you do just 'bzr pull' rather than utilities/rocketfuel-get?22:35
cjwatsonThat looks like lp-sourcedeps is out of date22:35
wgrantbdmurray: bzr up ~/launchpad/lp-sourcedeps/download-cache22:35
bdmurraythanks, so is utilities/rocketfuel-get the best way to stay up to date or something else?22:40
wgrantbdmurray: That's the best way, yeah.22:41
wgrantIt mostly just pulls devel, ups download-cache, and runs utilities/update-sourcecode22:42
cjwatsonCould I have a DB patch number for "ALTER TABLE Packageset RENAME COLUMN score TO relative_build_score;", please?  Bug 1000570.23:12
_mup_Bug #1000570: "Packageset.score" is badly named <qa-untestable> <tech-debt> <trivial> <Launchpad itself:In Progress by cjwatson> < https://launchpad.net/bugs/1000570 >23:12
cjwatsonI've already tested the packageset-score-rename-support branch linked to that bug both with and without that DB patch applied.23:14
StevenKcjwatson: 18-423:14
cjwatsonThanks.  Earlier than the tip patches in db-devel?23:14
cjwatsonI guess it doesn't matter23:14
StevenKIt does not23:14
cjwatsonOK.  I'll push it for review tomorrow; it's missed the upcoming NDT anyway, so no rush.23:16
StevenKcjwatson: DB patch requires FDT23:17
StevenKOh, sorry, I missed your other comment23:17
cjwatsonI was unclear.  I mean the code support branch has missed the upcoming NDT anyway, so no rush.  I understand that the DB patch will require an FDT.23:21
lifelesssinzui: https://bugs.launchpad.net/launchpad/+bug/1007208 may or may not interest you23:21
_mup_Bug #1007208: Searching for 'ev' in the 'add a member' picker fails <Launchpad itself:Triaged> < https://launchpad.net/bugs/1007208 >23:21
StevenKcjwatson: We have FDTs until at least Tuesday or Wednesday23:23
wgrantlifeless: I think that's a dupe.23:24
wgrantlifeless: A dupe of searching for mgz.23:24
wgrantWe certainly considered it in our picker work.23:24
lifelessanyone remember why we have both ~launchpad-reviewers *and* ~canonical-launchpad-reviewers ?23:33
bigjoolsless email?  more email? :)23:42

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