/srv/irclogs.ubuntu.com/2014/06/24/#launchpad-dev.txt

=== wallyworld_ is now known as wallyworld
=== mwhudson is now known as mwhudson-
=== mwhudson is now known as mwhudson-bip
=== mwhudson-bip is now known as mwhudson
=== mwhudson is now known as mwhudson-bip
xnoxhttps://code.launchpad.net/~xnox/lazr.authentication/fix-wsgi-intercept-0.6/+merge/22428713:01
=== jtv_ is now known as jtv
xnoxhttps://code.launchpad.net/~xnox/lazr.restfulclient/py3/+merge/22217213:03
xnoxare all of above managed by tumble thing? ( tarmac?)13:03
wgrantxnox: The trunks of both lazr.authentication and lazr.restfulclient are manually managed, not by Tarmac or PQM.13:22
wgrantxnox: What's with the weird spacing changes in your wsgi_intercept branch for lazr.restfulclient?13:23
xnoxwgrant: hehe. wsgi_intercept 0.5.x -> 0.6.x series wraps tracebacks into it's own exception thus interractive raised expection textual representation is now different13:27
xnoxwgrant: it does look odd, should i be fixing wsgi_intercept to print stuff as it used to?13:28
xnoxthe 0.6.0 came out in Novermber 2013.13:28
wgrantxnox: I'd quite like to know why those spacing changes happen; they look a lot like a bug.13:30
xnoxwgrant: yeah, running without any ...., all headers are printed without space after colon13:31
wgrant                self.output.write(k + b':' + v + b"\n")13:32
wgrantThat sure looks buggy to me.13:32
xnoxyeah, that's wrong.13:32
wgrantTrivial fix, too, fortunately :)13:33
xnoxi'll upload that into debian & propose upstream....13:33
xnoxcan buildout use system-wide packages somehow ?13:33
wgrantxnox: It *can*, but we prefer not to at this point.13:33
wgrantWe currently support running Launchpad on lucid, precise and trusty.13:34
wgrantWe backport some packaged things like python-debian, but it's often more trouble than it's worth.13:34
wgrant(Particularly when it's a non-backward compatible change like this; we'd have to upgrade the package and Launchpad in lockstep)13:35
wgrantHm13:35
wgrantSo that header format isn't actually illegal. Whitespace isn't mandatory.13:35
wgrantBut I've never seen any non-terrible app not prefix the value with a single space.13:35
xnoxyeah, all dumps i've ever seen have a space.13:36
* xnox ponders if it is in-fact in the HTTP RFC or some such.13:36
wgrant" The field value MAY be preceded by any amount of LWS, though a single SP is preferred."13:36
cjwatsonbeat me to it13:36
cjwatsonwas literally just selecting that text :)13:37
wgrantHeh13:41
xnoxopened a merge proposal upstream, now making upload into debian.13:42
wgrantExcellent.13:43
wgrantThat'll save a lot of trouble with fixing tests in lp:launchpad too.13:43
wgrant"fixing"13:43
xnoxand all of openstack13:44
xnoxalthough they might not be doing doctests as much as launchpad does.13:44
xnoxwgrant: barry did review this one https://code.launchpad.net/~xnox/lazr.restfulclient/py3/+merge/222172 can you please merge it?13:57
xnoxor should barry do the merging?13:58
wgrantxnox: Doing.13:58
wgrantIf GitHub's JS stops hanging my Firefox...14:00
xnoxwgrant: you merge lazr on github and then mirror to launchpad? *giggle*14:00
wgrantHeh, no, was looking at your PR14:01
wgrantxnox: I note a potential problem. The new wsgi_intercept doesn't seem to have the zope.testbrowser or mechanize support that the old one does, and Launchpad uses that.14:01
wgrantSo it'll work for lazr.restfulclient, but not Launchpad itself.14:01
xnoxwgrant: horum. I'm working on python3-launchpadlib at the moment, so that should be fine. ev kind of have vetoed for me to go into python3 launchpad project =)))))))14:02
wgrantHeh14:03
wgrantIt's just a bit weird that we have to keep Launchpad using an old version, but I guess we'll live.14:03
wgrantxnox: Can you set a commit message on https://code.launchpad.net/~xnox/lazr.restfulclient/py3/+merge/222172?14:03
xnoxmeh, as long as there are no security issues....14:03
xnoxwgrant: done.14:04
wgrantI don't trust that library enough to let it near production :P14:04
wgrantIt's test-only.14:04
evxnox: hey now, I said you can spend your time however you like – I'm not your manager – but there were seemingly better places to invest that time in Launchpadland.14:06
xnoxev: yeah, rumour has it you are after python3-launchpadlib though ;-)14:07
wgrantDefinitely :)14:07
wgrantPorting launchpadlib is pretty tractable once the lazr.restful test dep is pushed out-of-process.14:07
wgrantlp:launchpad is just about impossible without splitting it up.14:07
xnoxev: for some sort of steam-punk engine14:08
ev:)14:08
wgrantxnox: Landed, thanks.14:09
wgrantPoke me if you need anything else.14:09
xnoxwgrant: my wsgi-intrcept fix was pulled upstream already =)14:11
wgrantNice.14:13
wgrantcjwatson: That launchpad-buildd ltsp change seems a bit weird. Is ltsp in the closure that ubuntu-rtm will have, for example?14:29
cjwatsonWe could make it i386-only in line with the old make-chroot.sh, I guess14:30
cjwatsonThough I suppose we'll still have i386 in rtm14:31
cjwatsonIt is weird, I just couldn't think of anything better, suggestions welcome14:31
cjwatsonwgrant: Is the CSV file that appears to be written by lp:~wgrant/lp-ftbfs-report/production available anywhere for http://qa.ubuntuwire.com/ftbfs/ ?  I can't make the obvious URLs work, and I assume the main page is a server-side redirect of some kind15:04
wgrantcjwatson: It's at the totally logical and obvious URL of http://people.ubuntuwire.org/~wgrant/rebuild-ftbfs-test/primary-utopic.csv15:06
cjwatsonAha, thanks.15:08
cjwatsonI was wondering about exposing data over time somehow15:09
xnoxwgrant: may i request review/merge of https://code.launchpad.net/~xnox/lazr.authentication/fix-wsgi-intercept-0.6/+merge/224287 ?! that one seems wsgi_intercept genuinely staying spec compliant and lazr.authentication was not.15:53
xnoxplease =)15:53
cjwatsonwgrant: Do you have any idea what the occasional build upload failures we see where the upload log just says "Server said:" are about?  I understand it's some kind of librarian glitch but not a lot more.  I've seen two so far with livefs builds, which is a fairly annoying rate17:54
cjwatson(particularly without retry)17:54
=== mwhudson-bip is now known as mwhudson
wgrantcjwatson: Hm, I guess it might be more common with livefses because they're bigger.23:06
wgrantIt's some librarian glitch which I've not been able to reproduce locally.23:06
wgrantWe should probably just fix it by making the client retry.23:06

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