[06:55] <wgrant> lifeless: Ah, I'm glad you have a better solution than granting launchpad.View on private teams just because of a PPA subscription.
[07:11] <lifeless> wgrant: I try
[08:45] <mwhudson> morning
[08:46] <wgrant> Evening.
[08:48] <lifeless> day.
[08:50] <StevenK> G'Tag
[08:51] <wgrant> That's not very Czech.
[08:52] <StevenK> Oh, sorry. Dzień dobry
[10:02] <poolie> leonardr, wbn to extract whatever groundcontrol uses for this and make it sit on top of a generic authenticate mechanism, and be the default gtk impl
[10:03] <leonardr> poolie, i don't think it's worth it, given that the common case will be to use the credentials manager packaged with ubuntu
[10:17] <poolie> jml wow ec2 test is pretty classy
[10:36] <lifeless> poolie: you've seen bzr selftest --parallel=ec2 ?
[10:36] <poolie> heard of it
[10:36] <lifeless> :P
[10:36] <poolie> i think i tried it and hit some snags
[10:37] <poolie> i think for me it looked like a bad latency/throughput tradeoff :)
[10:37] <poolie> maybe i should try it
[10:37] <wgrant> But with LP it's different, since the test suite is damn slow?
[10:37] <poolie> lifeless,  i'd actually like a variation that runs remotely on my desktop
[10:37] <poolie> yeah basically
[10:47] <leonardr> poolie, want to review the launchpadlib branch we talked about yesterday?
[10:47] <leonardr> https://code.edge.launchpad.net/~leonardr/launchpadlib/improve-workflow/+merge/29849
[10:48] <poolie> i'd be delighted
[10:49] <leonardr> poolie: the reason there are no automated tests is we don't have the framework for them
[10:50] <poolie> leonardr, you know when i said 'rhinos' i meant the internal mailing list about this? not the animals?
[10:50] <leonardr> poolie: i thought you meant the o'reilly book (after whom the list is presumably named)
[10:51] <mwhudson> the problem with launchpad for ec2 is that instance setup is quite slow of course
[10:51] <mwhudson> we could have an instance with the results of 'make schema' baked into it, that would help a bit
[10:51] <poolie> nice cover letter
[11:05] <poolie> leonardr, reviewed, very nice but i have some tweaks
[11:05] <leonardr> ok
[11:48] <leonardr> poolie, maybe you can help me understand how os.open works
[11:48] <poolie> leonardr, maybe i can :)
[11:48]  * poolie waves
[12:02] <wgrant> Is ShipIt still a Launchpad parasite?
[12:03] <wgrant> Ah, lunch, I guess.
[12:03] <mwhudson> wgrant: yes
[12:04] <mwhudson> wgrant: to both bits
[12:04] <wgrant> mwhudson: :(
[12:04] <mwhudson> wgrant: although there was talk of not rolling out launchpad to shipit app servers any more
[13:36] <mwhudson> gary_poster: it seems our VirtualHostRequestPublicationFactory doesn't really implement IRequestPublicationFactory, because __call__ returns a request *factory*, not a request
[13:36] <mwhudson> gary_poster: does this ring any bells?
[13:39] <mwhudson> or hm
[13:39] <mwhudson> maybe the docs are just broken
[13:41] <mwhudson> yeah, i think so
[13:50] <gary_poster> mwhudson: (Sorry no reply before) That sounds likely to me, fwiw.  If it works with the rest of the Zope machinery, then that sounds like a good argument.
[13:50] <mwhudson> gary_poster: np
[14:37] <maxb> I appears there is one or more Git imports in the Reviewed state which somehow I am Forbidden to see
[14:37] <maxb> This manifests as a Forbidden error when I try to page through the list of them, on one particular page
[14:39] <maxb> It appears to be number 459 in the list
[14:40] <maxb> and mumber 498 too
[14:41] <mwhudson> maxb: hmm
[14:41] <mwhudson> maxb: sounds worth of bug filing
[14:42] <maxb> https://code.edge.launchpad.net/+code-imports/+index?field.rcs_type=GIT&field.rcs_type-empty-marker=1&field.review_status=REVIEWED&field.review_status-empty-marker=1&submit=Submit+Query&start=497&batch=1
[14:42] <maxb> Can you view that?
[14:43] <maxb> https://code.edge.launchpad.net/+code-imports/+index?field.rcs_type=GIT&field.rcs_type-empty-marker=1&field.review_status=REVIEWED&field.review_status-empty-marker=1&submit=Submit+Query&start=458&batch=1
[14:43] <maxb> or that?
[14:43] <maxb> hoping to find someone to figure out which ones are the problem, so I can say something more definite than an index in search results which will change
[14:45] <mwhudson> maxb: i can't see that
[14:45] <maxb> hmm. I guess we'll need a losa
[14:45] <mwhudson> but i can see the branch name in the traceback
[14:46] <mwhudson> i assume it's a private branch-only project
[14:46] <maxb> makes sense
[14:51] <james_w> leonardr: hi, is there a solution for using launchpadlib.load in (launchpad) tests that doesn't involve hardcoding the root uri or accessing a private attribute? I thought I had seen a branch from you to make load work with relative URIs.
[14:58] <leonardr> james_w: yes, in the latest version of launchpadlib you can pass a relative url to load()
[14:58] <leonardr> or, actually, in the latest lazr.restful
[14:58] <james_w> leonardr: is that landed in launchpad yet?
[14:58] <leonardr> lazr.restfulclient
[14:58] <leonardr> yeah, it's lazr.restfulclient 0.9.20, revno 102
[14:59] <james_w> grep restfulclient versions.cfg
[14:59] <james_w> lazr.restfulclient = 0.9.14
[15:00] <leonardr> oh, i see what you mean
[15:00] <leonardr> is it part of the launchpad application, not is it on the launchpad site
[15:00] <james_w> I'll try an upgrade branch
[15:01] <leonardr> i'll do a branch upgrading the packages later this month
[15:01] <leonardr> remind me to do it, if you would
[15:04] <james_w> leonardr: you don't want me to do it now?
[15:05] <james_w> leonardr: is there some way that I can mark the place in the code where I will put the ugliness for replacement at that point?
[15:05] <leonardr> james_w: sure, if you want to take care of it i'd be very happy
[15:06] <leonardr> i didn't know that was what you meant by 'upgrade branch'
[15:06] <james_w> leonardr: latest versions of lazr.restfulclient, launchpadlib and lazr.restful?
[15:06] <leonardr> let me double check, but yes
[15:06] <james_w> i.e. should I upgrade all of them?
[15:07] <leonardr> james_w: yes. send me the mp, but if the ec2 test passes, it's fine
[15:08] <james_w> leonardr: any gotchas you can think of that would save me time?
[15:08] <james_w> any dependencies that should be upgraded or anything?
[15:08] <leonardr> james_w: the only thing i'm worried about is whether 'make' will generate a reasonable-looking apidoc
[15:09] <leonardr> wait, actually it doesn't matter, because i didn't release that launchpadlib revision as a new version of launchpadlib
[15:09] <leonardr> so it should be fine
[15:09] <james_w> ok
[15:10] <poolie> hi james_w
[15:10] <james_w> hi poolie
[15:10] <poolie> new post: http://blog.launchpad.net/api/three-tips-for-faster-launchpadlib-api-clients
[15:11] <poolie> lifeless, one nice thing about having everything in debs would be less crap about 'please run link-external-sourcecode' etc
[15:12] <james_w> I'm actually starting to like the approach taken by buildout etc. for development
[15:12] <benji> poolie: utilities/link-external-sourcecode -p ~/launchpad/lp-sourcedeps/
[15:13] <james_w> poolie: co-located branches would make that less of an issue too :-)
[15:14] <poolie> true
[15:15] <poolie> hm, though you'd still need to rebuild them etc
[15:23] <mwhudson> writing unit tests for publication details makes me want to stab things
[15:26] <mwhudson> ILaunchBag needs to die
[15:27] <poolie> ILaunchBagBiter
[15:29] <mwhudson> ILaunchPoo
[16:02] <kiko> flacoste, mwhudson: who currently runs our mootbot?
[16:03] <mwhudson> kiko: no idea
[16:20] <leonardr> poolie, https://bugs.edge.launchpad.net/launchpadlib/+bug/605462
[16:22] <mwhudson> gary_poster: this diff adds a vostok layer: http://pastebin.ubuntu.com/463574/
[16:22] <gary_poster> looking
[16:22] <mwhudson> gary_poster: this one adds a very simple new view for the root http://pastebin.ubuntu.com/463575/
[16:26] <gary_poster> mwhudson: for first patch, I don't understand change to lib/lp/code/xmlrpc/tests/test_codehosting.py : why did it say "vostok" in the first place?
[16:27] <mwhudson> gary_poster: the scripts used to be run on a machine called that, i guess i went for 'realistic' test data
[16:27] <gary_poster> heh, what a coincidence
[16:28] <mwhudson> yeah
[16:28] <mwhudson> i changed it because i want grep -ir vostok to only find this stuff :)
[16:28] <gary_poster> heh, cool, makes sense
[16:33] <gary_poster> mwhudson: oooh, I like your root template
[16:33] <mwhudson> gary_poster: :)
[16:38] <gary_poster> mwhudson: a 50% review: looks good to me.  It is sad that it requires so much machinery, though we do this so infrequently I suppose it is not worth thinking about
[16:41] <mwhudson> gary_poster: thanks
[16:42] <james_w> you know, it might be useful for us to have vostok.launchpad.net as an "unsupported" interface to launchpad.net.
[16:45] <mwhudson> james_w: i guess you looked at the diffs
[16:46] <james_w> I did
[16:46] <mwhudson> cool
[16:48] <mtaylor> thumper: around?
[16:49] <mwhudson> mtaylor: sorta
[16:49] <mwhudson> mtaylor: if you say stuff he'll see it i imagine
[16:49] <mtaylor> mwhudson: heh. something seems to be quite off with merge proposal diff
[16:49] <mtaylor> diffs
[16:50] <mwhudson> mtaylor: in what way?
[16:50] <mtaylor> I've got three up: https://code.edge.launchpad.net/swift/+activereviews (from pipeline, depend on each other) and the unmerged revision list for each is off
[16:50] <mtaylor> as is the diff that was emailed and the one that is displayed in the UI
[16:51] <mtaylor> mwhudson: if you don't have super-launchpad perms, I'll have to add you to a team so you can see it
[16:51] <mwhudson> mtaylor: i don't have the perms any more
[16:51] <mtaylor> mwhudson: ok. lemme add you real quick
[16:52] <mtaylor> mwhudson: if you look at https://code.edge.launchpad.net/~mordred/swift/add-doc-package/+merge/29896
[16:52] <mtaylor> mwhudson: that merge should contain revs 16 and 17 - but even if it just contained rev 17 the diff still should not be empty
[16:52] <mwhudson> mtaylor: bah, i can't see that
[16:52] <mwhudson> Unauthorized: (<Branch u'~mordred/swift/add-doc-package' (361880)>, 'landing_targets', 'launchpad.View')<br />
[16:53] <mtaylor> hrm. lemme add you somewhere else...
[16:53] <mtaylor> mwhudson: try again
[16:53] <mwhudson> success
[16:54] <mtaylor> woot
[16:55] <mwhudson> mtaylor: i think the unmerged revisions list being too big has been like that always
[16:55] <mtaylor> mwhudson: no - that's not the problem ...
[16:55] <mtaylor> mwhudson: sorry, I didn't mean unmerged revs...
[16:56] <mtaylor> mwhudson: if you look at the grey list of revs under "Add a review or comment" ... it should be listing 16 as well... and then the diff should be quite different/larger
[16:56] <mwhudson> mtaylor: thumper agrees that it's ****ed
[16:56] <mtaylor> yay
[16:56] <mwhudson> (he's now sitting next to me)
[16:56]  * mtaylor cries a bit
[16:57]  * mtaylor was going to teach new people how to work with merge requests and why they're wonderful today... 
[16:57] <mtaylor> is it possible that is's ****ed because of all the private nonsense?
[16:59] <mwhudson> mtaylor: trying to reproduce something
[16:59] <mwhudson> mtaylor: very unlikely
[16:59] <mtaylor> ok
[17:00] <mwhudson> mtaylor: can you delete the problem proposal and try again?
[17:00] <mtaylor> mwhudson: sure
[17:00] <mtaylor> mwhudson: there are two problematic ones... shall I delete bothj?
[17:01] <mwhudson> mtaylor: which is the other problem one?
[17:01] <mwhudson> but no, just delete one for now
[17:03] <mtaylor> https://code.edge.launchpad.net/~mordred/swift/rework-dh-install
[17:03] <mtaylor> ok
[17:04] <mwhudson> mtaylor: yeah, the diff on https://code.edge.launchpad.net/~mordred/swift/rework-dh-install/+merge/29891 is just the diff from r13 isn't it?
[17:05] <thumper> mtaylor: hi
[17:05] <thumper> mtaylor: I've opened my laptop again
[17:05] <mwhudson> mtaylor: same empty diff again
[17:05] <mtaylor> thumper: hi!
[17:05] <mtaylor> mwhudson: bleh
[17:05] <mwhudson> mtaylor: yes
[17:07] <thumper> mtaylor: I'm sitting next to abentley discussing your diff
[17:07] <thumper> or lack thereof
[17:07] <mtaylor> thumper: did I break somethignm
[17:07] <mtaylor> ?
[17:07] <thumper> mtaylor: maybe
[17:07] <thumper> mtaylor: I seem to recall something like this happening before
[17:07] <thumper> but I don't remember why
[17:07] <thumper> or what we did about it
[17:09] <mtaylor> yay!
[17:09] <mtaylor> thumper: I did push the first branch (~mordred/swift/bzr-builddeb) ... then I reconfigured to pipeline
[17:09] <mtaylor> then I re-pushed it
[17:10] <mtaylor> not sure if it's relevant
[17:10] <mwhudson> mtaylor: you should probably remove me from those teams
[17:12] <mtaylor> mwhudson: I will
[17:13] <mtaylor> mwhudson: do you not need it to see things anymore?
[17:13] <mwhudson> mtaylor: i'm letting thumper and abentley worry about it :-)
[17:13] <mtaylor> mwhudson: cool
[17:13] <thumper> mtaylor: bad news
[17:13] <thumper> mtaylor: when we manually do what the script should be doing
[17:13] <thumper> we get a diff
[17:13] <thumper> so...
[17:13] <thumper> sounds like a bug in our diff generation
[17:14] <mtaylor> yay!
[17:14] <thumper> please demonstrate with a different project
[17:14] <mtaylor> thumper: should I try deleting an recreating all three merge props?
[17:14] <mtaylor> mwhudson: done
[17:14] <mwhudson> mtaylor: cool
[17:15] <thumper> mtaylor: you could try adding a rev to the source branch of the one with the bad diff
[17:15] <thumper> this would force a new diff to be generated
[17:15] <thumper> alternatively, try merging the prerequisite into the source
[17:15] <thumper> if the tip isn't already
[17:15] <thumper> it may be an edge case
[19:58] <james_w> no ec2test mail makes james_w a sad panda
[19:58] <james_w> phew, there it is, just really slow