/srv/irclogs.ubuntu.com/2011/05/02/#launchpad-dev.txt

=== Ursinha-afk is now known as Ursinha
lifelessthanks python 2.700:23
lifelessString or Integer object expected for key, unicode found00:23
wgrantWhat for?00:27
wgrantSome fake dict?00:27
lifelessDBM00:27
lifelesshttps://bugs.launchpad.net/testrepository/+bug/77521400:28
_mup_Bug #775214: On python 2.7: String or Integer object expected for key, unicode found <Testrepository:New> < https://launchpad.net/bugs/775214 >00:28
wgrantAh :(00:29
lifelesswgrant: the fun of it is it that one can expect python 3.x to be diametrically opposed00:35
wgrantlifeless: Surely 3.x expects bytes/int?00:35
wgrant(if not I may cry)00:37
=== micahg_ is now known as micahg
lifelesswgrant: for a dict interface?00:49
lifelesswgrant: I haven't started a 3.x port of testrepository yet.00:49
wgrantlifeless: Yes.00:49
lifelessI may tonight or something; right now am just yak shaving my way back up to a working desktop00:49
wgrantSince Unicode makes very little sense in DBM, surely...00:49
lifelesswgrant: no less than it does in the sql interface postgresql uses00:50
wgrantlifeless: Hm? postgresql does Unicode...00:50
lifelesswgrant: yes00:51
lifelessdbm doesn't do collation00:52
lifelessso implementing unicode for a user is simply encode on the way in, decode on the way out.00:52
lifelessno good reason not to support that00:52
wgrantMmm00:52
lifelessthe only reason not to that I can think of is that its slow00:56
lifelessbut thats python's choice of string model more than anything else00:56
wgrantlifeless: https://lp-oops.canonical.com/oops.py/?oopsid=1944DQ27802:34
wgrantHeat updates are expensive.02:35
wgrant(although this seems like a particularly inefficient way to do it)02:35
StevenKThere was a change for that in the deployment report I noticed02:35
wgrantYeah, but different.02:36
StevenKOf course it is. :-)02:36
lifelesswgrant: https://lp-oops.canonical.com/oops.py/?oopsid=1944DQ278#repeatedstatements should be helped by abels change?02:43
wgrantlifeless: I thought that looked specific to Bug.markAsDuplicate.02:44
lifelesswgrant: I only read the cover letter02:47
lifelessso it may need an equivalent change02:47
wgrantlifeless: Except this is somewhat deeper :(02:49
=== lifeless changed the topic of #launchpad-dev to: PQM in RC mode | https:/​/​dev.launchpad.net/​ | On call reviewer: - | https:/​/​code.launchpad.net/​launchpad-project/​+activereviews
wgrantlifeless: How well do you know Storm's codebase?03:47
wgrantI am considering digging around to see why inherited References don't have the correct class, while normal properties do.03:49
wgrantBut the descriptor/metaclass stuff is not entirely clear.03:49
lifelessI can generally find what I am looking for03:57
lifelessnot far enough along to have a sense of how to tackle things tastefully03:57
wgrantAh.03:57
wgrantk03:57
lifelessso you have a reference on class A03:57
lifelessand a concrete class B03:58
wgrantYes.03:58
lifelessand the reference cls handle when you access B.reference is A03:58
wgrantThe Property descriptor handles that nicely.  But Reference does not.03:58
lifelessor03:58
lifelessis it that you have a reference into the class hierarchy ?03:59
wgrantPackageBuild.archive is inherited by BinaryPackageBuild.archive.03:59
wgrant'BinaryPackageBuild.archive == foo' creates a query for PackageBuild instead.03:59
wgrantBinaryPackageBuild.archive_id == foo works fine.04:00
wgrantBecause the descriptor recreates the column.04:00
wgrantWith the correct class.04:00
lifelessnow04:01
lifelessthis is with sqlobject sugar right04:01
lifeless?04:01
wgrantNo.04:01
wgrantRaw Storm.04:02
lifelessPropertyPublisherMeta looks interesting04:02
wgrantWell. BinaryPackageBuild is SQLBase, but PackageBuild and SourcePackageRecipeBuild are native.04:02
wgrantAnd I tested it with native non-LP Storm.04:03
lifelessPropertyRegistry04:03
wgrantMm, it's not that interesting.04:03
wgrantI think that's mostly for string property lookups.04:03
wgrantYes.04:04
wgrantclass PropertyRegistry(object): """ An object which remembers the Storm properties specified on classes, and is able to translate names to these properties. """04:04
wgrantSo not relevant here.04:04
wgrantIt's just the descriptors.04:04
wgrantlifeless: Compare Property.__get__ and Reference.__get__04:04
wgrantProperty uses _get_column, which creates a new PropertyColumn based on the right class. Reference just returns self.04:05
wgrant(where obj is None, obviously)04:05
lifelessyes04:09
LPCIBotProject windmill build #227: STILL FAILING in 1 hr 3 min: https://lpci.wedontsleep.org/job/windmill/227/04:14
wgrantlifeless: So, BuildFarmJobSet.getBuildsForBuilder04:19
wgrantThis method is The One.04:19
lifelesswgrant: you have a fix for storm ?04:20
wgrantDid you envision any particular solution to it?04:20
wgrantNo, I just worked around it by using _id.04:20
lifelesswgrant: what file is that in ?04:20
lifelesswgrant: did you file a bug ?04:20
wgrantlib/lp/buildmaster/model/buildfarmjob.py04:20
wgrantI didn't.04:20
lifelesswgrant: please file one04:20
wgrantI will. I spent most of Friday afternoon dealing with this so was pretty displeased by the end, so didn't end up filing a bug.04:21
wgrantBug #682989 is probably the same thing.04:22
wgrantOr rather similar.04:22
=== Ursinha is now known as Ursinha-afk
lifelesswgrant: so that query04:26
lifelesswgrant: one option is to union04:26
wgrantThat's what I was thinking.04:26
wgrantBut the sort may be slow.04:26
wgrantI was going to half-perform the migration in a transaction on DF and see what happens.04:27
wgrantBut populating BPB takes forever.04:27
wgrantlifeless: Although I guess key-based offsets with a union could be done rather cheaply.04:29
wgrants/rather/very/04:29
lifelessyes04:31
lifelessthe key question is whether sql will sensibly do partial iteration on each input04:32
lifelessUNION ALL should let it do this04:32
lifelessI'm looking at uses04:32
lifelessbuilder.getBuildRecords is one04:32
lifelessits binary_only flag will permit a single-table lookup04:32
wgrantI believe that should be the only user.04:34
lifelesslooks like it04:35
lifelesssadly getBuildRecords is used as a name for other things04:35
wgrantYes.04:35
wgrantIt's on like 5 or 6 other classes.04:36
lifelessI don't think its sensible to report back to the start of time on builders04:36
wgrantIt's not essential, no.04:36
wgrantI'd considered adding a threshold if the sort was too slow.04:37
wgrantBut it may not be.04:37
lifelessIf we consider dynamic provisioning of builders04:37
lifelessit becomes actively undesirable04:37
lifelesshmm, no wallyworld today04:41
wgrantThis week.04:41
wgrantOr next.04:41
wgrantHe's in Budapest.04:41
lifelessoh right04:41
lifelessanyone know how to land stuff on oops-tools ?04:45
wgrantpqm-submit?04:48
=== almaisan-away is now known as al-maisan
lifelessgrrr05:58
lifelessPATCH -> no idea what was changed05:58
wgrantOh?05:58
lifelessin oops-tools05:58
* wgrant vomits a bit.06:02
wgrantThe old bug subscription is awful :/06:02
wgrantNot obvious how to fix without reverting and deconflicting.06:02
wgrantThe old bug subscription *JS* is awful, that is.06:03
wgrant    // Determine if this is a dupe.06:03
wgrant    var is_dupe;06:03
wgrant    var icon_parent_div = icon_parent.get('parentNode');06:03
wgrant    var dupe_id = 'dupe-' + person.get('css_name');06:03
wgrant    if (icon_parent_div.get('id') === dupe_id) {06:03
wgrant        is_dupe = true;06:03
StevenKLPCIBot: I'm also not sure about the recent Jenkins picture change06:03
wgrant    } else {06:03
StevenKDoh06:03
wgrant        is_dupe = false;06:03
StevenKlifeless: ^06:03
wgrant    }06:03
lifelessStevenK: the new logo?06:03
wgrantIgnoring the whole if (foo) bar = true; else bar = false; thing, that's still a bit bad.06:03
StevenKlifeless: Yah06:03
lifelessI like it06:04
StevenKThe old one was mostly one colour, this is ... too red06:04
wgrantStevenK: Agreed.06:04
StevenKAnd French, rather than British. :-P06:05
wgrantlifeless: I don't think this needs reversion.06:05
wgrantIts only effect is minor.06:06
wgrantEverything else is still fine.06:06
wgrantAnd it doesn't revert cleanly.06:06
wgrantAlthough that's fixable...06:06
lifelessare you familiar with curtis question work?06:06
lifelessthe last remaining qa item06:06
wgrantuntestable, I believe, but let me check.06:07
StevenKOh dear. The last 55 windmill builds on Jenkins have failed.06:07
wgrantStevenK: Yes, there is that one test which fails only in Windmill :(06:07
wgrantlifeless: It is unused, but there are QA instructions in the bug. I will grab spm.06:08
=== al-maisan is now known as almaisan-away
=== almaisan-away is now known as al-maisan
jtvwgrant: thanks for checking out the latest generate-contents-files... did it all work then?06:43
wgrantjtv: Yup!06:44
wgrantAll looks good.06:44
jtvWhat a relief.06:45
jtvwgrant: maybe we should just schedule that bugger then06:47
jtvah no, downtime06:47
jtv:(06:51
wgrantJa.06:51
jtvHmm that means about the same thing in all 3 Indo-European languages I can think of that have that word, so I get your drift.06:52
jtvIn Spanish, however, it could be interpreted as you laughing at me which would be less than kind.06:53
wgrant:(07:00
jtvBy the way, wgrant, isn't this a holiday for you?07:01
jtvGrrr this edge connection isn't very suitable for actual work.  Lightning seems to have fried my dsl modem because I was too lazy or worried about signal quality to loop the phone line through the UPS.  I'll try once more and if not successful, get a new one.07:01
wgrantjtv: No, not a holiday here.07:02
wgrantShould it be?07:02
jtvLabour Day.07:02
wgrantIn the UK, maybe.07:02
wgrantOurs was in March.07:02
wgrantNSW's is in October.07:02
wgrantQLD is today, though.07:03
jtvHere, certainly.  The reasoning seems to be that there must be a day without labour specifically for the occasion of labour day, and just having the day off because it's Sunday isn't enough.07:03
StevenKIt's "May Day" in the UK, isn't it?07:04
jtvQuite possibly.07:04
StevenKNorthern Terrority also have a 'May Day' public holiday.07:05
jtvMay Day is when pilots stop working and just let their planes drop to the ground, right?07:05
wgrantNT doesn't exist.07:05
StevenKwgrant: Wha?07:07
jtvThis could be bad news for any inhabitants the NT may have.07:08
jtvMeanwhile, how can it be a national holiday for Graham and Julian but not for Gavin?07:09
wgrantIt has a population of like 200000. It doesn't exist.07:09
wgrantjtv: Swap day?07:09
wgrantMaybe.07:09
LPCIBotProject windmill-devel build #1: FAILURE in 1 hr 4 min: https://lpci.wedontsleep.org/job/windmill-devel/1/07:09
jtvwgrant: no, the list I'm looking at would still show a national holiday.07:09
StevenKwgrant: By your reckoning, the small towns dotted around Australia don't exist either.07:10
wgrantPerhaps he just forgot.07:10
wgrantStevenK: They're not a whole semi-state.07:10
StevenKIt's a terriority!07:10
wgrantRight, it's like a state except more pathetic.07:10
StevenKWe should be trying to emulate them! They have less politicans.07:11
wgrantTrue, true.07:11
StevenKAnd windmill-devel failed, surprise.07:14
=== al-maisan is now known as almaisan-away
jtvStevenK: I'm looking into the "synchronize updates" button for the packages diff page.  Do we have code landed to synchronize a package?08:10
StevenKThe code has existed for ... eons08:11
spmÆons?08:12
lifelessnight all;08:13
jtvStevenK: damn, connection bounced again... where does this code live?08:14
LPCIBotProject windmill-db-devel build #228: STILL FAILING in 1 hr 5 min: https://lpci.wedontsleep.org/job/windmill-db-devel/228/08:14
* jtv traces through the call chain08:15
StevenKjtv: I'm not what the button calls, but likely IArchive.syncSources()08:16
jtvAh, I thought that was a recent development.  Thanks.08:16
stubWhat review type is for a release critical? rc or release-critical or what?08:42
StevenKrelease-critical08:43
StevenKI think08:43
stubhttps://code.launchpad.net/~stub/launchpad/pending-db-changes/+merge/59640 if the RM is in the house. I'm out of here to enjoy this holiday :)08:46
StevenKHm. We don't have RMs anymore?08:49
wgrantlifeless is the perma-RM.08:49
StevenKHeh08:52
adeuringgood morning08:52
rvbaHi wgrant. If you get a chance, would you take a look at https://code.launchpad.net/~rvb/launchpad/change-perm-sync/+merge/59341 ? I've modified the packages copier method carefully but since the modification is rather sensitive, I'd be happy if you could take a look at this. You will see that I've solved the problem about passing a None person to the copier method by avoiding this case with code in the view code. Please also take a l08:57
rvbaook at adeuring's remark in the MP's comments.08:57
wgrantrvba: Looking.09:03
rvbawgrant: great, thank you.09:03
wgrant405+        :param strict_component: True if access to the specific component for09:05
wgrant406+            the package is needed to upload to it. If False, then access to09:05
wgrant407+            any package will do.09:06
wgrantAccess to any *component* will do.09:06
rvbawgrant: right ... so I'll have to change the doc in the other method as well (where it came from)09:06
wgrantAh. Indeed.09:06
wgrantrvba: Hmm, so anonymous users are prevented from doing everything by line 19 of the diff?09:07
rvbawgrant: yes, this protects the call to sync_source.09:08
wgrantrvba: That's a bit riskier than I'd like :/09:09
rvbawgrant: yeah, I confess I'm not really happy with that either.09:09
wgrantrvba: Could you make checkCopy take a check_permissions arg in addition to the person?09:09
rvbaand set this to False by default?09:09
wgrantNo, True. But having check_permissions=True and person=None will forbid the copy.09:10
wgrantYou will have to change the existing callsites to use check_permissions=False, but then at least the dangerous action is explicit.09:11
wgrantAnd this is the very definition of dangerous action.09:11
rvbaok, I've done it this way to avoid having to change the call sites ... but your solution is bar better ...09:11
wgrantIf there is one security hole we don't want in Launchpad, it's probably in this method.09:12
rvbaall right. I'll do this.09:13
wgrantThanks.09:13
rvbawgrant: Thank _you_ ;)09:13
=== henninge changed the topic of #launchpad-dev to: PQM in RC mode | https:/​/​dev.launchpad.net/​ | On call reviewer: henninge | https:/​/​code.launchpad.net/​launchpad-project/​+activereviews
=== almaisan-away is now known as al-maisan
MonkZhi together, I want to setup a intranet launchpad is there a howto? with public/private projects, account management etc09:56
henningeMonkZ1: I am not aware of such a howto. It certainly is not supported "out of the box" but will require some work.10:15
henningeMonkZ1: I know that others have done it but I don't know how sucessful they were.10:16
MonkZ1henninge: there is no need for "out of the box" (mach ich gern). just need a light guideline10:19
henningeMonkZ1: I am not aware of anthing like that, at least not on offcial LP pages.10:21
=== al-maisan is now known as almaisan-away
MonkZ1henninge: ok thanks10:23
=== almaisan-away is now known as al-maisan
MonkZ1henninge: the "private project"-feature is free available?10:30
henningeMonkZ1: AFAIK we have private bugs, private branches and private teams.10:30
MonkZ1henninge: ok nice10:30
henningeMonkZ1: these are available to commercial subscriptions.10:31
MonkZ1:(10:31
henningeMonkZ1: https://launchpad.net/+tour/join-launchpad#commercial10:32
MonkZ1henninge: so i need to subscribe to get the possibility of a private * in a intranet environment on a selfhosted launchpad?10:33
henningeMonkZ1: no10:33
henningeMonkZ1: sorry, misunderstood you10:33
henningeMonkZ1: you can use your own instance as you like it.10:34
henningeMonkZ1: you get the exact same source tree that Launchpad.net is using.10:34
henningeMonkZ1: so all these features are available.10:34
MonkZ1henninge: aaah k this was the bit i need10:34
MonkZ1henninge: thx10:35
henningebitte schön ;)10:35
MonkZ1henninge: ach dann hätten wir auch auch deutsch quatschen können ;)10:35
MonkZ1bye10:36
henningeMonkZ1: no, this is an English language channel ... :-P10:40
henningeTschüß10:40
wgrantjtv: So ArchiveDependency would have a series at both ends?11:59
jtvwgrant: hadn't thought of that--it'd be a bit of a misnomer then wouldn't it?12:17
jtvThen again, with component & pocket already in there...12:17
wgrantjtv: You propose it has at least one series, though?12:18
wgrantAnd if there is one there must surely be two.12:18
jtvThen yes.12:19
wgrantOtherwise how would it work?12:19
wgrantArchiveDependency on Ubuntu Natty... for which series?12:19
jtvYes, I see that now.12:20
wgrantGreat.12:20
wgrantThen I mostly agree with your idea of merging them.12:20
jtvPhew.  I thought that was going to be a stumbling block.  :)12:21
wgrantI haven't run through all the cases yet, but it seems like it should work...12:21
jtvwgrant: the operative question remains of course: would it make our lives any better?  There's probably enough flexibility in that schema to cause us a few headaches.12:29
wgrantjtv: I'm not sure it adds significant complication.12:31
wgrantIt mostly makes things simpler.12:31
wgrantAnd a little less inflexible.12:31
jtvVery, very relieved to hear that.12:31
jtvApart from the fact that it'd take actual work to implement, of course.12:32
wgrantI'm trying to think how the implicit primary archive dependency would work.12:33
wgrantI guess it would be implicit per-series.12:33
jtvWhat are the problems with making it explicit?12:33
wgrantie. if there's not an explicit one for that series and the primary archive.12:33
wgrantWell, maintaining that automatically is difficult.12:34
wgrantAdded a new series to Ubuntu? Need to add 20000 new ArchiveDependency rows.12:34
jtvThat is rather a lot, yes.  What are all those archives?12:35
wgrantPPAs.12:35
wgrantLots of PPAs.12:35
jtvPart of the idea though is that for the general case, an archive dependency doesn't _have_ to be restricted to a series.12:37
jtvThat's where the headaches come from.12:37
wgrantRight.12:37
jtvBut it might just help avoid that per-series cost.12:37
wgrantMy original proposal or yours will both work fine. Yours is probably a bit cleaner. But maybe more expensive.12:39
wgrantBut...; ArchiveDependency.component and ArchiveDependency.pocket really need to be series-specific anyway.12:39
wgrantSo we might as well merge AD and DSP and solve everything.12:39
jtvhenninge: saw the request to open Oneiric translations?  I'm on feature rotation and rather busy... are you on maintenance by any chance?12:45
wgrantWhere does that go? And what's involved?12:45
StevenKwgrant: I'd suggest you reply to jtv's mail with how to merge AD and DSP?12:45
wgrantStevenK: I need to think about it.12:46
wgrantThe trivial solution is sort of obvious.12:46
wgrantBut it's not clear that it's perfect.12:46
jtvwgrant: you're asking about the translations opening?  It's a matter of running a script, much like init-from-parent, and it should be automated similarly.12:47
StevenKwgrant: My thought was DSP at *least* needed to grow pocket12:47
wgrantStevenK: Yes.12:47
wgrantStevenK: Julian disagrees.12:47
StevenKOr we create a Series table that maps all this crap12:47
wgrantBut he's not here today, so stuff him :)12:47
wgrantjtv: Right.12:47
wgrantjtv: It's fairly cheap nowadays with sharing and stuff?12:47
jtvy12:48
wgrantjtv: Which list/address was notified?12:48
StevenKwgrant: Okay, will you at least agree to replying with "Oh, here's this trivial solution I thought of."12:49
jtvwgrant: the former translations team still has a mail alias12:49
wgrantStevenK: Add two series fkeys to ArchiveDependency. Adjust code to cope. Done!12:49
wgrant(delete DistroSeriesParent too)12:50
wgrant(bonus points for making DSD point to an ArchiveDependency)12:50
wgrantjtv: Ah. That's a bit sad.12:50
StevenKwgrant: Bah, we just added it12:50
wgrantStevenK: Yes, and that's why I argued about it.12:50
wgrantBecause it was premature and ill-thought.12:51
StevenKAnd I wrote a 2,000 line branch to *use it*12:51
wgrantSure. But it's mostly reusable.12:51
wgrantJust need to change the class it looks at.12:51
wgrantIt's the ArchiveDependency code that needs to change significantly.12:51
wgrantDSP-using code just has to use the other class.12:51
jtvwgrant: I'm surprised to hear we were thinking along such similar lines all the time.12:53
jtv(including the "premature and ill-thought" bit)12:53
wgrantHeh.12:53
wgrantYour combined solution is better than the one I had come up with.12:54
wgrantBut I knew the existing one was entirely inadequate :)12:54
wgrantThis is why I like non-trivial discussion before making large changes :)12:56
=== benji changed the topic of #launchpad-dev to: PQM in RC mode | https://dev.launchpad.net/ | On call reviewer: henninge, benji | https://code.launchpad.net/launchpad-project/+activereviews
henningejtv: I was pinging you to ask you about that ... ;-)13:00
jtvAh!13:00
henningejtv: do we have it documented somewhere?13:01
jtvErrr yes somewhere I think...13:01
wgrantHow hard is it?13:01
wgrantCan NRCP just say to poke a LOSA to do it?13:01
jtvwgrant: not hard at all.13:01
jtvIt used to be a big deal where lots of things could go wrong and bugs would surface with merciless savagery.13:02
wgrantYeah.13:02
jtvBut nowadays, yes, a LOSA could do it or better yet, a job runner.13:02
wgrantAnd you had to take down LP for 12 hours.13:02
wgrantThose were the good old days.13:02
jtv12 hours?  you kid had it easy13:03
henningejtv: also, I am wondering if "Mail rosetta@..." is still a good instruction for the Ubuntu guys.13:03
jtvhenninge: we were just talking about that--you're right, it isn't.13:03
henningeright13:04
wgrantI have my cursor on the NRCP edit link :)13:04
wgrantIs there a LOSA doc I can link to?13:05
jtvIt should be a simple matter of running scripts/copy-translations-from-parent.py -d ubuntu -s <new series> and, once it's done, enabling imports for the series.13:05
wgrantNothing under LPHowTo like there is for branch-distro?13:05
henningejtv, wgrant: I can see to it that it happens this time around and wirte up a doc for it.13:07
jtvI'm searching the wikis atm13:07
henninge(which would mainly include those two instructions)13:07
henningejtv: cool. I have to relocate right now. Please let me know what you find ... ;-)13:08
jtvOK13:08
wgrantjtv: https://answers.launchpad.net/launchpad/+question/154984 <- is that just a matter of changing the series on +admin?13:16
jtvwgrant: the great irony of the translations opening is that the code relies on DistroSeries.parent_series.13:17
jtvwgrant: looking...13:17
wgrantjtv: Of course :)13:17
huwshimiAnyone around that might be able to help me figure out why I can't get Launchpad running locally?13:38
wgranthuwshimi: What does it do?13:42
wgrantrvba: What's this -updates thing?13:42
huwshimiwgrant: I can't make a connection between the browser and the local server. At least that's what I suspect.13:42
huwshimiwgrant: launchpad.dev just times out13:43
rvbawgrant: the card says exaclty this "Synching to a released series should put packages in -updates" ... but I haven't talked to anyone yet (and Julian is only back tomorrow)13:43
huwshimiwgrant: and the server does not log any events13:43
rvbawgrant: https://code.launchpad.net/~rvb/launchpad/sync-to-updates/+merge/5965313:43
wgranthuwshimi: ping launchpad.dev13:43
wgrantWhich address is it trying?13:43
wgrantrvba: I saw the diff, and I think the idea of hardcoding it like that is silly.13:43
wgrantIt is probably what Julian wants.13:44
wgrantBut it is still silly.13:44
rvbawgrant: :) I'll be glad to hear about any improvement you would care to suggest.13:44
huwshimiwgrant: Ah, it's trying to use my ip address at home13:44
huwshimiwgrant: I am a long way from that router :)13:45
huwshimiwgrant: I should have noticed that before13:45
rvbawgrant: not that the destination pocket was previously hardcoded to "release".13:45
rvbas/not/note/13:46
=== Ursinha-afk is now known as Ursinha\
=== Ursinha\ is now known as Ursinha
wgrantrvba: Right. We need to think how pockets are going to be handled.13:51
wgrantrvba: At the moment we handle !Release for source or destination. Hardcoding this is probably not the solution.13:51
wgrantEr.13:51
wgrantWe *don't* handle !Release for source or destination.13:51
lifelessStevenK: wgrant: I'm not RM; RM role is dissolved.13:55
wgrantlifeless: But you are the effective RM, aren't you?13:55
lifelessStevenK: wgrant: rc exceptions can be handed out by reviewers as long as it *is* rc (that is we can't deploy without it)13:55
wgrantOh. I didn't hear that.13:55
lifelesswgrant: I don't have any permission bits that I'm aware of that haven't been delegated out to be team responsibilities13:56
rvbawgrant: what do you mean exactly "we don't *handle* release for source or destination" ...?13:56
lifelesswgrant: I'll be closely involved, this is our single highest risk activity13:56
jcsackettrc also means less than it used to, since most things can go out shortly after.13:56
lifelesswgrant: did you get spm cycles for qastaging ?13:57
wgrantrvba: Different distros will want to do different things with pockets. At the moment we are hackishly hardcoded to ignore -backports and -proposed when generating DSDs. Now we're adding more hacks to map release into -updates when something has released. This is just going to become an ever-increasing stack of hacks.13:58
lifelessugh13:58
wgrantlifeless: For which? The questions QA?13:58
lifelesshow do you turn off 'desktop integration' for launchpadlib13:58
rvbawgrant: I see.13:58
lifelesswgrant: applying the db patch to qastaging13:59
wgrantlifeless: No :/13:59
wgrantlifeless: I'll grab $US_LOSA.13:59
wgrantWhen they appear.13:59
lifelesscool13:59
deryckMorning, all.14:00
lifelessso, desktop integration. How does one turn that off?14:00
deryckhenninge, standup ping14:00
lifeless[I don't trust every python script with private access to lp]14:00
rvbawgrant: so a possibility would be to abstract the behaviour of things related to pockets (as much as possible ... and especially new things) so that whenever this pocket-thing get cleaned up properly we don't have to modify things everywhere ... correct?14:01
wgrantlifeless: Well, you have POSIX to argue with about that..14:01
benjilifeless: take a look at https://dev.launchpad.net/LandingChanges#Credential%20storage; there I describe how to force the file-based crendential storage14:01
lifelesscool14:01
wgrantrvba: Right, the ArchiveDependency rework we discussed earlier could solve this.14:01
lifelessI need to hunt down whomever wanted this carte blanche approach and determine their real needs14:02
lifelessso we can make it sane14:02
wgrantlifeless: Or do you just not trust them to not do things out of stupidity, not malice?14:02
lifelessstupidity mainly14:02
lifelessI want to know when client X is going to futz with my not-journaled webapp thank-you-very-much14:02
wgrantMaybe one day our operating systems won't suck and we can run potentially malicious code.14:03
lifeless(its why I tend not to use fat clients for most web services I use)14:03
wgrantlifeless: All protests on this front were ignored.14:04
wgrantlifeless: Well, dismissed with cries of approximately "DX! DX!"14:04
wgrantIIRC14:04
lifelessbenji: thats file based, but does it restore the previous *one file per app* logic14:04
lifelessbenji: because thats what I want; I want each /app/ to get oked14:04
wgrantlifeless: bwahaha.14:04
wgrantbwahahahah.14:04
wgrantYou'll have to hack the code.14:04
rvbawgrant: Yeah ... but the ArchiveDependency rework will not happen tomorrow I'm afraid so we will have to figure out a way to deal with this pocket related work in a way that will make the future work easy (well, not to painful let's say ;)).14:04
wgrantAnd undeprecate the APIs.14:04
lifelessbenji: think android14:04
wgrantrvba: Why does -updates have to be dealt with now?14:05
lifelesswgrant: we may have time for oni; the big thing is finding out *who* drove it.14:05
benjilifeless: nope, it's still one file for all; the app can force it's own store, but off the top of my head I don't know of a way for the user to do it14:05
rvbawgrant: no other reason than this being a task on the kb board with high priority14:05
lifelessEvery time I spoke to a dev about it, the requirements were coming from some unspecified 'users'.14:05
benjihmm, there might be a way, let me look14:05
lifelessbenji: thanks!14:06
lifelessI'm crashing, but if you find something, I should see it in backlog - or you could drop me a pointer in mail if you felt so inclined14:06
wgrantrvba: Looks like Priority: Normal to me. But hmm.14:07
lifelessnight all14:07
wgrantNight lifeless.14:07
rvbawgrant: you're right ;)14:08
wgrantAnyway, we have Julian back tomorrow.14:09
wgrantIt looks like the hack is done.14:09
wgrantSo if he wants it it's there.14:09
rvbawgrant: sure ... but the hack itself is a one liner (sort of) so I'm very much in favour of making this more clean ... especially since, like you said, this whole pocket stuff will have to be reworked.14:10
wgrantrvba: It's not critical that this is fixed now, and we may have a proper solution soon. So I'd be a little averse to landing more hacks.14:11
rvbawgrant: I completely understand ... and I agree.14:11
wgrantIf Ubuntu was complaining about this *now*, sure :)14:11
wgrantThat would be the Soyuz Way™.14:12
wgrantBut that is fortunately not the case right now.14:12
henningeadeuring: https://bugs.launchpad.net/ubuntu/+source/wxwidgets2.8/+bug/55982214:12
_mup_Bug #559822: editra is provided by both the editra package and python-wxtools and conflicts <apport-package> <i386> <lucid> <verification-done> <editra (Ubuntu):Fix Released> <wxwidgets2.8 (Ubuntu):Fix Released by d.filoni> <editra (Ubuntu Lucid):Fix Released> <wxwidgets2.8 (Ubuntu Lucid):Fix Released> < https://launchpad.net/bugs/559822 >14:12
jcsackettwgrant: isn't the Soyuz Way mor of a patent-pending thing? :-P14:12
rvbaalthough I'm very much in favour of figuring out a way to do things and preparing for the future at the same time. if that's possible that it :).14:13
wgrantjcsackett: Indeed.14:13
rvbawgrant: thanks for keeping an eye on my doings in the Soyuz code base though ;)14:14
wgrantrvba: Sorry about this, but I want to try to minimise the piles of hacks on top of piles of hacks that are already happening :(14:15
wgrant(and have been happening forver)14:15
rvbawgrant: no irony here ... I was about to ask you about this14:15
rvbawgrant: "Started to work on new task: syncing packages in a released series should put packages in the Updates pocket.  Will discuss with someone knowledgeable about Soyuz. "14:15
wgrantHeh14:16
rvbawgrant: from this morning call notes ... see ;)14:16
rvbaanyway, I'll do something else now ... and land the other work we've been talking about ...14:16
* rvba adds wgrant to *the* list14:17
rvba(of the people to whom I owe a beer)14:17
wgrantFortunately in lots of places you won't be able to fulfil that :P14:18
rvbaI reckon in Irland it's ok.14:21
StevenKrvba: wgrant doesn't drink. Perhaps you'll need to settle for warm milk or something.14:34
wgrantI do... just not where it's US-illegal :)14:35
rvbaStevenK: wgrant I'm sure we will find a way to settle this :P14:35
wgrantHeh.14:35
StevenKwgrant: Actually, there is no restriction in Ireland, as long as you have guardian permission, so if you ask your mother, you should be good. :-P14:37
=== salgado is now known as salgado-lunch
benjihenninge: I claimed https://code.launchpad.net/~sinzui/launchpad/question-email-3/+merge/59574 but wanted to be sure you weren't working on it.15:01
henningebenji: I am not but thanks for keeping me informed. ;)15:02
benjithanks15:02
sinzuibenji: I was going to beg jcsackett to review that since it contains a 900 line fix to doctest that required a lot fixes to preserve the tests since the infrastructure changed.15:12
benjisinzui: oh, ok15:12
jcsackettbenji: i consider myself begged. i'll take a look at it in a moment.15:13
jcsacketter, actually, ^ sinui15:13
jcsackett...i fail at typing.15:13
sinzuibenji: The real change is very small.15:13
benjiyeah, it looked like mostly test fallout15:13
LPCIBotProject windmill-db-devel build #229: STILL FAILING in 1 hr 7 min: https://lpci.wedontsleep.org/job/windmill-db-devel/229/15:24
jcsackettsinzui: looking at your branch now.15:24
jcsackettsinzui: approved, with a minor quibble on some classnames (as listed in my comment).15:40
sinzuiokay thanks15:40
jcsackettsinzui: i am very excited to have all of this landed. :-)15:40
sinzuijcsackett: testing the the previous branch shows two failures. One I already have a fix for, and one very preplexing...15:41
sinzuiAt least I know I wrote a very good test for qastaging15:41
adeuringsinzui: do you see bad side effects if the result of Person.getAdministratedTeams()  would be cached? (too many call of this method are the cause of bug 768443)15:42
_mup_Bug #768443: ProductSeries:+index timeout with many releases <timeout> <Launchpad itself:In Progress by adeuring> < https://launchpad.net/bugs/768443 >15:42
sinzuiwell15:42
adeuringthere are other places where we could cache the result15:43
sinzuiadeuring: We want to remove that method. It forces the team owner to be a member of the team. I believe there are two call sites last month. One that can easily be factored out and one that looked tricky15:43
sinzuiadeuring: I think callsites should be using team.adminmembers15:44
=== al-maisan is now known as almaisan-away
adeuringsinzui: ah, that would be another option. thanks!15:44
sinzuiadeuring: mailing lists has one call site. It can switch to team.adminmembers. It has a test that says the owner can configure the list...we can remove that15:45
* sinzui looks for the other callsite15:45
adeuringsinzui: that's in lp.bugs.browser.structuralsubscription15:46
sinzuiadeuring: sorry. I am wrong. I misread the method15:46
* sinzui reads the code and sips coffee15:46
sinzuiadeuring: I am +1 to add caching to getAdministratedTeams. I believe the only oddness we need to think about is when the user in the interaction is was added/removed to team.owner either directly or indirectly. This same user cannot add himself, but he can remove himself. Since the operation to change yourself is separate from where we show that list. I think there is 0% chance of caching oddness today15:50
adeuringsinzui: yeah, stuff like is what concerns me too. I'll look a bit closer at thie issue15:51
sinzuiadeuring: Subscriptions are only place that list your teams. I believe there is a bug reported that it is not possible to see and change the teams you admin. I think the person so fixes that bug will need to clear the cache after the change operation.15:52
adeuringright15:54
adeuringsinzui: there is alreay a method Person.clearInTeamCache() It can and should clear the new cache too, I think16:13
sinzuiadeuring: may not. I am not a member of may teams I own16:14
adeuringah, right. so I'll ignore possbile problems with the cache for now -- could not find any real issues..16:15
sinzuiadeuring: InTeam() is broken because it forces the team.owner to be a member. It should not. So our choices are to add temporarily add the cache clear rule, but we would want to remove it. inTeam() should instead be changed to not include the team owner.16:16
adeuringsinzui: maybe. But that's more than I want to change for this bug ;)16:17
=== salgado-lunch is now known as salgado
jcsackettis anyone else having issues with "make run" ? http://pastebin.ubuntu.com/602346/16:57
jcsackettah, looks like memcached is gone post-update.17:01
abentleybenji: Could you please review https://code.launchpad.net/~abentley/launchpad/nonascii-email/+merge/59671 ?18:33
benjiabentley: gladly18:33
benjiabentley: done with https://code.launchpad.net/~abentley/launchpad/nonascii-email/+merge/5967118:47
abentleybenji: thanks.18:48
benjinp18:48
* jcsackett is down one machine.18:56
sinzui\o/ NotifcationRecipientSet.remove() has always been broken19:03
* sinzui prepared an apocalyptic fix.19:05
dobeysinzui: is that like cooking heroin from brimstone and hellfire?19:30
sinzuidobey: eek19:31
sinzuidobey: apocalypse  is a tag meaning code code and tests need to moved to where people can find them. The two were separated for years so it is very difficult to discover the code has always been broken19:32
jcsackettso named, if i recall, b/c the people doing the fixing were nicknamed the four horsemen.19:33
jcsackettwhich invites the question, how is your horse, sinzui?19:34
dobeysinzui: it was just your choice of words :)19:35
sinzuihmm, well that has meta insinuations as well19:36
sinzuijcsackett: my wife was muling laundry down the steps this morning and fell. A child had left shoes on the steps. We pondered going to the emergency room19:37
sinzuijcsackett: so I ran up stairs where I stepped on a shovel my wife left on the steps, puncturing my foot.19:38
jcsackettsinzui: good lord! are you too relatively okay?19:38
jcsacketts/you too/you two/19:38
sinzuijcsackett: My wife tells me she had no idea I would be running up the steps to help her; she claims left the shovel on the steps as a trap for the children.19:39
sinzuiShe is making my open the jars and I am limping a bit19:39
jcsackettsinzui, you frequently take the cake for anti-charmed life.19:42
jcsackettand i thought nuking my macbook today was a bad day.19:42
dobeyheh19:42
sinzuijcsackett: I think I can top than. On Easter day I drove my children to their grand parents. I noticed my speedometer did not work. We made jokes about how we should get their in no time at zero speed. I also noted that the the miles we were traveling were not ticking away. Near our destination the car started to chug and shake.  I had run out of fuel, though the gage was at 50%. I marched my well-dressed children to a co19:47
sinzuirner by the road to wait for a relatives to find us. ;)19:47
* jcsackett can no longer tell when sinzui is joking19:47
=== m4n1sh_ is now known as m4n1sh
sinzuijcsackett: I think mundane misfortunes are best expressed as humour. I make them bearable.19:52
jcsackettsinzui: fair.19:52
=== m4n1sh_ is now known as m4n1sh
lifelessmorning20:33
jcsackettwhat's the magic to run a javascript test? the usual "bin/test -t pattern" isn't doing it.20:46
abentleyjcsackett: You open the html file in Firefox.20:50
jcsackettabentley: huh. okay.20:53
jcsackettabentley: so do we just have some plumbing to open all of those via windmill or something on ec2?20:53
abentleyjcsackett: yes, there's a particular layer you can run.  I forget its name, but it was announced in launchpad-dev in the last month or so.20:54
jcsackettabentley: okay, thanks.20:54
flacostelifeless: https://code.launchpad.net/~flacoste/launchpad/ppr-enhancements/+merge/5970021:36
flacosteif you care to review my collected enhancements to the ppr21:36
flacosteit's a big branch (844 lines), turned out that the trickiest part was changing the histogram resolution :-)21:37
sinzuijcsackett: do you have time to mumble?21:53
jcsackettsinzui: sure, just one moment.21:54
jcsackettsinzui: i think the problem is on my end.21:56
jcsacketti have a number of things failing today.21:56
sinzuijcsackett: I just restarted since I saw you speaking but heard nothing21:57
lifelessflacoste: hi21:57
lifelessvoip/skype?21:58
flacostelifeless: voip is good21:58
lifelessext #?21:59
flacoste735621:59
lifelesscurrently unavailable21:59
flacostelifeless: try again22:00
jcsackettsinzui:i have gone through the wizard again and appear to have a functioning microphone.22:01
jcsackettsinzui: hm. i'm guessing you still do not hear me?22:01
jcsackett...sound settings have never been right on this machine...22:02
sinzuiI do not, but I hear the pings from this app and music22:02
jcsackettsinzui: yeah, problem is on my end, i'm sure.22:02
jcsackettsinzui: give me a few more moments, and if we cannot get it working, just call me?22:02
sinzuijcsackett: does sound preferences show the mic working22:02
sinzui...the input tab22:03
jcsackettsinzui: it does. i ran through the wizard and had functioning sound. :-/22:03
jcsacketter, microphone.22:03
sinzuiI do not use the wizard. It died when I tried to configure it. I guess it did what it needed to. I use the pulse sound preferences panel that is found in the sound indictor.22:04
* sinzui sees mumble and can adjust the volume/input of speakers and mic22:04
sinzuijcsackett: I see your status changing, but I no longer see an indication that you are speaking.22:06
sinzuimaybe you can use deafen to send a morse coded message22:07
lifelessmatsubara: hi; I have an oops-tools patch, how do I land it?22:11
=== benji changed the topic of #launchpad-dev to: PQM in RC mode | https://dev.launchpad.net/ | On call reviewer: - | https://code.launchpad.net/launchpad-project/+activereviews
matsubaralifeless, you need to set a commit message and set the mp as approved22:22
sinzuijcsackett: https://launchpad.net/~mactel-support/+archive/ppa22:34
=== matsubara is now known as matsubara-afk
sinzuijcsackett: https://help.ubuntu.com/community/MacBook5-1/Lucid22:37
sinzuihttps://help.ubuntu.com/community/MacBook5-1/Natty22:37
lifelessflacoste: http://developers.facebook.com/blog/post/358/22:40
sinzuijcsackett: the brightness instruction on the natty page does indeed fix bightness22:56
jcsackettsinzui: that is awesome.22:57

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