[00:29] <wallyworld_> just saw that steve jobs has died
[00:29]  * StevenK adds 1 to his internal number of channels that have mentioned it.
[01:37] <wallyworld_> lifeless: did skype drop out or is it just me?
[01:38] <StevenK> wgrant: Can haz another look at my MP?
[01:46] <wallyworld_> thumper: ping!
[01:46] <lifeless> wgrant: sinzui: did skype hiccup ?
[01:49] <wgrant> Oh :(
[01:49] <wgrant> So yeah, I think we know what we're doing.
[01:49] <wgrant> Almost.
[01:50] <wgrant> Multipillar bugs with disclosure views are still a bit of an issue.
[01:50] <wgrant> But I think we have better ideas now.
[01:50] <wgrant> Email probably works.
[01:50] <wgrant> I need to experiment with how they interact with policies.
[01:50] <wgrant> They may make it easier.
[01:50] <sinzui> wgrant, yes. they do need more work, but I feel we have a path to solve the issue
[01:50] <wgrant> I am tempted to go with policies now and then see how multipillar bugs fall out.
[01:51] <sinzui> wgrant, I was thinking of adding a report task to the kanban. How many pillars share private bugs?
[01:51] <wgrant> sinzui: I was running reports on Tuesday to work that out.
[01:51] <wgrant> sinzui: Using all historical data, too.
[01:52] <wgrant> sinzui: (any bug that is private now, or has ever had the visibility changed)
[01:52] <wgrant> sinzui: I didn't end up finishing them, but I can finish the queries if you do want to know this.
[01:52] <sinzui> oh, I had not considered that
[01:52] <sinzui> I think we should know this because I am certain someone will ask us
[01:54] <wgrant> sinzui: It is relevant because many security bugs end up public.
[01:56] <wgrant> sinzui, lifeless: I have a set of scripts to populate an empty DB.
[01:56] <wgrant> With celebrities.
[01:56] <wgrant> And an initial user.
[01:57] <sinzui> oh sweet
[01:57] <wgrant> And then another one to create Ubuntu and set it up for Soyuz.
[01:57] <wgrant> I wrote them around two years ago, but updated them a few months ago.
[01:57] <wgrant> https://code.launchpad.net/~wgrant/launchpad/bootstrap-db-from-scratch\
[01:58] <sinzui> mailing lists require a script since it is not 100% in the db
[01:59] <wgrant> http://bazaar.launchpad.net/~wgrant/launchpad/bootstrap-db-from-scratch/revision/10987 is the guts of it.
[01:59] <wgrant> karma and celebrities are the ugly bits.
[01:59] <wgrant> No.
[02:00] <wgrant> That would be nice, but slow and require lots of test fixes.
[02:00] <wgrant> right.
[02:01] <wgrant> These create a minimal DB that you can use a harness on to create more stuff.
[02:04] <wgrant> wallyworld_, lifeless, StevenK: eg. I have my base bootstrap-lp-db script which just creates minimal celebrities and a user for the caller, then make-ubuntu-sane to create sensible Ubuntu series. I imagine there would be lots of scripts like make-ubuntu-sane.
[02:06] <wgrant> We use make initscript-start on production.
[02:06] <wgrant> Not make run.
[02:12] <wgrant> *Especially* using the factory.
[02:18] <StevenK> wgrant: Can haz review?
[02:51] <StevenK> wgrant: bootstrap-lp-db was what lifeless was talking about it, which sounds like a good idea to me.
[02:51] <wgrant> Not quite what lifeless was talking about.
[02:51] <wgrant> I believe lifeless was talking about a script to generate something like the current sampledata.
[02:51] <wgrant> With bugs and projects.
[02:51] <wgrant> But not with SQL.
[02:52] <lifeless> yes
[02:53] <StevenK> wgrant: I was hoping my refactoring could remove more lines than it added, but alas.
[02:57] <thumper> wallyworld_: pong
[02:58] <wallyworld_> thumper: that utf-8 branch has a test failure on ec2 but not locally and my encoding foo is not that great https://pastebin.canonical.com/53926/
[02:58]  * thumper looks
[02:59] <wallyworld_> thumper: i'll paste the code snippet
[02:59] <wallyworld_> thumper: https://pastebin.canonical.com/53927/
[03:00] <thumper> and it works locally?
[03:00] <wallyworld_> the string 'hello \xce\xa3' is the correct non-unicode representation of the unicode string
[03:00] <wallyworld_> yes
[03:01] <wallyworld_> in the debugger, both the unicode and non-unicode versions appear correctly as hello (sigma)
[03:01] <wallyworld_> so i know the strings are correct
[03:01] <thumper> damn
[03:01]  * thumper has no idea
[03:01]  * wallyworld_ also has no idea
[03:02] <thumper> wallyworld_: ask someone who uses utf-8 more, like jtv or jelmer :)
[03:02] <wallyworld_> will do
[03:16] <wgrant> lifeless: http://paste.ubuntu.com/634326/
[03:23]  * StevenK prods wgrant about his MP so he can land it
[03:25] <wgrant> StevenK: Much better, thanks.
[03:26] <nigelb> Hrm, no stub yet :(
[03:26] <StevenK> stub has been awake for hours.
[03:26] <StevenK> wgrant: Thanks! Tossing it at ec2.
[03:27] <nigelb> StevenK: I managed to screw up my MP enough that I had to redo it :D
[03:27] <nigelb> So, now I need approval again :(
[03:28] <StevenK> nigelb: Been there, done that.
[03:28] <nigelb> StevenK: That's comforting :)
[03:35] <stub> nigelb: Where is the mp?
[03:38] <nigelb> stub: Nevermind. I just noticed you approved it.
[03:39] <nigelb> I must've lost the email :(
[03:42] <nigelb> lifeless: ohai, I see that you're OCR today, could you land something for me? :)
[03:42] <lifeless> nigelb: maybe; I'm EODish
[03:43] <StevenK> Point me at it
[03:46] <nigelb> lifeless: ah
[03:46] <nigelb> StevenK: https://code.launchpad.net/~nigelbabu/launchpad/create-description-5283/+merge/78220
[03:46] <nigelb> lifeless: It Thu that you start really early and end early?
[03:56] <StevenK> nigelb: Is there a bug for that work?
[03:57] <nigelb> StevenK: ah, right. Let me link.
[03:57] <StevenK> It's just a db patch, so it doesn't matter much
[03:57] <nigelb> StevenK: (In case it isn't apparent 5283)
[03:58] <nigelb> oh no. Someone removed that feature
[03:58] <wgrant> Not quite.
[03:58] <nigelb> when linking a bug it would suggest the number that's in the branch name
[03:58] <wgrant> It only works with numbers of 5 or more digits.
[03:58] <nigelb> bah!
[03:58] <wgrant> To minimise false-positives.
[03:58] <StevenK> lol
[03:58] <nigelb> StevenK: linked :)
[03:59] <nigelb> I'm going to change the dev wiki db patch page to say the right year.
[04:02] <StevenK> nigelb: lp-landed to db-devel
[04:03] <StevenK> r11051 of db-devel
[04:03] <nigelb> StevenK: that was fast.
[04:03] <nigelb> You didn't need a test run?
[04:04] <StevenK> It's a DB patch!
[04:04] <nigelb> heh
[04:04] <StevenK> What could possibly go wrong, etc, etc
[04:04] <nigelb> Famous last words etc
[04:12] <wallyworld_> StevenK: i had a db-patch fail last week. it got picked up by ec2 luckily. it was a combination of not null column and a trigger processing issue. so it depends on the nature of the change whether it should go through ec2
[04:14] <nigelb> wallyworld_: Its a new column, probably less risk :)
[04:14] <wallyworld_> nigelb: understood. i was just making a point about db patches sometimes needing ec2 :-)
[04:15] <lifeless> nigelb: earlier , not really early atm
[04:15] <lifeless> nigelb: but ye
[04:15] <lifeless> s
[04:16] <lifeless> wallyworld_: everything gets ec2 by default :)
[04:16] <wallyworld_> lifeless: yes. i was trying to be gentle :-)
[04:16] <wallyworld_> with my assertion that ec2 be used
[04:23] <StevenK> wgrant: So, are you happy enough for populate-bprc to land, and if it starts being crap, we change the query?
[04:28] <wgrant> StevenK: I'd prefer if we knew it wasn't crap, and that it was the way we wanted to go.
[04:29] <StevenK> wgrant: My timing of the query was 'acceptable', TBH
[04:29] <StevenK> And it didn't kill DF
[04:36] <StevenK> wgrant: So given the two choices of "It will make me sad" or "I won't abide, and will revert it." is it? :-)
[04:39] <wgrant> StevenK: Well, we need to decide whether we want it in our main DB right now.
[04:39] <wgrant> And whether the query doesn't suck.
[04:40] <StevenK> I thought we had these arguments already?
[04:45] <StevenK> 51 polls have been created this year. I am disappoint.
[04:46] <lifeless> StevenK: land
[04:46] <nigelb> StevenK: Kill it :D
[04:46] <StevenK> lifeless: land populate-bprc?
[05:27] <nigelb> StevenK: :(
[05:27] <nigelb> buildbot failed.
[05:30]  * StevenK waits for SSO
[05:34] <StevenK> Huh, weird.
[05:34] <StevenK> lifeless: land populate-bprc?6170 tests
[05:34] <StevenK> 1334 passed
[05:34] <StevenK> 76 failures
[05:34]  * StevenK waits for SSO[B[B
[05:34]  * StevenK glares at his mouse
[05:34]  * wgrant glares at StevenK.
[05:35] <wgrant> Ah, I see the buildbot failure is noticed already.
[05:35]  * wgrant waits for SSO.
[05:35] <wgrant> Heh, you broke all the triggers.
[05:35] <StevenK> Ah. Shall I revert?
[05:37] <wgrant> Yes.
[05:37] <wgrant> And ec2 things in future...
[05:37] <StevenK> Doing so.
[05:42] <nigelb> What'd I break?
[05:42] <StevenK> All of the triggers.
[05:42] <nigelb> Ugh.
[05:42] <nigelb> How?
[05:42] <nigelb> Or rather, how do I fix.
[05:43] <StevenK> wgrant: Reversion tossed at PQM.
[05:45]  * StevenK breaks PQM into tiny pieces and resubmits
[05:46] <wgrant> nigelb: The lp_person table and the lp_mirror_person_* triggers in trusted.sql. You need to replace them in a patch, like you did with bugtask.statusexplanation's FTI trigger.
[05:46] <wgrant> Wow.
[05:46] <wgrant> They depend on column order.
[05:46] <wgrant> stub: Make them go away :(
[05:46] <StevenK> Haha
[05:46] <StevenK> lp_person and lp_mirror_person_* are parasites.
[05:47] <wgrant> Actually.
[05:47] <wgrant> We rerun trusted.sql each time.
[05:47] <wgrant> So you can probably just edit the functions in-place.
[05:47] <StevenK> wgrant: Revert is r11053
[05:48] <StevenK> wgrant: It has CREATE OR REPLACE TRIGGER?
[05:48] <stub> yes, they can be edited in place. But we can't alter the lp_* table definitions.
[05:48] <stub> They will go away when ISD obtains that information through other channels, or decides it isn't needed at all.
[05:50] <wgrant> But they've got no motivation to fix this :/
[05:50] <stub> nigelb: How is your PL/pgSQL? I might need to take over that patch.
[05:50] <stub> They do, as their schema updates are a pain in the arse because their databases are tied into ours.
[05:54] <nigelb> stub: I suck :)
[05:55] <nigelb> stub: I'm glad to defer to you :)
[05:55] <nigelb> Hrm, awkward timing for that quit.
[05:55] <nigelb> :P
[05:56] <nigelb> wgrant / StevenK - I seem to be opening quite a can of worms lately :)
[05:56] <stub> bouncy bouncy
[05:57] <nigelb> stub: In case you missed that, please take over :)
[05:57] <stub> nigelb: ok
[07:08] <nigelb> stub: hey, can you let me know once its done, so I can work on the follow up branches? :)
[07:53] <lifeless> whats the status of lazr-js ?
[07:56] <wgrant> Merged into LP and abandoned.
[07:56] <wgrant> But I think launchpad-results still uses a tiny bit of it.
[07:58] <lifeless> wondering whether to mass close its bugs
[07:58] <lifeless> or move them to lp
[07:58] <lifeless> or ...
[07:58] <wgrant> Ignore them forever :)
[07:59] <lifeless> jelmer: bug 515918 - do we close it ?
[07:59] <_mup_> Bug #515918: package_name and suite not passed to dailydeb <lp-soyuz> <recipe> <Launchpad itself:Triaged> < https://launchpad.net/bugs/515918 >
[07:59] <nigelb> wgrant: I think those numbers are lying.
[08:00] <wgrant> nigelb: Which?
[08:00] <nigelb> wgrant: /topic
[08:00] <nigelb> Mainly because it doesn't account for the fact that new bugs are opened and old ones are closed.
[08:01] <wgrant> Well, it depends what you want to measure.
[08:01] <lifeless> nigelb: its not reporting rate
[08:01] <nigelb> lifeless: shouldn't it?
[08:02] <nigelb> Or should it be a fraction with closed numbers as well to give a comparison.
[08:02] <nigelb> Now it looks like nothing's happening :P
[08:02] <lifeless> nigelb: its not, and never has been, a fraction
[08:02] <lifeless> nigelb: perhaps I'm missing your point
[08:03] <wgrant> Hm, only 57 launchpad-project criticals from before this year.
[08:03] <nigelb> lifeless: Well, wht I'm trying to say is, well, it looks like work isn't being done. Which is wrong.
[08:03] <wgrant> Work is being done, but progress is not being made.
[08:03] <wgrant> The intention is to drive the critical queue to 0.
[08:03] <wgrant> That's what that graph measures.
[08:04] <nigelb> Ok, that explains it :)
[08:04] <wgrant> (it was ~250 when it started)
[08:04] <wgrant> Actually, started at 210.
[08:05] <nigelb> I should agree to take a pie to the face at spring UDS.
[08:05] <nigelb> Seeing the pace at which its going I may never have to :D
[08:06] <wgrant> Yep :)
[08:06] <wgrant> Also, Launchpadders aren't really at UDS any more, so there'll hardly be anyone to pie you :P
[08:06] <nigelb> I thought there's always a small representation?
[08:06] <nigelb> Anyway, there's ex-launchpadders in plenty.
[08:07] <nigelb> I'm sure joey or kiko or jml would be happy to have the pleasure :P
[08:08] <jml> wgrant: that's great (the only 57)
[08:08] <jml> wgrant: although, huh, we're good at adding critical bugs, it seems
[08:09] <wgrant> jml: I thought it would be well over 100.
[08:09] <wgrant> But apparently not.
[08:09] <wgrant> jml: Note that lots are old bugs, like timeouts that only appeared as we dropped the timeout.
[08:10] <wgrant> There are 81 timeout bugs, and I would be surprised if even 10 of them were actually new bugs.
[08:11] <jml> ahh right.
[08:11] <jml> what's the problem with those? are they unusually hard to fix?
[08:11] <wgrant> But I believe flacoste is currently analysing this sort of thing.
[08:11] <wgrant> No.
[08:12] <wgrant> I think the maintenance squads are suffering from a combination of taking too many big tasks at once, and possibly having too many people leaving :)
[08:12] <nigelb> leaving?
[08:15] <lifeless> jml: wgrant: the remaining old timeouts are hard to fix
[08:15] <lifeless> we nabbed most of the low hanging fruit already
[08:16] <wgrant> lifeless: Some of them are, yes. But lots are not.
[08:59] <lifeless> wgrant: can you parse bug 397165 for me
[08:59] <_mup_> Bug #397165: Upload queue changes should be more restricted <lp-soyuz> <soyuz-upload> <Launchpad itself:Triaged> < https://launchpad.net/bugs/397165 >
[09:06] <wgrant> lifeless: The web UI is pretty good (I think everything matches what's there, except possibly that Rejected->Accepted might not be exposed), but the internal API and command-line tools allow many transitions that should not be permitted.
[09:12] <lifeless> bug 405805 might qualify as critical even
[09:12] <_mup_> Bug #405805: Upload processing may reach a transaction deadlock when closing bugs <lp-soyuz> <soyuz-upload> <Launchpad itself:Triaged> < https://launchpad.net/bugs/405805 >
[09:14] <wgrant> lifeless: The upload processor doesn't really do transactions.
[09:15] <wgrant> For example, it's the reason "zopeless" mail isn't queued until the end of the transaction.
[09:26]  * StevenK tries to figure out the xx-ppa-workflow.txt error
[09:30] <StevenK> I think the test is amusing the buggy case of team renaming.
[09:49] <stub> Does a branch rolling back a rollback get a rollback= stanza in the commit message?
[09:50] <lifeless> no
[09:50] <lifeless> the rollback stanza is for the qa tagger
[09:50] <lifeless> to tell it that it fixes the bad-revision-xxxx
[09:51] <lifeless> so the range of revisions that can't be deployed is known
[09:55] <wgrant> We really need to fix the branch scanner.
[09:56] <wgrant> Needs to be faster, probably needs to do partial progressive scans of fresh branches (like we do with codeimports), and needs to destroy branchrevision :(
[10:13] <jelmer> wgrant: progressive scans would be a good start
[10:14] <wgrant> It can fortunately be done manually, but it's awkward and slow.
[11:35] <wallyworld_> jelmer: are you free to help me with a unicode/utf-8 problem?
[11:36] <jelmer> wallyworld_: hi
[11:36] <jelmer> wallyworld_: sure, what's up?
[11:37] <wallyworld_> hi. i'm not an expert sadly
[11:37] <wallyworld_> here's the issue - i have changed mp and branch emails to utf-8 encode the diff
[11:37] <wallyworld_> a test fails on ec2 but passes locally
[11:37] <wallyworld_> i'll pastebin the test, just a sec
[11:38] <wallyworld_> the test: https://pastebin.canonical.com/53927/, the failure: https://pastebin.canonical.com/53926/
[11:38] <wallyworld_> i'm not sure why is passes locally and fails on ec2
[11:39] <wallyworld_> the strings are correct - in my debugger, both the unicode and utf-8 versions are correctly represented
[11:40] <jelmer> wallyworld_: have you tried running locally with LC_ALL=C ?
[11:40] <wallyworld_> jelmer: what does that option do?
[11:43] <jelmer> wallyworld_: It's an environment variable
[11:43] <wallyworld_> sure :-) what does it affect?
[11:44] <jelmer> wallyworld_: It's the variable with the current language and encoding
[11:45] <wallyworld_> ok. and i assume that's what ec2 uses?
[11:45] <jelmer> wallyworld_: it influences, among other things, how bzr will encode things for the terminal
[11:45] <wallyworld_> the environment we run under ec2 i mean
[11:45] <wallyworld_> ok
[11:45] <jelmer> wallyworld_: We've seen some issues related to this when bzr is run with LC_ALL=C (ascii only) versus LC_ALL=en_AU.UTF-8 (UTF-8)
[11:46] <wallyworld_> ok. i'm trying it now
[11:51] <wallyworld_> jelmer: it still passes locally
[11:52] <jelmer> wallyworld_: the odd thing appears to be that you specify decode=True (which I assume means "return unicode") but you're getting back a plain string
[11:53] <wallyworld_> jelmer: decode=True tells the Python email library to reverse any encoding used when the message was constructed
[11:53] <wallyworld_> yes, wonder why a plain string is returned
[11:54] <wallyworld_> i'll have to dig around the email libs a bit i guess
[11:55] <wallyworld_> jelmer: btw, i only modified that test. the test used to only use non-unicode strings and i added the u'hello ...' to test the utf-8 encoding
[11:56] <wallyworld_> thanks for the help btw
[12:00] <jelmer> wallyworld_: Sorry, still digging..
[12:01] <wallyworld_> oh ok. thanks!. i've fired up the debugger and am looking to see if i missed anything
[12:01] <jelmer> wallyworld_: do you understand what the :3 comes from?
[12:02] <jelmer> I mean, not that the result is different, but why does it come up with ":3" ?
[12:02] <wallyworld_> jelmer: the unicode string i used is u'hello (sigma)' where (sigma) is the greek letter
[12:02] <wallyworld_> i guess that's what the decoded string comes out as on ec2
[12:03] <jelmer> but why would sigma become :3 ?
[12:03] <wallyworld_> i wish i knew
[12:03] <jelmer> It would be more understandable if it replaced it with \uffff or something
[12:03] <wallyworld_> yeah. locally, both strings print as the correct thing
[12:04] <wallyworld_> by "print", the debugger does a str() on them any they display correctly
[12:15] <cjwatson> anyone want to comment on the thoughts I posted in bug 572128?
[12:15] <_mup_> Bug #572128: Ubuntu Archive translations are missing Index metadata file <Launchpad itself:New> <debmirror (Ubuntu):Confirmed> < https://launchpad.net/bugs/572128 >
[12:18] <danilos> jelmer, wallyworld_: locale stuff can also support transcription to ASCII, though you usually have to explicitely ask for it
[12:19] <wallyworld_> danilos: i can't correlate what's happening here though. why would it pass locally and fail on ec2?
[12:19] <danilos> though, this doesn't sound like a reasonable transcription
[12:20] <danilos> wallyworld_, I would attribute it to ec2 developing feelings towards you (iow, sounds weird, yes :))
[12:20] <wallyworld_> i have a unicode string u'hello Σ' which is converted to utf-8, encoded as quoted-principal, decoded, and compared
[12:20] <wallyworld_> all the encoding etc happens in the python email libs
[12:21] <wallyworld_> i'm at a loss
[12:21] <wallyworld_> danilos: i have feelings for ec2 as well but it's not love
[12:21] <nigelb> heh
[12:22] <danilos> wallyworld_, judging from how it's working out for you, I'd say it's mutual :P anyway, I did see some differences between python on ec2 image and recent releases; have you had someone try it out locally on lucid as well?
[12:23] <wallyworld_> danilos: ah no. but that's a good idea, thanks. something environmental sounds like a reasonable culprit. and if it is, i wonder how to solve it
[12:24] <danilos> wallyworld_, "unset LANG LC_ALL" locally?
[12:24] <wallyworld_> i guess get it to fail first and go from there :-)
[12:24] <wallyworld_> danilos: jelmer suggested LC_ALL=C which i tried
[12:25] <wallyworld_> i'll try unset
[12:25] <danilos> wallyworld_, check with "locale" what the resulting settings are (sometimes LANG can override stuff on GNU systems)
[12:25] <cjwatson> LANG never overrides LC_ALL with GNU software
[12:25] <cjwatson> some crappy non-GNU software gets it wrong though :)
[12:26] <StevenK> I can never remember the order
[12:26] <wallyworld_> LANG=en_AU.UTF-8 for me
[12:26] <StevenK> LC_ALL LANG LANGUAGE ?
[12:26] <danilos> cjwatson, ah, ok, so how about LANGUAGE? I know there's a specific GNU extension
[12:26] <cjwatson> LANGUAGE > LC_ALL > LC_* > LANG
[12:26] <cjwatson> although only for the purposes of message categories; for all other locale categories, LC_ALL > LC_* > LANG
[12:27] <wallyworld_> with all those unset, it still works locally
[12:27] <danilos> wallyworld_, just trying out your test gives me https://pastebin.canonical.com/53948/, but I guess that's a bug you are fixing :)
[12:27]  * wallyworld_ cries
[12:27] <cjwatson> perhaps EC2 has a non-existent locale set in its environment or something
[12:28] <wallyworld_> could be, i'll have to look at how our images are generated
[12:28] <StevenK> ec2test calls os.environ.pop() on LANG, LC_ALL and LC_TIME
[12:28] <wallyworld_> StevenK: danilos suggested it may be a lucid thing
[12:29]  * wallyworld_ needs to reinstall lxc again
[13:01] <deryck> Morning, all.
[13:01] <rvba> Hi jcsackett!
[13:01] <rvba> Morning deryck.
[13:02] <jcsackett> morning rbva. :-)
[13:02] <jcsackett> er, rvba. :-P
[13:53] <jcsackett> sinzui: we're good to land the branch that kills the privacy banner feature flag now, right?
[13:53] <sinzui> yes
[13:53] <jcsackett> excellent.
[13:54] <jcsackett> out it goes.
[14:15] <flacoste> gary_poster, benji: (not urgent) can one of you have a look at https://code.launchpad.net/~mwhudson/launchpad/permit_timeout_from_features-on-participation-bug-861510/+merge/78355
[14:16] <benji> flacoste: I will momentarily.
[14:16] <gary_poster> thanks benji
[14:16] <benji> np
[14:16] <gary_poster> jcsackett I am finished with my regular morning calls, and can talk when you are ready.
[14:17] <gary_poster> no rush
[14:17] <jcsackett> gary_poster: awesome. i am actually free now, if you like. potentially this will be a short conversation. :-P
[14:17] <jcsackett> mumble work for you?
[14:17] <gary_poster> heh ok cool jcsackett.  skype usually works better, but I can do the mumble thing.  1 sec and I'll get on
[14:39] <danilos> mrevell, btw, any idea how did this break: bug 826634?
[14:43] <mrevell> bug 826634
[14:43] <mrevell> Come on robot, give me a link
[14:44] <mrevell> danilos, Crumbs. I've no idea
[14:46] <bigjools> mrevell: amazing that nobody asked for wikis on your survey
[14:46] <danilos> mrevell, oh, so the help files are on translations.launchpad.net and the links point to launchpad.net
[14:50] <mrevell> danilos, Oh, strange
[14:51] <mrevell> bigjools, Yeah, well, it was only a handful of people replied. I bet if we asked more people then wikis would come up.
[14:53] <cjwatson> is there a way to stop /var/tmp/archive/ from being cleaned up after failing archivepublisher tests?
[14:55] <bigjools> cjwatson: post-mortem debug
[14:55] <cjwatson> as in pdb?  ok
[14:55] <bigjools> yeah, there's a test runner option
[14:55] <bigjools> or you can remove the cleanup code
[15:35] <sinzui> jcsackett, mumble?
[15:37] <jcsackett> sinzui: sure, one moment.
[15:39] <benji> flacoste: I don't know if you want to engage further but FYI: I just added a comment to https://code.launchpad.net/~mwhudson/launchpad/permit_timeout_from_features-on-participation-bug-861510/+merge/78355 (didn't feel good about approving it, so I just commented)
[15:40] <flacoste> benji: ok, thanks
[16:03] <gary_poster> bigjools, if you have a moment to look at https://bugs.launchpad.net/launchpad/+bug/869185 I'd appreciate it.  Please triage it, or if it is faster to tell me what to do about it, do that instead and I'll follow up.
[16:03] <_mup_> Bug #869185: P-a-s file ignored (even on cocoplum) <Launchpad itself:New> < https://launchpad.net/bugs/869185 >
[16:05] <bigjools> gary_poster: ok
[16:05] <gary_poster> thank you much
[16:05] <bigjools> gary_poster: gah it's a mix of more than one bug
[16:05] <gary_poster> oh :-/
[16:06] <bigjools> gary_poster: I can explain for your delectation and delight if you like
[16:06] <gary_poster> bigjools, I'm happy to listen, sure :-)
[16:06] <bigjools> do you know about the P-a-s file?
[16:06] <gary_poster> and learn even
[16:06] <gary_poster> no
[16:06] <bigjools> ok, it's used when creating builds for sources and acts as an override to whatever the source thinks it should build on
[16:07] <bigjools> so it can say !armel to exclude armel for example
[16:07] <gary_poster> ok
[16:07] <bigjools> the ubuntu guys rely on it so that they can take packages from Debian with no changes
[16:07] <gary_poster> I figured it was somehinng like that from context
[16:08] <bigjools> so bug #1 is that when we sync files from Debian using the new derived distros stuff, it doesn't use the P-a-s file at all
[16:08] <_mup_> Bug #1: Microsoft has a majority market share <iso-testing> <ubuntu> <Clubdistro:Confirmed> <Computer Science Ubuntu:Confirmed for compscibuntu-bugs> <dylan.NET.Reflection:Invalid> <dylan.NET:Invalid> <EasyPeasy Overview:Invalid by ramvi> <GenOS:In Progress by gen-os> <GNOME Screensaver:Won't Fix> <Ichthux:Invalid by raphink> <JAK LINUX:Invalid> <LibreOffice:In Progress by bjoern-michaelsen> <Linux:New> <Linux Mint:In Progress> <The Linux OS P
[16:08] <bigjools> argh go away mup
[16:08] <nigelb> heh
[16:08] <gary_poster> :-)
[16:09] <gary_poster> I se
[16:09] <gary_poster> e
[16:09] <bigjools> bug two (ha! screw you mup) is that it also didn't appear to work for a regular upload of the libx86 package
[16:09] <gary_poster> :-)
[16:09] <gary_poster> ah ok.  So the first is a non-critical feature bug (?) and the second is a critical regression?
[16:09] <bigjools> he's talking about cocoplum since that's where the ubuntu uploads go
[16:09] <gary_poster> gotcha
[16:10] <bigjools> so I need to split this into two
[16:10] <bigjools> bear with me
[16:10] <gary_poster> cool
[16:15] <bigjools> gary_poster: ok I updated the original bug a bit and reference the new one in it
[16:16]  * gary_poster looks
[16:17] <gary_poster> bigjools, why is first one (869185) not regression/critical?
[16:17] <bigjools> gary_poster: I was just thinking about that
[16:17] <bigjools> it's only a single package, hence my hesitation
[16:17] <gary_poster> right
[16:17]  * gary_poster leaves the decision to you :-D
[16:18] <bigjools> thanks :)
[16:18] <gary_poster> fwiw, based on your explanation, if you had left it to me I would have said critical
[16:19] <gary_poster> (on the basis of "the rules")
[16:19] <bigjools> rules are there to bend
[16:19] <gary_poster> :-) fair enough
[16:20] <bigjools> I am just finding out if it's more widespread, if so it's a critical regression
[16:20] <gary_poster> ok
[16:20] <bigjools> no idea *how* since nobody changed that code lately ...
[16:20] <cjwatson> I think I have a fix for bug 572128; would anyone care to have a *cough* post-implementation chat about it? :-)
[16:20] <_mup_> Bug #572128: Ubuntu Archive translations are missing Index metadata file <Launchpad itself:New> <debmirror (Ubuntu):Confirmed> < https://launchpad.net/bugs/572128 >
[16:20] <gary_poster> cjwatson I was just seeing if danilos were around to look at that bug
[16:21] <gary_poster> I don't think he is :-P
[16:21] <cjwatson> I posted general approach in the bug comments, and http://paste.ubuntu.com/703479/ appears to pass tests for me
[16:22] <bigjools> cjwatson: I'd love to but kinda busy, I can chat tomorrow if you don't find anyone before then
[16:22] <cjwatson> sure, that will be fine, pretty tired now anyway
[16:22] <bigjools> I hear ya
[16:23] <bigjools> cjwatson: since you're here, are you aware of any other failures to use P-a-s for uploaded packages?
[16:23] <gary_poster> cjwatson, from a pure triaging perspective, is this a regression
[16:23] <cjwatson> I'll assign myself in the meantime
[16:23] <gary_poster> (572128) on the LP side
[16:24] <cjwatson> gary_poster: yes, albeit one that's my fault
[16:24] <gary_poster> :-) ok
[16:24] <gary_poster> I'll triage as such.  thanks cjwatson
[16:24] <cjwatson> (and a regression by cooperation between LP and mirroring tools)
[16:25] <cjwatson> bigjools: I haven't *noticed* any, although that doesn't necessarily say much - doko is much better informed here than I am
[16:25] <cjwatson> due to his work on test rebuilding
[16:25] <bigjools> yeah, I was trying to reach him
[16:26] <cjwatson> TBH it's the kind of thing I habitually write off since historically it's been lost in the noise of real failures
[16:26] <bigjools> cjwatson: the other side of the coin - do you know any that are ok that are in p-a-s?
[16:26] <cjwatson> now that we're getting those down to reasonable levels ...
[16:27] <cjwatson> bigjools: grub2 was fine last time I uploaded it, although it also has an explicit Architecture line
[16:27] <bigjools> ok - I wonder if p-a-s has a fault itself for libx86 then
[16:27] <cjwatson> easy to check
[16:27] <cjwatson> %libx86: !armel !hppa !ia64 !m68k !mips !mipsel !powerpc !sh4 !sparc  # <sys/io.h>
[16:28] <cjwatson> looks right to me
[16:28] <bigjools> me too
[16:28] <bigjools> weird
[19:46] <cr3> benji: quick lazr.restful/wadllib question for you, does this error ring a bell: UnsupportedMediaTypeError: This resource doesn't define a representation for media type text/plain
[19:47] <benji> cr3: I don't think I've seen that one before but I would suspect there's an adapter you can provide that would make it work.
[19:47] <cr3> benji: it seems to happen when an object is returned with a 304
[19:49] <cr3> benji: the same api url, when returning a 200, works fine. but, when it has not been modified, that's when I get: https://pastebin.canonical.com/53985/
[19:49] <benji> cr3: that's weird; since a 304 can't have a body then there's no reason to be looking for a representation (or for it to have a content-type at all)
[19:50] <cr3> benji: indeed, the appserver says it's returning 0 bytes, the representation might be in the header though
[19:51] <benji> cr3: well, 304 is Not Modified so there shouldn't be any representation anywhere
[20:12] <jelmer> bug 369752
[20:12] <_mup_> Bug #369752: Don't allow branch type to be selected on branch add form <easy> <lp-code> <ui> <Launchpad itself:Triaged> < https://launchpad.net/bugs/369752 >
[22:15] <sinzui> jcsackett, mumble?
[22:34] <sinzui> wallyworld__, https://launchpad.net/launchpad/+milestone/3.0
[22:37] <wallyworld__> wgrant: here's the branch lp:~wallyworld/launchpad/utf8-encode-diffs-861979 and here's the test lp.code.mail.tests.test_branch.TestBranchMailerDiff.test_generateEmail_with_diff
[22:52] <wallyworld__> wgrant: https://pastebin.canonical.com/53948/
[23:38] <wgrant> wallyworld_: AssertionError: 'hello \xce\xa3' != 'hello :3'
[23:38] <wallyworld_> wgrant: yes, that's what ec2 says too
[23:38] <wallyworld_> so it's a lucid vs oneiric issue
[23:38] <wgrant> Yep.
[23:38]  * wgrant adds D
[23:38] <wgrant> -D
[23:39] <wallyworld_> bollocks
[23:39] <wallyworld_> i could force the encoding to base64 to see if that helps
[23:39] <wgrant> Content-Transfer-Encoding: quoted-printable
[23:39] <wgrant> hello =3A3
[23:39] <wgrant> Is that as expected?
[23:39] <wgrant> Apparently not.
[23:40]  * wallyworld__ sighs. another unity/compiz crash
[23:41] <wgrant> you don't just restart unity in that case?
[23:46] <wallyworld__> wgrant: it took out quassel as well this time
[23:47] <wallyworld__> everything went belly up
[23:47] <wgrant> Hah
[23:51] <wallyworld__> wgrant: i've pushed a change, can you try again?
[23:52] <wgrant> wallyworld__: Have you seen the hack at the top of sendmail.py?
[23:52] <wgrant> To force quoted-printable?
[23:52] <wgrant> I wonder if that might be relevant.
[23:52] <wgrant> Because set_payload is indeed writing =3A3, which is :3.
[23:53] <wallyworld__> wgrant: you mean in encodeOptimally()
[23:53] <wallyworld__> ?
[23:53] <wgrant> wallyworld__: No, at module load time.
[23:53] <wgrant> It overrides the default encoidng.
[23:53] <wgrant> del Charset.CHARSETS['utf-8']
[23:53] <wgrant> Charset.add_charset('utf-8', Charset.SHORTEST, Charset.QP, 'utf-8')
[23:53] <wgrant> Charset.add_alias('utf8', 'utf-8')
[23:54] <wallyworld__> oah, didn;t notice that
[23:54] <wallyworld__> well that sucks
[23:54] <wgrant> http://paste.ubuntu.com/703671/
[23:54] <wgrant> That is indeed relevant.
[23:54] <wgrant> set_payload actually crashes until you do that.
[23:55] <wgrant> Interestingly, in python2.7 it doesn't crash initially.
[23:55] <wgrant> It encodes UTF-8 with base64.
[23:56] <wallyworld__> wgrant: so on python 2.6, base64 encoding of utf-8 crashes?
[23:56] <wgrant> And with the charset hack installed, 2.7 encodes to =CE=A3
[23:56] <wgrant> wallyworld__: It seems so.
[23:57] <wallyworld__> but i just tried pythin 2.6 and base64 locally and it worked
[23:57] <wgrant> wallyworld__: I wonder if set_payload doesn't actually like unicode in 2.6.
[23:58] <wgrant> Perhaps it takes the charset that the payload is encoded in, not a charset in which the payload should be encoded.
[23:58] <wgrant> The docs are not clear.
[23:58] <wgrant>         if str(charset) != charset.get_output_charset():
[23:58] <wgrant>             self._payload = charset.body_encode(self._payload)
[23:58] <wgrant> I think the payload is meant to be pre-encoded.
[23:59] <wallyworld__> argh, ffs. another unity crash
[23:59] <wallyworld__> i don't think the payload is meant to be pre-encoded