| wallyworld_ | just saw that steve jobs has died | 00:29 |
|---|---|---|
| * StevenK adds 1 to his internal number of channels that have mentioned it. | 00:29 | |
| wallyworld_ | lifeless: did skype drop out or is it just me? | 01:37 |
| StevenK | wgrant: Can haz another look at my MP? | 01:38 |
| wallyworld_ | thumper: ping! | 01:46 |
| lifeless | wgrant: sinzui: did skype hiccup ? | 01:46 |
| wgrant | Oh :( | 01:49 |
| wgrant | So yeah, I think we know what we're doing. | 01:49 |
| wgrant | Almost. | 01:49 |
| 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:50 |
| 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:51 |
| 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:52 |
| wgrant | sinzui: It is relevant because many security bugs end up public. | 01:54 |
| 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:56 |
| 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:57 |
| sinzui | mailing lists require a script since it is not 100% in the db | 01:58 |
| 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. | 01:59 |
| wgrant | That would be nice, but slow and require lots of test fixes. | 02:00 |
| wgrant | right. | 02:00 |
| wgrant | These create a minimal DB that you can use a harness on to create more stuff. | 02:01 |
| 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:04 |
| wgrant | We use make initscript-start on production. | 02:06 |
| wgrant | Not make run. | 02:06 |
| wgrant | *Especially* using the factory. | 02:12 |
| StevenK | wgrant: Can haz review? | 02:18 |
| 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:51 |
| lifeless | yes | 02:52 |
| StevenK | wgrant: I was hoping my refactoring could remove more lines than it added, but alas. | 02:53 |
| thumper | wallyworld_: pong | 02:57 |
| 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:58 | |
| wallyworld_ | thumper: i'll paste the code snippet | 02:59 |
| wallyworld_ | thumper: https://pastebin.canonical.com/53927/ | 02:59 |
| 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:00 |
| 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:01 | |
| thumper | wallyworld_: ask someone who uses utf-8 more, like jtv or jelmer :) | 03:02 |
| wallyworld_ | will do | 03:02 |
| wgrant | lifeless: http://paste.ubuntu.com/634326/ | 03:16 |
| * StevenK prods wgrant about his MP so he can land it | 03:23 | |
| wgrant | StevenK: Much better, thanks. | 03:25 |
| nigelb | Hrm, no stub yet :( | 03:26 |
| StevenK | stub has been awake for hours. | 03:26 |
| StevenK | wgrant: Thanks! Tossing it at ec2. | 03:26 |
| 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:27 |
| StevenK | nigelb: Been there, done that. | 03:28 |
| nigelb | StevenK: That's comforting :) | 03:28 |
| stub | nigelb: Where is the mp? | 03:35 |
| nigelb | stub: Nevermind. I just noticed you approved it. | 03:38 |
| nigelb | I must've lost the email :( | 03:39 |
| 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:42 |
| StevenK | Point me at it | 03:43 |
| 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:46 |
| StevenK | nigelb: Is there a bug for that work? | 03:56 |
| 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:57 |
| 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:58 |
| nigelb | I'm going to change the dev wiki db patch page to say the right year. | 03:59 |
| StevenK | nigelb: lp-landed to db-devel | 04:02 |
| StevenK | r11051 of db-devel | 04:03 |
| nigelb | StevenK: that was fast. | 04:03 |
| nigelb | You didn't need a test run? | 04:03 |
| 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:04 |
| 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:12 |
| 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:14 |
| lifeless | nigelb: earlier , not really early atm | 04:15 |
| lifeless | nigelb: but ye | 04:15 |
| lifeless | s | 04:15 |
| 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:16 |
| StevenK | wgrant: So, are you happy enough for populate-bprc to land, and if it starts being crap, we change the query? | 04:23 |
| wgrant | StevenK: I'd prefer if we knew it wasn't crap, and that it was the way we wanted to go. | 04:28 |
| StevenK | wgrant: My timing of the query was 'acceptable', TBH | 04:29 |
| StevenK | And it didn't kill DF | 04:29 |
| 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:36 |
| 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:39 |
| StevenK | I thought we had these arguments already? | 04:40 |
| StevenK | 51 polls have been created this year. I am disappoint. | 04:45 |
| lifeless | StevenK: land | 04:46 |
| nigelb | StevenK: Kill it :D | 04:46 |
| StevenK | lifeless: land populate-bprc? | 04:46 |
| nigelb | StevenK: :( | 05:27 |
| nigelb | buildbot failed. | 05:27 |
| * StevenK waits for SSO | 05:30 | |
| 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:34 | |
| 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:35 |
| wgrant | Yes. | 05:37 |
| wgrant | And ec2 things in future... | 05:37 |
| StevenK | Doing so. | 05:37 |
| 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:42 |
| StevenK | wgrant: Reversion tossed at PQM. | 05:43 |
| * StevenK breaks PQM into tiny pieces and resubmits | 05:45 | |
| 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:46 |
| 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:47 |
| 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:48 |
| 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:50 |
| nigelb | stub: I suck :) | 05:54 |
| nigelb | stub: I'm glad to defer to you :) | 05:55 |
| nigelb | Hrm, awkward timing for that quit. | 05:55 |
| nigelb | :P | 05:55 |
| nigelb | wgrant / StevenK - I seem to be opening quite a can of worms lately :) | 05:56 |
| stub | bouncy bouncy | 05:56 |
| nigelb | stub: In case you missed that, please take over :) | 05:57 |
| stub | nigelb: ok | 05:57 |
| nigelb | stub: hey, can you let me know once its done, so I can work on the follow up branches? :) | 07:08 |
| === almaisan-away is now known as al-maisan | ||
| === wgrant changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: - | Critical bugs: 262 - 0:[#######=]:256 | ||
| === wgrant changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: - | Critical bugs: 262 - 0:[########BOOM | ||
| lifeless | whats the status of lazr-js ? | 07:53 |
| wgrant | Merged into LP and abandoned. | 07:56 |
| wgrant | But I think launchpad-results still uses a tiny bit of it. | 07:56 |
| 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:58 |
| 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. | 07:59 |
| 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:00 |
| 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:01 |
| 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:02 |
| 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:03 |
| nigelb | Ok, that explains it :) | 08:04 |
| wgrant | (it was ~250 when it started) | 08:04 |
| wgrant | Actually, started at 210. | 08:04 |
| 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:05 |
| 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:06 |
| nigelb | I'm sure joey or kiko or jml would be happy to have the pleasure :P | 08:07 |
| jml | wgrant: that's great (the only 57) | 08:08 |
| jml | wgrant: although, huh, we're good at adding critical bugs, it seems | 08:08 |
| 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:09 |
| wgrant | There are 81 timeout bugs, and I would be surprised if even 10 of them were actually new bugs. | 08:10 |
| 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:11 |
| 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:12 |
| === rvba changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: rvba* (allenap) | Critical bugs: 262 - 0:[########BOOM | ||
| lifeless | jml: wgrant: the remaining old timeouts are hard to fix | 08:15 |
| lifeless | we nabbed most of the low hanging fruit already | 08:15 |
| wgrant | lifeless: Some of them are, yes. But lots are not. | 08:16 |
| 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 > | 08:59 |
| 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:06 |
| 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:12 |
| wgrant | lifeless: The upload processor doesn't really do transactions. | 09:14 |
| wgrant | For example, it's the reason "zopeless" mail isn't queued until the end of the transaction. | 09:15 |
| * StevenK tries to figure out the xx-ppa-workflow.txt error | 09:26 | |
| StevenK | I think the test is amusing the buggy case of team renaming. | 09:30 |
| stub | Does a branch rolling back a rollback get a rollback= stanza in the commit message? | 09:49 |
| 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:50 |
| lifeless | so the range of revisions that can't be deployed is known | 09:51 |
| wgrant | We really need to fix the branch scanner. | 09:55 |
| 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 :( | 09:56 |
| jelmer | wgrant: progressive scans would be a good start | 10:13 |
| wgrant | It can fortunately be done manually, but it's awkward and slow. | 10:14 |
| === al-maisan is now known as almaisan-away | ||
| wallyworld_ | jelmer: are you free to help me with a unicode/utf-8 problem? | 11:35 |
| jelmer | wallyworld_: hi | 11:36 |
| jelmer | wallyworld_: sure, what's up? | 11:36 |
| 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:37 |
| 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:38 |
| wallyworld_ | the strings are correct - in my debugger, both the unicode and utf-8 versions are correctly represented | 11:39 |
| jelmer | wallyworld_: have you tried running locally with LC_ALL=C ? | 11:40 |
| wallyworld_ | jelmer: what does that option do? | 11:40 |
| jelmer | wallyworld_: It's an environment variable | 11:43 |
| wallyworld_ | sure :-) what does it affect? | 11:43 |
| jelmer | wallyworld_: It's the variable with the current language and encoding | 11:44 |
| 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:45 |
| wallyworld_ | ok. i'm trying it now | 11:46 |
| wallyworld_ | jelmer: it still passes locally | 11:51 |
| 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:52 |
| 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:53 |
| wallyworld_ | i'll have to dig around the email libs a bit i guess | 11:54 |
| 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:55 |
| wallyworld_ | thanks for the help btw | 11:56 |
| jelmer | wallyworld_: Sorry, still digging.. | 12:00 |
| 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:01 |
| 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:02 |
| 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:03 |
| wallyworld_ | by "print", the debugger does a str() on them any they display correctly | 12:04 |
| === almaisan-away is now known as al-maisan | ||
| 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:15 |
| danilos | jelmer, wallyworld_: locale stuff can also support transcription to ASCII, though you usually have to explicitely ask for it | 12:18 |
| 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:19 |
| 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:20 |
| 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:21 |
| 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:22 |
| 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:23 |
| 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:24 |
| 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:25 |
| 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:26 |
| 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:27 |
| 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:28 |
| * wallyworld_ needs to reinstall lxc again | 12:29 | |
| === jcsackett changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: rvba* (allenap), jcsackett | Critical bugs: 262 - 0:[########BOOM | ||
| deryck | Morning, all. | 13:01 |
| rvba | Hi jcsackett! | 13:01 |
| rvba | Morning deryck. | 13:01 |
| jcsackett | morning rbva. :-) | 13:02 |
| jcsackett | er, rvba. :-P | 13:02 |
| === al-maisan is now known as almaisan-away | ||
| 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:53 |
| jcsackett | out it goes. | 13:54 |
| 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:15 |
| 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:16 |
| 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:17 |
| danilos | mrevell, btw, any idea how did this break: bug 826634? | 14:39 |
| mrevell | bug 826634 | 14:43 |
| mrevell | Come on robot, give me a link | 14:43 |
| mrevell | danilos, Crumbs. I've no idea | 14:44 |
| 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:46 |
| mrevell | danilos, Oh, strange | 14:50 |
| 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:51 |
| cjwatson | is there a way to stop /var/tmp/archive/ from being cleaned up after failing archivepublisher tests? | 14:53 |
| 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 | 14:55 |
| sinzui | jcsackett, mumble? | 15:35 |
| jcsackett | sinzui: sure, one moment. | 15:37 |
| 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:39 |
| flacoste | benji: ok, thanks | 15:40 |
| === beuno is now known as beuno-lunch | ||
| === rvba changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: jcsackett | Critical bugs: 262 - 0:[########BOOM | ||
| 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:03 |
| 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:05 |
| 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:06 |
| 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:07 |
| 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 |
| === salgado is now known as salgado-lunch | ||
| gary_poster | :-) | 16:08 |
| 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:09 |
| bigjools | so I need to split this into two | 16:10 |
| bigjools | bear with me | 16:10 |
| gary_poster | cool | 16:10 |
| bigjools | gary_poster: ok I updated the original bug a bit and reference the new one in it | 16:15 |
| * gary_poster looks | 16:16 | |
| 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:17 | |
| bigjools | thanks :) | 16:18 |
| gary_poster | fwiw, based on your explanation, if you had left it to me I would have said critical | 16:18 |
| gary_poster | (on the basis of "the rules") | 16:19 |
| bigjools | rules are there to bend | 16:19 |
| gary_poster | :-) fair enough | 16:19 |
| 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:20 |
| 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:21 |
| 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:22 |
| 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:23 |
| 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:24 |
| 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 |
| === salgado is now known as salgado-lunch | ||
| cjwatson | due to his work on test rebuilding | 16:25 |
| bigjools | yeah, I was trying to reach him | 16:25 |
| 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:26 |
| 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:27 |
| cjwatson | looks right to me | 16:28 |
| bigjools | me too | 16:28 |
| bigjools | weird | 16:28 |
| === beuno-lunch is now known as beuno | ||
| === salgado-lunch is now known as salgado | ||
| === deryck is now known as deryck[lunch] | ||
| === deryck[lunch] is now known as deryck | ||
| 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:46 |
| 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:47 |
| 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:49 |
| cr3 | benji: indeed, the appserver says it's returning 0 bytes, the representation might be in the header though | 19:50 |
| benji | cr3: well, 304 is Not Modified so there shouldn't be any representation anywhere | 19:51 |
| 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 > | 20:12 |
| sinzui | jcsackett, mumble? | 22:15 |
| sinzui | wallyworld__, https://launchpad.net/launchpad/+milestone/3.0 | 22:34 |
| 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:37 |
| wallyworld__ | wgrant: https://pastebin.canonical.com/53948/ | 22:52 |
| === wallyworld_ changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: wallyworld | Critical bugs: 262 - 0:[########BOOM | ||
| 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:38 |
| 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:39 |
| * wallyworld__ sighs. another unity/compiz crash | 23:40 | |
| wgrant | you don't just restart unity in that case? | 23:41 |
| wallyworld__ | wgrant: it took out quassel as well this time | 23:46 |
| wallyworld__ | everything went belly up | 23:47 |
| wgrant | Hah | 23:47 |
| wallyworld__ | wgrant: i've pushed a change, can you try again? | 23:51 |
| 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:52 |
| 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:53 |
| 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:54 |
| wgrant | Interestingly, in python2.7 it doesn't crash initially. | 23:55 |
| wgrant | It encodes UTF-8 with base64. | 23:55 |
| 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:56 |
| 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:57 |
| 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:58 |
| wallyworld__ | argh, ffs. another unity crash | 23:59 |
| wallyworld__ | i don't think the payload is meant to be pre-encoded | 23:59 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!