/srv/irclogs.ubuntu.com/2012/04/17/#launchpad-dev.txt

lifelessfjlacoste: oh hai00:30
lifelessfjlacoste: if you want to catch up in the evening or whatever, just let me know00:30
fjlacostelifeless: i think i'll pass today, sorry00:30
lifelessfjlacoste: is cool00:30
lifelessfjlacoste: hows the conf ?00:30
fjlacostelifeless: pretty good00:32
StevenKwgrant: Can haz another look?01:03
wgrantStevenK: Done01:09
* StevenK looks for "This is terrible."01:09
StevenKOMG, referencing a bug that *isn't* bug 933766.01:18
_mup_Bug #933766: Update bug to use information_visibility_policy <bad-commit-14986> <bugs> <disclosure> <qa-untestable> <Launchpad itself:In Progress by stevenk> < https://launchpad.net/bugs/933766 >01:18
* wallyworld stabs thunderbird. using 3GB of memory and thrashing disk trying to get up to date with qastaging inbox. still waiting.....01:34
StevenKwallyworld: Using Thunderbird to read the staging inbox? Ha. Hahaha. HAHAHAHAHAHA01:35
wallyworldyeah, i know now it is a bad idea :-(01:36
rick_hlifeless: ping, if you have a sec can you peek at the MP for the email notice stuff. I've got two failing tests to fix, but shouldn't change much. https://code.launchpad.net/~rharding/launchpad/email_notice_95948201:36
StevenKwallyworld: Seriously, though, turn off the agressive caching, since Thunderbird loves to cache the entire message.01:36
rick_hlifeless: the gpg and oauth adjustments are in a follow up branch I'm still working on getting01:36
wallyworldStevenK: is there a better alternative like pine or something worth considering?01:37
StevenKHahaha, pine.01:37
rick_hofflineimap + mutt ftw!01:37
StevenKIs your knowledge of MUAs like 10 years old?01:37
wallyworldnever used those but perhaps i need to01:37
rick_hI'm a big mutt user01:37
cjwatsonmutt's generally fine for large mailboxes but do enable its header cache.01:37
rick_hyea, definitely01:37
rick_hhttp://blog.mitechie.com/2011/11/20/an-updated-email-config-2-offlineimap-mutt-and-dovecot-ftw/01:38
cjwatsonExactly my combination01:38
StevenKwallyworld: I think the staging inbox howto thingy shows how to configure offlineimap01:38
* StevenK tends to be evil, and just uses wgrant to look at the staging inbox for him ...01:39
wallyworldStevenK: yes, i think it does. i never figured it would be mandatory to use it :-)01:39
rick_hmy license plate is 'cli4lif' :)01:39
wallyworldfunny01:39
StevenKwallyworld: It depends if you have 48GiB of RAM available and 3 days to wait? :-P01:39
wallyworldStevenK: i only have 4GB :-(01:40
StevenKAnd a crappy i7 Atom that randomly reboots.01:40
StevenKBut that's what you get for buying a laptop based on the fact that it has a numeric keypad. :-P01:41
wallyworldit hasn't done that the past couple of days, touch wood01:41
wallyworldi bought it because it has a decent screen resolution01:41
wallyworld1600x90001:41
StevenKwallyworld: My X201 is 1280x800, but I spend my time working on a 24" 1080p LCD.01:46
wallyworldStevenK: my main monitor is a 23" display but i also like to have a decent laptop screen too01:47
StevenKOh bugger, I put off voting in the DPL elections so long they elected zach without me.01:47
wgrantStevenK: That's Disclosure Supreme, Flawless, and Marvellous Plan to you.02:17
StevenKHaha02:17
StevenKTempted to raise that on the call tomorrow. :-)02:17
* StevenK stabs Banshee four or five times.02:23
StevenKI plug my mp3 player in, start Banshee, it spews 200 lines or so of Mono garbage tracebacks and then segfaults.02:24
spm"feature"02:25
StevenKAnd the Banshee devs complained when we switched the default away?02:25
StevenKThere is a reason I use Quod Libet as my music player.02:25
spmexaile here. used to be a die hard amorak fanboi, but around a year or 3 ago they made me cry. so I gave up.02:26
StevenKI gave up on amorak when it caused my (at the time) dual Athlon to melt into a puddle.02:27
spmi think it was the crashing everytime I asked it to do something complex like "play this tune" that was the main kicker for me02:27
StevenKHaha02:27
spmmuch like how my previous smartphone would lock dead requiring a hard reset with complex tasks like receiving an SMS02:28
* wgrant just uses Rhythmbox :)02:29
StevenKRhythmbox started acting very strange for me last release so I went back to the loving Python embrace of Quod Libet.02:30
spmis that the current default? if Y, I'm sure I tried it, and gave up on it rather quickly02:30
wgrantI used Banshee while it was the default, since Rhythmbox was indeed pretty broken.02:31
StevenKI remember trying Exaile for a day (I think it was 0.22) and giving up in disgust, but I can't recall why.02:31
lifelessStevenK: just think, your vote would not have changed anything02:31
wgrantBut switched back in early Precise.02:31
StevenKlifeless: Meh, thank you for making me feel SO much better.02:31
spmheh02:31
lifelessStevenK: I failed to vote too02:31
* StevenK goes to buy lunch and probably drown.02:32
* wgrant stabs GIST in the face03:33
StevenKwgrant: Oh?03:33
wgrantEverything goes fine on dogfood until I try to FTI03:34
StevenKI didn't drown, but it was a close run thing. Apparently, Sydney is going to get 140mm of rain over the next two days.03:34
wgrantHeh03:34
wgrantAhaaa03:35
wgrantOnce the index is hot, fti is fast03:35
wgrantStevenK: You probably have insane team memberships03:43
wgrantStevenK: How do https://bugs.dogfood.launchpad.net/ubuntu/+bugs?field.searchtext=kernel and https://bugs.launchpad.net/ubuntu/+bugs?field.searchtext=kernel compare for you?03:43
lifeless(Error ID: OOPS-71f5806d495ce2928a38ecae3e91b757)03:45
lifelessand03:45
lifeless    At least 47 queries/external actions issued in 2.00 seconds03:45
wgrantErm03:45
wgrant2s?03:45
wgrantShould be at least 603:45
wgrantOh, that's a prod oops03:46
lifeless2s on dogfood03:46
wgrantRight, that makes more sense.03:46
wgrantSo, success? :)03:46
wgrant(this early version of the feature flag has prejoining disabled, so there's a bit of death by a thousand queries still in play, so it will really be faster)03:47
lifelesswgrant: nice03:48
StevenKwgrant: I was lunching, do you still want me to check?04:28
wgrantStevenK: More data is always nice.04:29
wgrantEven though the data so far suggests that the Disclosure Supreme, Flawless, and Marvellous Plan lives up to its title.04:30
StevenKwgrant: Hah, so. Timeout on prod, and 29second on DF04:30
wgrantStevenK: What if you retry?04:31
wgrantThe fti and most of the table had probably dropped out of DF's cache04:31
wgrantGiven they're like 2GB all up, and DF has 4GB...04:31
StevenKwgrant: 9.01 seconds on prod and 2.76 seconds on DF.04:31
wgrantThat's more like it.04:32
StevenKI thought 9.01 should have timed out.04:32
wgrantIt must have ticked over 9 right after the timeout check.04:32
wgrantStevenK: What are the timings for the two BugTaskFlat queries?04:32
wgrantin the DF query log04:32
StevenKSigh, mawson is all the way over there.04:32
wgrantNo04:32
* StevenK reaches for it.04:32
wgrantClick link at top of page04:32
wgrant"XX queries/external actions"04:33
wgrantit's a link04:33
wgrantexpands query log04:33
lifelessonly if you're logged in and in the lp dev group there04:33
StevenK509ms and 1202ms.04:33
wgrantWhich StevenK is04:33
wgrantHuh04:33
wgrantSlow04:33
wgrantk04:33
StevenKlifeless: Pft, I'm a duck on DF.04:33
wgrantBurn!04:34
StevenKBurn which bit?04:34
wgrantYou04:34
wgrantYou're a witch :)04:34
StevenKReally?04:34
StevenKwgrant: Looks like my team memberships and such are a bit more insane than lifeless'?04:35
wgrantOr lifeless may not have been logged in04:36
wgrantAlthough04:36
wgrantYours should be faster...04:36
lifelessI wasn't logged in04:36
wgrantsince you're a duck04:36
wgrantIt will ship all the perm checks04:36
wgrantskip04:36
wgrantSigh04:36
wgrantneed more ram04:37
wgrantMy desktop has 4x more than mawson :(04:37
lifelesshah, sso puts a spurious : after the fields in the what-fields-you-get confirmation form04:37
wgrantYeah04:37
wgrantHas for ages04:37
wgrant /ever04:37
StevenKwgrant: But your desktop was manufactured sometime this century. :-)04:38
lifelesswgrant: 143 queries/external actions issued in 11.12 seconds04:38
lifelessAJAX Log ↓04:38
wgrantYeah, DF doesn't do so well with a cold cache04:38
lifeless1895ms 04:38
lifelessSQL-main-slave: SELECT * FROM ((SELECT Person.account04:38
wgrantI've just been browsing Launchpad's bugs, so Ubuntu will be long gone04:38
wgrantHeh04:38
wgrantSo that's just listing your team administerships04:38
lifeless2248ms 04:38
StevenKI think mawson's disks have begun to fossilize04:38
lifelessSQL-main-slave: SELECT BugTaskFlat.bugtask FROM04:38
wgrant:(04:39
lifeless4991ms 04:39
lifelessSQL-main-slave: SELECT COUNT(*) FROM BugTaskFlat04:39
lifeless143 queries/external actions issued in 5.14 seconds04:39
lifelessAJAX Log ↓04:39
wgrantI guess I should fix the last few tests failures and get this onto qas.04:39
lifelesslots of SQL-main-slave: SELECT TeamParticipation.id, TeamParticipation.person, TeamParticipation.team FROM TeamParticipation WHERE TeamParticipation.person = 2980965 AND TeamParticipation.team = 43091404:40
lifeless1606ms 04:40
lifelessSQL-main-slave: SELECT BugTaskFlat.bugtask04:40
wgrantYeah, those aren't even my fault :/04:40
lifeless1764ms 04:40
lifelessSQL-main-slave: SELECT COUNT(*) FROM BugTaskFlat WHERE BugTaskFlat04:40
lifeless60ms 04:40
lifelessSQL-main-slave: WITH teams AS (SELECT team from TeamParticipation WHERE person=2)SELECT combinedbugsummary.distroseries04:40
lifeless :)04:40
wgrantlifeless: I think those TP checks might be subscriptions04:40
wgrantDeciding whether you're subscribed or not04:41
wgrantPretty hilarious way to go about it, which is why it's completely plausible.04:41
lifelesswhy would that page care ?04:41
wgrantlifeless: It has the new bug subscriptions stuff on it.04:47
wgrantAnything touched by that is cursed forever.04:47
wgrantTo be 500ms slower than it would otherwise be.04:48
StevenKwallyworld: Can haz QA for r15100?04:54
wallyworldStevenK: still waiting for my qas inbox to finish syncing04:54
* StevenK grumbles about having to feature flag the UI changes.04:54
* wallyworld grumbles about yui tests in lxc core dumping :-(04:57
wgrantwallyworld: You've enabled X forwarding in SSH?05:00
wgrantDown to 92000 messages in the staging inbox...06:20
wgrantNearly there06:20
wallyworldwgrant: i don't use ssh with the lxc setup is use, i just sudo lxc-start -n lpdev and get a login prompt directly06:39
wgrantwallyworld: Ah, that would do it.06:42
wgrantwallyworld: No X server06:42
wgrantwallyworld: Perhaps try xvfb-run06:43
wallyworldah right. will do thanks.06:43
wgrantwallyworld: Did you make any progress on QAing poolie's size limit?06:43
wallyworldwgrant: *still* waiting for fucking qas inbox to download everything06:43
wgrantwallyworld: I just deleted 430000 mails from it06:43
wgrantYou might want to restart thunderbird06:43
wgrantIt's got like 4000 now.06:43
wallyworldalready have. i can delete about 50000 at once. it's been swapping all day :-(06:44
wgrantI did 100000 at a time. Just had to click Continue about 100 times.06:44
wgrantMake sure you disable the message pane when doing bulk operations.06:44
wgrantOtherwise it will try to render a list of threads.06:45
wallyworldthat would help06:45
wallyworldi've lost count of the number of "script has stopped" dialogs i've clicked on06:45
wallyworldplus when the disk thrashes, the desktop stops accepting mouse clicks :-( not sure whether to blame unity or X or compiz06:46
wgrantSounds like you're swapping heavily :)06:46
wgrantNeed moar rams06:46
wallyworldyeah. normally get by fine with what i have06:47
wgrant4GiB?06:47
wallyworldyep06:47
=== _mup__ is now known as _mup_
wallyworldwgrant: i see you sent a large email to qas.07:18
wallyworldmy qas inbox finally cuaght up07:18
StevenKIt only took all day.07:18
StevenKIs your laptop still thrashing?07:18
wallyworldyeah :-(07:18
wallyworldno07:18
wallyworldjust finished07:18
wallyworldafter 1000000 thunderbird restarts and clicking on Continue dialogs07:19
StevenKMaybe you'll use offlineimap next time. :-P07:20
lifelessI've just purged it07:21
lifeless42K messages, 0 need for them07:21
wallyworldStevenK: well, i didn't think i needed to :-)07:21
lifelessyou don't07:21
lifelessif folk delete as they qa07:21
wallyworldtoo bad we don't clear qas inbox more often07:21
StevenKlifeless: You tell funny jokes.07:21
lifelessStevenK: apparently so07:22
wallyworldwhy don't we automatically purge once every couple of days?07:22
wgrantlifeless: Did you just delete *everything*?07:22
StevenKlifeless: Suprised you didn't make a ruling in terms of non-LP members doing QA.07:22
wgrantI'd already deleted most of them.07:22
lifelessnoone has gotten theactivation energy up to do it07:22
wgrantAnd I think you nuked my test mail :(07:22
wallyworldwgrant: i read it, looked ok07:22
lifelesswgrant: there were 42K messages there, I wasn't selective07:22
wgrantwallyworld: Ah, thanks.07:23
wallyworldwgrant: i marked the bug as qaok07:23
wgrantlifeless: I usually leave everything from the last 24 hours07:23
wgrantwallyworld: Even better.07:23
wgrantwallyworld: I get "NotFound: Object: None, name: u'index.html'"07:27
wgrantwallyworld: Nothing about a default view07:27
wallyworldwgrant: i was going by the string handed to the exception constructor in the base class. but anyway, it still refers to index.html which implies a page is expected07:28
wgrantwallyworld: That's correct.07:28
wgrantWhen browsing to an object in a browser, it will try to render the default view.07:28
wgrantThe default view is index.html unless overriden.07:28
wgrantThis is normal.07:28
wgrantEg https://launchpad.net/package-sets07:29
wgrantSame for any object that doesn't have views07:29
wallyworldin this case i don't think it is because i see it as a case on an incomplete url07:29
wallyworldlike lp/net/launchpa07:29
wallyworldtypo07:29
wallyworld. not /07:29
wgrantStructurally it is a complete URL.07:29
wgrantOverriding as you've done is of minimal utility and breaks the convention around the rest of the codebase and every other ZTK app.07:30
wallyworldoh alright. not worth arguing about. i'll change it07:30
wgrantTo a user the difference is 007:31
wgrantThey don't see the traceback.07:31
wgrantHah07:34
wgrantI'm going to accidentally fix the bug where milestones are duplicated on the advanced search page.07:34
wgrantHow convenient.07:34
lifelessoh noes07:55
lifelesswe can't have less bugs07:55
lifelessbugs are currency07:55
StevenKYou would say that.07:55
lifelessYHBT HAND HTH07:56
bigjools 07:57
bigjoolsbwaha07:58
czajkowskiI need a dictionary for lifeless07:58
bigjoolsit's funny because that's what StevenK trots out07:58
lifelessczajkowski: urban dict probably has them all08:02
lifelessczajkowski: or perhaps the new new hackers dict08:02
czajkowskilifeless: would you mind haveing a look at juju bug against lp please. https://bugs.launchpad.net/launchpad/+bug/98353008:03
_mup_Bug #983530: "charms" needs branch name consistency <juju:Invalid> <Launchpad itself:New> < https://launchpad.net/bugs/983530 >08:03
czajkowskilifeless: a swift google was helpful08:03
lifelessSpamapS: hey ^08:03
czajkowskimy brain doesnt seem to want to fucntion at this hour of the morning08:03
lifelessSpamapS: what is the problem ?08:03
wgrantlifeless: See backscroll08:03
adeuringgood morning08:03
wgrantFrom about 8 hours ago08:04
SpamapSlifeless: in charms, we often use 'bzr push lp:charms/foo' to create new charms.08:24
SpamapSlifeless: this creates lp:~yourname/charms/series/foo/trunk08:25
SpamapSlifeless: however, when branch-distro is run to copy forward and do the stacking jitterbug, it uses lp:~yourname/charms/series/foo/series08:25
SpamapSlifeless: the charm store code specifically targets 'trunk' because we need users to be able to have a way to say "this is my personal published version of foo"08:26
lifelesswait, what ?08:26
SpamapSlifeless: so we can't really use branch metadata becaus we can't have two foo's for any one user08:26
SpamapSor rather we can't have two foo's for any one user+series08:27
lifelesshow does that impact the blessed charms ?08:27
SpamapSlifeless: the charmstore maps lp:~auser/charms/precise/foo/trunk to 'juju deploy cs:~auser/precise/foo'08:28
SpamapSlifeless: the blessed charms do have a 1:1 series<->charmname mapping, so I did suggest that we should just use that to workaround this. But it stands to reason that branch-distro should probably agree with push-to-create08:28
lifelessSpamapS: are they able to say cs:~auser/precise/foo/trunk, if they want to ?08:29
SpamapSlifeless: no08:29
SpamapSlifeless: though I believe that is desired08:29
SpamapSits not part of the store *now*08:29
lifelessso for users, this is irrelevant, because its only the blessed charms that are branched08:30
SpamapSright08:30
lifelessthe blessed charms should be listed as the development focus branch, in LP's package metadata08:30
lifelessso they shouldn't be affected either08:31
lifelessunless the charm store isn't using that pointer (which it should)08:31
SpamapSit is to map lp:charms/foo -> cs:foo08:31
SpamapS(which is the default, so 'juju deploy foo'08:31
SpamapSlifeless: the troubling part is the inconsistency08:32
SpamapSlifeless: I know we can mold the charm store loader to just load the dev focus branches as blessed regardless of their name. but they'll still have weird names08:32
SpamapSlifeless: also when copying forward, one thing that happened was any charms that were manually ported forward to precise were named 'trunk', and so were not detected as already existing.08:33
wallyworldwgrant: can haz +1 on that review?08:35
wgrantwallyworld: Ah, sure. Was just waiting for the diff to update, then forgot.08:45
wallyworldwgrant: np. thanks. i am trying to make the next bb run :-)08:58
=== almaisan-away is now known as al-maisan
StevenKcjwatson: It matters little, but spot the duplication in your most recent landing:09:37
StevenK120+ distro = bpph.distroseries.distribution09:37
StevenK121+ script = self.makeScript(bpph.distroseries.distribution)09:37
cjwatsonStevenK: reasonable point; I was copying from test_getDirtySuites_returns_suite_with_pending_publication and test_getDirtySuites_ignores_suites_without_pending_publications and didn't notice09:45
cjwatsondoesn't seem worth another landing to fix though, really?09:46
=== al-maisan is now known as almaisan-away
lifelessSpamapS: ok, so I think that the bug may need some fine tunuing ;)11:00
lifelessgnight11:00
StevenKcjwatson: Nah. Like I said, it matters very little. If you care, you can do a drive-by in your next branch. Or you don't care, either is fine.11:09
StevenKczajkowski: :-(11:25
czajkowskiStevenK: now what have I done.11:25
StevenKczajkowski: Your G+ reply, you bad person.11:25
czajkowskiStevenK: hi we've clearly never met if you're only realising that now :)11:26
czajkowskiI blame the blessed charms for todays corruption!11:26
StevenKHah11:26
StevenKWhich makes it SpamapS fault.11:27
StevenKAnd the universe makes sense again.11:27
=== almaisan-away is now known as al-maisan
=== matsubara-afk is now known as matsubara
dpmhi all, could someone help me with an issue with translations? We've got the calligra source package in Ubuntu and its translations do not seem to get imported. Here are the details:13:08
dpmThe imports queue is empty save for some manual uploads:13:08
dpmhttps://translations.launchpad.net/ubuntu/precise/+source/calligra/+imports13:08
dpmThe po files were uploaded separately with the calligra-l10n package:13:09
dpmhttps://translations.launchpad.net/ubuntu/precise/+source/calligra-l10n/+imports13:10
dpmIt's a KDE translation, which makes it a bit special and means the PO files will not be imported into any calligra-l10n template, but rather on the corresponding template in the calligra source packad13:11
dpm*package13:11
dpmSo my question is whether someone could help me determine whether the PO files will end up in the right package or whether I should start to worry13:12
dpmI know there is a check somewhere in the LP code to determine if an upload corresponds to a KDE package and then takes care of all this, but I'm not familiar with the code other than knowing that this function exists13:13
dpmI just want to ensure the calligra translations end up in the right place and can be shipped in the language packs13:13
adeuringabentley: standup?13:32
rick_hdpm: hmmm, so there are imports on that l10n going back to Jan it looks like, 4k of them?13:55
rick_hjtv: ping, I'm told you might be a translations master and know something about how this works?13:56
dpmrick_h, yeah (not sure about the translations part, though :). Back then the calligra package did not have any templates, so the translations were uploaded but had nowhere to go, as PO files only get imported if there is a corresponding template. Last week the templates were created and a new upload was made, which should in theory solve the issue. The way LP works, the 4k old translations will remain there, I believe, as there is no template associate13:58
dpmd with them.13:58
rick_hdpm: ok, so there were some imports done after the 4/11 imports that didn't have a template?13:59
dpmrick_h, the templates were created last week (on the 10th or 12th, I think). All of the uploaded PO files before that date will remain in Needs Review (forever, I believe), whereas the PO files uploaded after should in theory get associated with a template and should be imported. And the "should get imported" part is what I'm trying to figure out :)14:02
rick_hdpm: ok, I've ping'd jtv, I'm not sure atm how the imports work like this and asking in our stand up didn't get me much further. Let's see if he gets back to us and if not I'll try to find more help/guidance.14:04
dpmrick_h, ok, thanks for your help. jtv is on his day off, I think14:06
gary_posterjml, are you still hoping to merge your three testtools branches today?  We're already making our own rollup branch, but we'd still love to have an official merge, let alone an official release.14:12
gary_posterWe'll need to do similar dances for testrepository (we have one MP with no reply, with another on its way) and subunit (we need a release of the trunk in order to get some tag support changes that Robert added recently).14:13
jmlgary_poster: yes. about to do so now.14:16
jmlgary_poster: sorry about the muck-around14:16
gary_posterawesome thanks jml.14:16
jmlgary_poster: I'm going to apply lifeless's suggestion from the yellow testr review14:16
jmlgary_poster: and parametrize wrap_result14:16
gary_postercool jml, thanks14:16
gary_posterbenji ^^14:16
jmlgary_poster: that will cause a conflict for you when you merge, but hopefully nothing too tough14:17
jmlfwiw, my excuse is that I moved house yesterday (taking the day off) and still don't have home internet today.14:17
gary_posterjml, ugh!  moving house is never easy, and sometimes difficult for months or years later. :-) thank you very much for getting this through14:18
jmlhaving fast Internet changes *everything*14:25
gary_poster:-)14:27
jmlgary_poster, benji: tagger, tsfr-fixup and wrap-result-in-concurrent-suite all merged & pushed (lp:testtools r253). Let me know if there are problems.14:36
gary_posterthank you very much jml14:36
jtvrick_h: sorry, I'm not doing any more work tonight!15:02
salgadojamestunnicliffe, you'll need to ask one of the LP tech leads to approve a feature flag change to enable the view to members of Linaro15:02
salgadothe feature flag will be something like "team:linaro registry.upcoming_work_view.enabled 1" (need to check the correct syntax/order)15:03
sinzuijcsackett, how goes the bug subscriber branch?15:04
salgadosinzui, maybe you can approve that feature flag (^) change?15:06
jcsackettsinzui: i spent some time reading through the whole of the various subscription code last night, so i've been able to iron out those inconsistencies and am making progress.15:11
jcsackettthere are some tests regarding the DB triggers i haven't gotten to looking at yet--there's a very good chance i'll be pinging you when i hit those. i expect they're going to be complicated.15:12
sinzuisalgado, sure, is the entry on the osa page?15:12
salgadosinzui, not yet. should I add it?15:12
jcsackettsinzui: would've pinged you earlier, but i was excited once i had things sorted out and just sort lost myself in fixing things.15:12
sinzuijcsackett, okay, please ping me to talk if you are getting frustrated. I can listen and sympathise, and sometimes help fix15:13
jcsackettsinzui: will do, and thanks. :-)15:14
sinzuisalgado, add it the the ff section on https://wiki.canonical.com/InformationInfrastructure/OSA/LaunchpadProductionStatus15:14
sinzuiyou can add me as the approver now since I agree that linaro needs to see the feature work15:14
salgadojamestunnicliffe, so, that (^) is what needs to be done, but the wiki page is internal-only so the next time you need to do it you'll just need to ask whoever approves the feature flag to add it to the wiki page as well15:15
salgadothanks sinzui!15:15
jamestunnicliffethanks salgado!15:16
jamestunnicliffe(and sinzui)15:17
czajkowskisinzui: oh wise one, should a bugs description be editabled after a fix released so in theory it's closed be allowed to happen ?15:23
jmlbac: hi15:23
jmlbac: I've had a bit of a look at https://code.launchpad.net/~bac/testrepository/bug-949950/+merge/102317. Do you want to discuss it here, or would you rather a put some notes on the MP?15:24
bachi jml15:24
bacjml: here is fine, as long as it is summarized on the MP15:24
sinzuiczajkowski, yes it should be editable so that confusing text is removed15:24
jmlbac: can do.15:24
jmlbac: I downloaded it and gave it a try. It looks like you'll only get test failures and start & end timestamps in the subunit stream.15:25
bacjml: oh, really15:26
jmlbac: yeah15:26
bacthat isn't great15:26
jmlbac: I didn't think it was what you wanted.15:27
bacjml: any thoughts on why that happens?15:27
jmlbac: it's not very obvious, but 'run' delegates to 'load'15:27
jmlbac: and 'load' uses the make_result() that your patch changes15:28
jmlto make something called 'output_result', which is the result responsible for UI output (which makes sense, I hope)15:28
bacyes15:28
jmlimmediately below that, is this:15:29
jml        filtered = TestResultFilter(output_result, filter_skip=False)15:29
jmland it's that 'filtered' result that gets used from then on15:29
jmlso output_result never even gets the events for successful tests15:29
jml(sorry for dragging this out, I'm figuring it out as I go along)15:29
bacjml, np15:30
jmlI think the right thing to do is to push that TestResultFilter call down into the UI make_result15:31
jmlbecause, really, choosing which results to display is a UI decision.15:31
bacjml: makes sense15:32
jmlbac: there'll probably be a bunch of annoying test fallout from that.15:32
jmlbac: sorry.15:32
bacjml: and you're suggesting not applying the filter when we select --subunit?15:32
jmlbac: uhh, yes, I think so.15:32
jmlumm sort of15:32
jmlbac: there's a part of me that thinks there should be another option that controls whether only failures are shown vs full results15:33
jmlbac: and that the default should stay as is regardless of whether the output is the pretty text output or a subunit stream.15:33
czajkowskisinzui: even after the bug has been fixed released?  surely editing the description then is confusing15:34
bacjml: so grow a '--full-results' (modulo spelling).  would that be only for the run command?15:35
czajkowskisinzui: an example is https://bugs.launchpad.net/launchpad/+bug/272826  which was edited today15:35
_mup_Bug #272826: "Ubuntero" inappropriate for female contributors <lp-registry> <Launchpad itself:Fix Released by barry> <Ubuntu Website:Fix Released by newz> <Ubuntu:Fix Released by communitycouncil> < https://launchpad.net/bugs/272826 >15:35
=== al-maisan is now known as almaisan-away
jmlbac: run & load yes.15:35
bacjml: ok.  so if you'll disapprove that MP i'll look at doing that this afternoon15:36
jmlbac: the other point of my feedback was going to be that load should have the option too. I think that's pretty trivial to code up.15:36
jmlbac: will do that, and put my notes there now.15:36
bacjml: cool, thanks15:36
jmlbac: thank you!15:36
sinzuiczajkowski, it is not a bug. Bug descriptions often need to be updated to distinguish past behaviours from modern ones, or provide proper urls, tests, or work arounds so that users do not report a new bug or reopen the existing one15:37
czajkowskisinzui: I suppose if it were called a bug summary it wouldb't be as bad, I guess calling it a descrption is just rather misleading in this case.15:39
sinzuiczajkowski, that is different issue. The bug title is not a title either15:39
czajkowskiindeed15:40
maxbIs there any known breakage with package copies at the moment? I'm trying to copy reusing binaries from natty to oneiric within a single PPA, and I'm getting a spurious binary conflict16:09
czajkowskimaxb: there was lp servers rebooted there at 16:00 UTC16:13
czajkowskifor about 10 mins16:13
maxbI'm getting a specific logic error, rather than a downtime-related issue16:13
=== salgado is now known as salgado-lunch
rick_habentley: ping do you have time for a quick review? https://code.launchpad.net/~rharding/launchpad/email_notice_extras_959482/+merge/10234116:24
rick_habentley: some drive by linting at the top makes it look bigger than it is16:24
jmlCan someone please land https://code.launchpad.net/~jml/launchpad/expose-commercial-on-create-ppa/+merge/10190716:38
rick_hjml: sure thing, will pull that down16:38
jmlrick_h: thank you16:39
=== matsubara is now known as matsubara-lunch
rick_hjml: onto ec2 ok16:51
jmlrick_h: \o/16:51
abentleyrick_h: Sorry, was lunching.17:24
rick_habentley: np, no hury17:25
abentleyrick_h: LOC rationale?17:26
rick_habentley: sorry, it's been ok'd in the dep branch by lifeless as small hit to fix security issue17:27
abentleyrick_h: r=me17:29
rick_habentley: ty17:29
=== salgado-lunch is now known as salgado
=== matsubara-lunch is now known as matsubara
sinzuijcsackett, r=me18:24
jcsackettsinzui: thanks.18:42
dobeycan i beg someone to make a +downlaod page work for each series, and not have only the generic one for a project?19:25
sinzuidobey, We don't have any requirements for what +download should do anymore. It is truly fucked. I will not make anymore changes until someone provides the use case for it. It was once about packagers...and it certainly is not now19:44
sinzuidobey, Why does a series need +downloads?19:46
dobeysinzui: for packagers :)19:46
sinzuidobey, shouldn't the series page show the current release tarball then?19:47
sinzuidoes it need to show anything more?19:47
dobeyi suppose it could just be on the series page if uscan can handle that19:47
sinzuiah19:48
dobeyhttp://launchpad.net/ubuntuone-storage-protocol/+download .*/ubuntuone-storage-protocol-([0-9.]+)\.tar\.gz19:48
dobeythat's what is in debian/watch for example19:48
sinzuiI already reported a bug specifically for that situation, but some how it became unimporant and I stopped trying to please everyone19:48
dobeybut it wants to grab 3.0.0 which is for precise, even though i'm doing an SRU for oneiric, for example :)19:48
sinzuihttps://bugs.launchpad.net/launchpad/+bug/23179719:49
_mup_Bug #231797: no sensible way to use debian/watch files with launchpad hosted tarballs (no simple url-and-link list of all downloads) <lp-registry> <packaging> <releases> <Launchpad itself:Triaged> <devscripts (Ubuntu):Invalid> < https://launchpad.net/bugs/231797 >19:49
dobeybeing able to specify the series would be brilliant, but there's no way to do that right now19:50
dobeyyou can specify a link to a specific milestone, which isn't the same19:51
sinzuiI think we just want a permalink that is obvious from the url structure to support it. It is only for debian/watch case, so no one can screw it up again19:51
dobeyor can i use the series page perhaps?19:51
dobeyi guess i should try that19:52
sinzuidobey, https://launchpad.net/gdp/0.5.x shows the current release19:52
sinzuieg a real url with a tarball in it19:52
dobeyright19:53
sinzuibut I personally would not trust it since it is not designed for uscan19:53
dobeyand +download is?19:53
dobeyit has the exact same links on it, but it's paginated and has everything ever released19:54
sinzui+downloads is a semi0orderd list of all releases from all series. It is not clear who needs it since obsolete series are listed and it seams to encourage end-users to download and compile lots of tarballs19:55
sinzuiThe pagination was added to support some odd case were someone want to get old data, possibly because someone uploaded a .exe19:55
sinzuiOnce end-users were using that page and it was paginated to do fuck-knows-what, the people who needed it most were cut out. The whole mess it the result of a handful of well-meaning developers adding features without really verifying why if they accomplished something meaningful19:57
lifelesswouldn't it be nice if we could test for that19:58
sinzuidobey, do not trust anything in a "downloads" portlet it shows the latest uploaded thing, not the tarball, and not the actual version...again, there is not clear use-case for it. I think the intent was to encourage end-users to learn how gcc works19:59
lifelesssinzui: how are things; is there anything you want to catch up on - its been a few weeks since we last spoke19:59
dobeysinzui: well it it shows the latest uploaded thing for the series. i presume i can halfway trust it if i control what gets uploaded and when :)20:00
sinzuilifeless, we are not facing any technical issues, we are just struggling to to step on each other to get the UI ready for beta20:00
dobeysinzui: so probably safe enough to use for packages for projects i maintain? or should we get some more reasonable thing set up like a +download page for series?20:01
sinzuidobey, you can trust it if you trust the maintain who uploads. The portlet is correct for my work because I really do want packagers to the right thing..but I have the advantage oh having read the code and bug reports. I am still bitter as you can tell20:02
dobeyheh20:02
sinzuiI propose something special and testible for bots so that it is clear the feature is not to address someone's feelings...bot just need the correct data and nothing more20:03
dobeysure. sounds good20:04
lifelesssinzui: ok cool20:10
lifelesswgrant: you were asking about job clearing on restores...20:41
lifelesswgrant: I suspect we don't, systematically - [STAGING] Cron <launchpad@gandwana> $LP_PY /srv/staging.launchpad.net/staging/launchpad/cronscripts/process-job-source.py IPlainPackageCopyJobSource -q --log-file=INFO:/srv/staging.launchpad.net/staging-logs/process-job-source.IPlainPackageCopyJobSource.log20:41
lifeless2012-04-17 20:39:30 ERROR   Unhandled exception20:41
lifeless -> http://staging.launchpadlibrarian.net/99588237/gYMKrWK5lzGeNC4Db0iVcuSdQWW.txt (ERROR:  No such user20:41
lifeless)20:41
wgrantlifeless: Was I?21:59
lifelesswgrant: s/asking/speculating/ yesterday22:07
[reed]hey, I'm not sure where exactly I should be filing this... https://bugs.launchpad.net/launchpad/+bug/98441522:08
_mup_Bug #984415: Launchpad bugzilla integration causes Bugzilla-side warnings <Launchpad itself:New> < https://launchpad.net/bugs/984415 >22:08
[reed]is that the right place?22:08
lifelessthats fine yes.22:09
lifelesswhats happening ?22:09
=== almaisan-away is now known as al-maisan
=== al-maisan is now known as almaisan-away
[reed]lifeless: getting errors on Bugzilla's side every 10 min. when launchpad hits it :)22:29
[reed]trying to track down what's causing them22:29
=== salgado is now known as salgado-afk
=== StevenK changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: StevenK | Firefighting: - | Critical bugtasks: 4*10^2
wgrantrick_h: Did you get a text review of your new email template this time? Also, "security_field_changed" isn't a valid method name.23:32
wgrantThere are also various text issues in the notification strings in the code.23:33
lifelesswgrant: whats invalida bout it ?23:53
lifelesswgrant: being pep8?23:53
wgrantlifeless: Yes23:54
wgrantThat bit of PEP 8 illegal in Launchpad.23:54
wgrant+is23:54
lifelessahh well, life is too short23:56
lifelessI wrote that method23:56

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