/srv/irclogs.ubuntu.com/2012/05/01/#launchpad-dev.txt

wallyworld_ah right, glad we found out how it was happening00:00
StevenKLet me toss that ec2 and then forage for some breakfast.00:01
RyuGunsHey guys.01:04
RyuGunsI've been in the process of registering a new account...01:05
RyuGunsBut I am not allowed to.,.01:06
RyuGunsWait, actually, nevermind.01:06
RyuGunsHow do I merge accounts?01:07
bobweaverRyuGuns,  https://help.launchpad.net/YourAccount/Merging01:07
bobweaver:)01:07
RyuGuns:)01:08
RyuGunsFunk.01:24
RyuGunsHow do I change my ID?01:24
RyuGunsNevermind.01:24
lifelessRyuGuns: bobweaver: you might prefer #launchpad for customer support01:54
RyuGunshttps://launchpad.net/ubuntu+mobile+phone01:56
RyuGunsAnyone interested?01:56
StevenKwgrant: Free to review, or still distracted by *redacted*?05:16
StevenKAnd no wallyworld.05:25
StevenKMaybe bigjools' tech has broken TPG.05:25
wgrantStevenK: 'sup?05:25
StevenKwgrant: https://code.launchpad.net/~stevenk/launchpad/hack-itemwidget/+merge/10420205:25
wgrantoh god05:26
wgrantStevenK: Why are you setting term.name now?05:26
StevenKwgrant: Because the test in test_itemswidgets requires it05:27
StevenKAnd I'd rather just use self.assertRenderItem05:27
wgrantStevenK: r=me before lp melts again05:30
StevenKHaha05:30
wallyworldwgrant: StevenK: have you seen bug emails with the same info twice? eg "** Visibility changed to: Private" is listed twice in the one email06:59
wallyworldi seem to recall i may have seen this but am not sure07:00
StevenKI can't recall that one, personally.07:00
StevenKI a note to track down why information type: Public -> Private turns up twice when you change the information type via JS.07:01
wallyworldStevenK: that's what i'm seeing also but i'm fairly sure it happened before the info type field was introduced07:02
wgrantI don't recall seeing that before.07:09
wgrantI would suspect a regression.07:09
wallyworldwgrant: i seem to recall it may have happened for eg "** Visibility changed to: Private" which was before disclosure07:29
wallyworldbut i can't be 100% sure07:29
wgrantThat stuff was dangerously refactored a few months ago07:30
wgrantIt may have been a regression then.07:30
=== wgrant changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: - | Firefighting: - | Critical bugs: 3.47*10^2
czajkowskialoha08:14
jmlczajkowski: hi08:23
czajkowskimorning08:24
jmlok.08:40
jmlso I'm going to do some Launchpad hacking.08:40
jmlstand back.08:40
wgrantUhoh08:40
jmldoes it work in precise yet?08:40
wgrantWe're doomed08:40
wgrantUm08:40
wgrantWell08:40
wgrantMaybe.08:40
wgrantYou'll need python2.608:41
jmlI have Python 2.6!08:41
wgrantIt might Just Work™, then.08:41
wgrantI fixed the other issue yesterday.08:41
wgrantBut haven't retried since.08:41
czajkowskiwgrant: was sure that was one of the resonas why gmb was setting up an Ec2 for uds lp clinic was it not working on precise just yet08:43
jmlKeyError: 'STORM_CEXTENSIONS'08:43
wgrantDoing what?08:44
wgrantHow old is the tree?08:44
jmlI just pulled08:44
jmlbut I've forgotten everything I need to do to update08:44
wgrantmake clean build, to be safe08:44
jmlah, ok.08:44
* wgrant tries.08:45
jmlError: pg_config executable not found.08:51
jmlI get that in the middle of a 'make build'08:51
wgrantDo you have launchpad-developer-dependencies installed?08:53
wgrantI shall return after dinner. Hopefully germanium will give me everything I need while I eat.08:54
jmlwgrant: heh08:56
jmlwgrant: ta08:56
jmlwgrant: no, I don't. It seems to be broken.08:56
jmlah, you need python2.6-dev09:03
jmlI've just got python2.6 from https://launchpad.net/~fkrull/+archive/deadsnakes09:04
wgrantjml: Yeah, I'll probably replace it with python-all-dev and co. once I check it functions without 2.609:10
wgrantWe mostly just use the default version, but some things still point at 2.6, I believe.09:10
wgrantEverything of note has been updated to just use the default, so it will hopefully work.09:12
jmlcoole.09:13
jmlI'll keep hacking in my lxc for the moment.09:13
wgrantProbably better for your sanity anyway.09:13
jmlyeah.09:21
wgrantInteresting09:30
wgrantpg_createcluster now fails if it's run in a nonexistent locale.09:30
wgrantdpkg doesn't notice.09:30
wgrantlifeless: Any objections to dropping !precise host support from /Running/LXC?09:31
wgrantThe instructions are like half the size.09:31
lifeless+109:32
* wgrant mauls.09:32
wgrantjml: Just ran various tests successfully without 2.6, so I'll drop it from the deps.09:54
jmlwgrant: yay10:01
jmlwgrant: do you know why TestArchivePrivacySwitching is in LaunchpadZopelessLayer?10:28
wgrantjml: Probably because whoever wrote it was wrong.10:28
jmlwgrant: heh.10:29
wgrantjml: It should be ZopelessDatabaseLayer or DatabaseFunctionalLayer, since it probably doesn't need librarian etc.10:29
jmlwgrant: yeah, trying DFL now.10:29
wgrantThe difference between those two is now pretty much just the permission model.10:29
wgrantZopeless is still PermissiveSecurityPolicy10:29
jmlah cool.10:30
jmlwgrant: how much work does "pretty much" hide?10:31
wgrantjml: Working out what FunctionalTestSetup etc. does that is different.10:31
wgrantI merged most of it in a large branch series in September, but am yet to obtain the courage to complete the merge.10:32
wgrantBecauze Zope testing setup stuff is somewhat intimidating.10:32
wgrantIt's also not clear quite how it's best to do it.10:32
jmlyeah.10:33
wgrantBut I suspect an attribute on the test class to specify the security policy might work.10:33
wgrants/work/not be as terrible/10:33
jmlthat's what I was thinking10:33
jmland then one by one change the test classes to do something better.10:33
jmlI wonder if there's a better mechanism than attribute (subclassing maybe?)10:34
wgrantStupidLegacyTestCase? :)10:35
jmlwell, we need to separate concerns10:35
jmlStupidLegacyPermissiveTestCase10:35
wgrantHeh10:35
jmlactually, you could probably do something with testtools's `run_tests_with` thing.10:36
jml        publisher.prepareBreezyAutotest()10:36
wgrantThat's actually not a terrible idea.10:36
wgrantRun away10:36
jmloh yeah, that needs the librarian10:36
wgrantIt does, yeah.10:36
wgrantBecause inserting fake LFAs is hard, I guess.10:37
wgrantIt tries to upload chroots.10:37
wgrantIt could be taught not to participate in such stupidity, though.10:39
wgrantI'm not sure if our test suite's antics are back to being hilarious rather than depressing yet.10:40
jmlI'm guessing getPubSource has side-effects that belie its name10:45
wgrantOh yes10:45
wgrantmakePubSourcePlusSomeOtherBitsActually10:46
wgrantYou can usually get away with the factory's makeSourcePackagePublishingHistory these days10:46
wgrantSoyuzTestPublisher is deprecated and new usage of it incurs wrath.10:46
jmloh10:49
jmlthat's nice10:49
jmlISTR the last time I tried this sort of thing the accepted wisdom was "use SoyuzTestPublisher because you'll never be able to get it right any other way"10:49
jmlis there a safe way to use the Librarian in tests without using a layer that provides it?10:52
lifelessno10:59
lifelessthe layer is a thin shim on the fixture now10:59
wgrantjml: The appropriateness of STP depends on the test.10:59
lifelessbut you still need all the bits joined together10:59
wgrantYou can sometimes use the FakeLibrarian.10:59
wgrantTranslations uses it for something in like one test.11:00
jmlIf there were a safe way, then lots of test runs could be made a bit faster easily. I could, for example, switch this test to ZopelessDatabaseLayer and then just use that safe way to run this test with the librarian.11:02
lifelesszopeless is no cheaper than zopeful11:02
jmllifeless: I wasn't contending that point.11:03
jmlZopelessDatabaseLayer is heaps cheaper than LaunchpadZopelessLayer though.11:03
jmllibrarian, memcache and rabbit all take their toll.11:04
cjwatsonI don't seem to have much luck with FakeLibrarian, usually.  Maybe I'm not smart enough.11:09
cjwatsonI tend to get incomprehensible storm tracebacks.11:09
jmlbattery threat!11:25
=== rick_h_ changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: rick_h | Firefighting: - | Critical bugs: 3.47*10^2
jmlhttps://code.launchpad.net/~jml/launchpad/narrow-commercial-celebrity/+merge/104236 up for review.11:30
jmlrick_h_: poor chap, you always seem to end up with my sad offerings.11:30
rick_h_jml: hah, all good11:30
jmlI have to go find electricity. Will be back sooner than you'd like.11:30
czajkowskiand yet he comes back for more11:30
czajkowskicant be that bad11:30
rick_h_jml: ok, going to grab some grub, so will take me longer than you'd like :P11:31
jmlok. I'm back.12:00
jmlrick_h_: are you far into reviewing that branch, because I thought of some cleanups that I can tack on to it.12:02
czajkowskijml: we have date/times now for the LP clinic at uds https://wiki.ubuntu.com/UDS-Q/LaunchpadClinic12:02
rick_h_jml: I'm peeking at it, but please go ahead and update and I'll get back to it in 10 then12:03
jmlrick_h_: ok, finally done12:28
rick_h_jml, ok, working on another review atm, but will get to it shortly12:28
jmlrick_h_: 30s per test run has a surprisingly strong negative effect on speed.12:28
rick_h_jml: tell me about it :/12:29
wgrantjml: I'm pretty sure it hasn't become worse since you defected. I guess you've just repressed the tramautic memories :(12:30
jmlwgrant: oh, I'm sure that's the case.12:31
StevenKThe brain does repress bad memories.12:31
jmlotoh, the nice thing with working on LP is there's so much obvious stuff to do12:31
jmlit's very hard to get stuck.12:31
wgrantIndeed.12:31
wgrantIndeeeeeeed.12:31
wgrantSo much to delete.12:31
wgrantLike Translations, Answers, Blueprints.12:32
wgrantAnd Soyuz to rip out.12:32
jmlor update, or fix, or refactor.12:32
jmlyeah12:32
jmlripping out the buildmaster would be fun12:32
jmlstep 1, rationalize the transactions12:32
wgrantIt, Soyuz and Translations have no business at all being in LP core.12:32
wgrantThe others might have some justification.12:32
jmlstep 2, xmlrpc (or whatever) interface12:32
jmlstep 3, yoink!12:32
wgrantYep12:33
wgrantIt just turns into a process which bridges the LP and buildd XML-RPC interfaces.12:33
jmlchunks of lp.codehosting could be ripped out pretty easily too.12:33
wgrantI'm likely to rip out lp.services.sshserver into a separate project in the next few months.12:33
wgrantAs part of destroying poppy.12:33
wgrantIt's mostly self-contained.12:33
jmlwgrant: oh, I made lp:txsshserver once upon a time with roughly that intent12:33
jmlwgrant: maybe you should take over that project when you do it?12:33
wgrantMaybe indeed.12:34
jmlit has no users.12:34
wgrantAll the better!12:34
jmlhmm. maybe I killed it :\12:34
wgrantOne thing I do like about Code(hosting) is that the architecture isn't entirely stupid, unlike say most of the rest of LP.12:34
wgrantapart from branchrevision.12:34
jmlwgrant: aww shucks.12:35
=== matsubara-afk is now known as matsubara
=== almaisan-away is now known as al-maisan
=== al-maisan is now known as almaisan-away
jmlis the a graph of LP LoC over time?13:03
wgrant<http://www.ohloh.net/p/launchpad>, but cjwatson had a better one of the delta13:04
wgrantI'm not sure how accurate ohloh's is.13:04
rick_h_heh, wasn't ready to seem my name up there in that13:05
rick_h_but as a re=me so it's ok :)13:06
rick_h_jml: so you're not removing the concept of commercial ppas, but this single getCommercialPPA helper then?13:07
jmlrick_h_: that's right.13:07
rick_h_ok, cool13:08
StevenKohloh's one tends to be out-of-date and imprecise13:08
jmlrick_h_: if we remove the commercial concept entirely, well, this is half (hah!) the work13:08
rick_h_jml: just making sure I'm following, saw the sea of red but some bits left and wanted to make sure I was 'getting' it right13:08
jml:)13:08
StevenKjml: The Soyuz's team blood, sweat and tears (mostly tears) went into making IArchive.commercial!13:08
jmlStevenK: sunk cost fallacy.13:09
wgrantWoah13:09
jmlStevenK: Sorry, I meant to say that we really appreciate it and hope some day we can return the favour.13:09
wgrantNever seen someone brave enough to invoke that on LP before :)13:09
StevenKjml: Oh, I know it's a sunk cost, I'm just saying think of our tears. :-)13:09
rick_h_tears wash away easily with some soap and water...all good13:09
StevenKwgrant: You mean what I said?13:10
wgrantNo, the sunk cost fallacy.13:10
StevenKAh.13:10
wgrantIt could be applied to harmlessly remove 90% of our functionality :)13:10
rick_h_how dare you use logic and management terminology in here! :)13:10
* StevenK removes 90% of the JS, thereby removing the need for convoy or jsbuild.13:11
* rick_h_ watches the site go boom!13:11
StevenKIt already was earlier today.13:11
rick_h_phew, glad I missed it then13:12
wgrantProduction thrice, qastaging once.13:12
jmlwgrant: well, there you'd compare cost/benefit of deleting with cost/benefit of user migration13:12
wgrantFun day.13:12
rick_h_ugh, what went kaput?13:12
jmloh, also, perplexing thing: https://code.launchpad.net/~jml/txsshserver/trunk (200); https://code.launchpad.net/txsshserver (404)13:12
wgrantjml: Deactivated project, probably.13:13
jmlah ok13:13
wgrant"This project is currently inactive"13:13
jmlrick_h_: anyway, I don't have commit so if you approve of that branch please land it for me.13:13
rick_h_jml: ah ok will do13:13
rick_h_going to have to footnote you on my ec2 bill :P13:13
jmlrick_h_: I left two LP instances running accidentally, I think to support some pair programming. I have a $300+ bill from last month.13:14
rick_h_ouch!13:14
jmlwgrant: how come you are so low on ohloh?13:14
wgrantjml: Hm?13:14
rick_h_I'm constantly afraid of that. I have two of my own I keep running to have to keep thinking "hmmm, 2 ok, >2 doh!"13:15
sinzuiwallyworld, got a bill for $1,50013:15
sinzuiIt ran for 45 days13:15
rick_h_oooh...45 days?!13:15
wallyworldsinzui: for ec2?13:15
rick_h_that must have been pre the large instances?13:15
jmlwgrant: you don't appear on http://www.ohloh.net/p/launchpad/contributors13:15
sinzuiyes13:15
wallyworldah, my bill13:15
rick_h_I'd have thought 45 days would be higher13:15
wgrantjml: I'm probably on like the third page.13:15
* wgrant hunts.13:15
jmlwgrant: 2nd13:15
rick_h_when I see these start ups with 100s of ec2 servers I cringe at that monthly bill13:16
wgrantAh, not too bad.13:16
wgrantI am the top committer of the last 12 months, though :)13:16
StevenKI think I'm on like the 6th page. :-(13:16
sinzuiWe can all run starry-eyed into the arms of canonistack13:16
wgrantAlthough sinzui is close behind.13:16
jmlhmm.13:16
rick_h_sinzui: yea, are there instructions for that?13:17
jmlI don't know how to get that report from ohloh13:17
StevenKsinzui: Sounds good to me. Except no one seems to care.13:17
wgrantjml: Code Analysis13:17
wgrantjml: You can click on people in the legend to turn them off.13:17
wgrant(ie. to dispose of PQM)13:17
wgrantI'm not quite sure how it picks who to show.13:17
wgrantLooks vaguely like the top 5 of the last year and then the top 513:17
wgrantof all time13:18
sinzuime? I have done bugger all for 12 months. I am atrifying.13:18
rick_h_jml: so side curiousity here, is this how people are getting listed in the software center? They have a commercial ppa and that's listed out as apps in software center?13:18
jmlrick_h_: yeah, that's right.13:19
jmlrick_h_: but only for commercial (i.e. they cost money) apps13:19
rick_h_jml: right, the newish paid apps stuff13:19
jmlyep13:19
rick_h_ok, just curious, wasn't sure how that stuff worked and this review has the brain thinking13:20
wgrantjml: Aren't the proprietary free apps also there?13:20
wgranteg. Vendetta Online13:20
wgrantOr whatever that free one is13:20
jmlthis work we're doing with LP now is aimed at people being able to upload apps to developer.ubuntu.com and have them packaged and published to a PPA automatically13:20
jmlwgrant: I don't know about proprietary free.13:21
cjwatsonwgrant,jml: http://people.canonical.com/~cjwatson/tmp/loc.png is a currently-up-to-date graph of delta per commit.  But it's kind of hard to read; it looks mostly nice and negative but the net change over that time period is in fact +2755.13:21
jmlwgrant: I know that libre+gratis goes to extras (or the archve)13:21
wgrantjml: I like to pretend that extras doesn't exist :)13:21
jml:)13:21
wgrantcjwatson: :(13:21
jmlcjwatson: do you have a graph of the integral of that?13:22
cjwatsonjml: that's ohloh's graph, isn't it?13:25
cjwatsonI guess you might want zoomed in and zero-based13:25
jmloh yeah.13:25
jmlI think I'll just hook up bzr & ggplot2 in my own spare time :)13:25
StevenKSome counter on lpqateam.c.c would be nice.13:25
jmlwhat I want is a command that tells me my score.13:25
rick_h_haven't you heard, you need graphite graphs these days to be cool13:26
StevenKI just want to see if I'm negative.13:26
rick_h_with hourly commit points and fancy colored moving charts13:26
rick_h_and we all put them on a display on the wall over our desks13:27
StevenKrick_h_: Speak for yourself, hipster.13:27
rick_h_lol13:27
rick_h_I'm on LP, I think that disqualifies me for any hipster movements13:27
StevenKHaha13:27
wgrantNah13:28
wgrantGitHub's too mainstream13:28
rick_h_when we get some BDD JS going then we'll chat about rushing house hipster13:28
jmlStevenK: yeah, if I'm negative that's a good thing to know. But I also want to be able to compete with people13:28
rick_h_where's jono when you need some new achievement badges13:28
rick_h_"negative LoC leader for month of May"13:29
StevenKIf we hook the LOC metric into Ubuntu Achievements, someone will get fired.13:29
rick_h_lol13:29
jmlStevenK: how so?13:29
StevenK... or I'll quit in disgust.13:29
StevenKjml: There's an implied :-P13:30
cjwatsonjml: but, http://people.canonical.com/~cjwatson/tmp/loc-cum.png13:30
jmlStevenK: oh, I just didn't (don't yet) get the joke.13:30
jmlcjwatson: thanks!13:31
StevenKI'm waiting for my UI work to end on prod.13:31
wgrantI wonder how it changes if we filter out database/schema13:31
wgrantSince that's append-only.13:31
StevenKThen I can rip out 2,000 lines of stuff.13:32
cjwatsonjml: (easy once I remembered 'smooth cumulative'.  gnuplot is ugly as sin but quick to hook trivial stuff up in.)13:33
jmlcjwatson: ah cool.13:33
wgrantbzrlib + matplotlib ftw13:34
jmlyeah. I've enjoyed the stuff I've done recently with R & ggplot213:35
wgrantEnjoying R? I think you're doing it wrong.13:35
jmlwebulating graphs is too hard though13:35
jmli.e. taking a one-off graph made and turning into something that can be hit on the web and is always relatively up-to-date13:35
jmlwgrant: the thing I really like is that it's really easy to use my data analysis again at a later point. Mostly my Python graphing stuff doesn't seem to work out like that.13:37
wgrantYeah13:37
jmlMaybe I should try that panda thing GvR posted on G+ about.13:37
jmlalso, R is actually pretty good for interactive mucking around with data13:37
jmleven if it's rather poor considered as a language.13:38
jmldistributionmirror_prober: is that a candidate for splitting out?13:39
wgrantYes.13:40
wgrantIt should in all probability be a separate database.13:40
wgrantThe interactions with LP internals are minimal.13:40
wgrantBut it should probably be deleted and replaced with an existing third-party solution13:40
jmlooh13:41
jmlits tests are an affront.13:42
jmlI'd like to fix them, but removing them & the code they test entirely would be much better.13:43
wgrantExactly.13:44
wgrantLots of Launchpad components like to pretend they're solving a unique problem.13:45
wgrantSo stuff like that happens :)13:46
jml:)13:46
jmljames_w: hello13:46
james_whi13:46
jmlI'm wondering whether I should make a celebrity team for which membership means "you can set commercial"13:46
jmlor whether I should re-purpose the software_center_agent celebrity somehow13:47
wgrantOr add a field to distribution which permits it.13:47
jmlwgrant: e.g. Distribution.commercial_team13:47
jml?13:48
wgrantnasty_proprietary_software_people, but yeah :)13:48
jmlhmm.13:48
deryckwgrant, hey.  I saw something in scrollback about lp being down for you guys.  are there incident reports for this?  or anything I can look at.13:49
jmlI need a better name than either of them.13:49
wgrantderyck: https://wiki.canonical.com/IncidentReports/2012-04-26-LP-release-performance-issues covers the first two of the day's issues, the other one is in the hands of IS, I believe.13:50
jml'commercial_admin' would be pretty good if it weren't already a heavily-loaded term13:50
wgrantYeah13:50
deryckwgrant, thanks!13:50
wgrantderyck: Alternatively we can replace commercial with a shut_up flag that anyone can set, I guess.13:50
wgrants/anyone/the archive owner/13:51
wgrantBah13:51
wgrantjml: ^^13:51
deryckheh.13:51
* deryck votes for shut_up flah, too, FWIW. :)13:51
deryckflag13:51
wgrantjml: I thought my team name was pretty catchy, though.13:51
jmlbigjools seemed to dislike the idea13:51
jmlwgrant: sure, it has zing. It could be easily misunderstood though.13:52
rick_h_abentley: ping, can you take a peek at this for me when you get settled? https://code.launchpad.net/~wallyworld/launchpad/revoke-access-delete-subscriptions-job/+merge/10419813:52
abentleyrick_h_: Yes, I'll have a look at that.13:53
rick_h_abentley: want to make sure it's all celery approved, not sure about the any special needs/better ways of doing things for that.13:53
rick_h_thanks!13:53
jmlwgrant: as it would read to me as "all of the propriety uploaders to d.u.c" rather than "the d.u.c bots"13:53
wgrantTrue, true.13:53
james_wjml, I'd lean towards re-using the celebrity, but wouldn't be against a field on distribution I don't think13:53
jmljames_w: one thing about the celeb is that (I think) it's a Person not a Team13:54
wgrantjml: It can be either.13:54
jmlwgrant: I mean that currently it is a Person.13:54
wgrantPeople are teams, war is peace, etc.13:54
wgrantAh13:54
jmlwgrant: I don't know what the fallout of changing it would be.13:54
wgrantIt would break.13:55
wgrantI think.13:55
jmlwell, yeah, that's generally what we mean by "fallout of change"13:55
wgrantSince I believe the XML-RPC API may actually set up an interaction as it.13:55
wgrantWhich won't work if it's a team.13:55
wgrantBut I may be misremembering, and it just passes it in as arguments to horrid methods, in which case it might work.13:55
wgrantThere's one easy way to find out...13:56
jmlI'm more worried about how we use it on our side13:56
jmlthat's harder to co-ordinate.13:56
wgrantHowso?13:56
wgrantHow do you use it on your side?13:56
wgrantI thought you mostly poked the xmlrpc-private API13:56
jmlI don't know.13:57
* jml looks13:57
wgrantAh13:58
wgrantThere's a single xmlrpc-private call, getOrCreateSoftwareCenterCustomer13:58
wgrantRest must be through lazr.restful.13:58
jmlyeah13:58
wgrantSo, you could create a team, add the agent to it, then make that team the celeb.13:58
rick_h_jml: have a bug to tie the MP against please?13:59
rick_h_well guess there's not a lot of QA for this removed is there :/13:59
jmlrick_h_: https://bugs.launchpad.net/launchpad/+bug/99262214:01
_mup_Bug #992622: getCommercialPPAs is not needed <tech-debt> <Launchpad itself:New> < https://launchpad.net/bugs/992622 >14:01
jmlrick_h_: as you wish.14:01
rick_h_jml: yea, that ec2 land wants things to all be setup right, picky little thing. :) thanks. Landing now14:02
jmlrick_h_: thanks! :)14:02
jmlhmm14:08
jmlI have to write this code somehow.14:09
jmlI prefer the 'shut_up' field approach, but suspect that it'll be rejected as Whiggery.14:11
jmljames_w: am probably going to start implementing it as field on Distro14:12
wgrantMy vote goes to shut_up.14:12
wgrantI don't see a good reason to restrict it.14:12
james_wshut_up on archive, or subscription?14:13
wgrantProbably archive.14:13
wgrantBut I'm not quite sure.14:13
james_weither way I think I prefer that too14:13
james_wassuming it's roughly similar in implementation time14:14
sinzuijcsackett, ping14:14
james_wno need for this to be special14:14
jmlI guess one archive owner could use it to secretly upload evil stuff to a PPA without other owners finding out14:14
jcsackettsinzui: pong14:14
jmljames_w: I think they are both similar and both fairly small.14:14
wgrantjml: Hm?14:14
jcsackettrick_h_: enjoying your first day of full reviewing? :-)14:14
sinzuijcsackett,  is there a card on the board that interests you?14:14
rick_h_jcsackett: wheeeeee14:14
wgrantjml: AIUI the flag would just control subscription notifications.14:14
james_wif it has to be available in the UI then it will probably be more work to implement, and be quite LOC-positive14:14
jcsackettsinzui: i was looking at banners without javascript.14:14
wgrantjml: "subscription" refers to packages from the archive, not upload notifications.14:15
jmlwgrant: ah, not upload notifications.14:15
wgrantUpload notifications go to the uploaders, which are the owner and anyone with an ArchivePermission14:15
sinzuijcsackett, do you want to talk about it?14:15
jcsackettsinzui: yes. hangout?14:15
sinzuiyes14:15
* sinzui looks for headphones14:15
jmlso, it would just be suppress_subscription_notifications or something to that effect.14:17
jmlok.14:17
jmlI'm going to relocate to somewhere that serves food & has internet.14:18
sinzuijcsackett, I am in the google messenger  hangout14:18
jcsacketthm. it's telling me you're offline14:19
sinzuiMy head is14:19
* sinzui tries again14:19
rick_h_abentley: thanks for checking that out, appreciate the assist14:54
abentleyrick_h_: no problem.14:54
jmlI haven't submitted a db patch since we switched to nodowntime stuff. what do I need to know?15:17
jmlhttps://dev.launchpad.net/Database/LivePatching doesn't seem to mention anything about adding, removing or renaming columns15:20
jmljames_w: so I'm thinking of doing this shut-up patch by re-purposing 'commercial' and then following up with a db patch to rename it.15:27
jmljames_w:  thoughts?15:27
jmlanyone?15:28
rick_h_jml: looking for the doc15:34
rick_h_my understanding is you need to have a MP against the devel-db branch and get it ok'd by db peeps (lifeless/etc) and then it has to land first15:35
cjwatsonjml: https://dev.launchpad.net/PolicyAndProcess/DatabaseSchemaChangesProcess15:35
rick_h_there you go and https://dev.launchpad.net/WorkingWithDbDevel15:36
jmlthat doc doesn't say anything about removing or renaming columns15:36
cjwatsonadding columns comes under cold patches15:36
jmlnor is there anything I can see that categorizes new columns as either 'hot' or 'cold'15:36
jmlcjwatson: ah15:36
cjwatsonby virtue of not being explicitly under hot patches15:36
jmlright.15:37
cjwatsonso it'll be a fastdowntime thing15:37
jmlthanks.15:37
cjwatsonif you're removing columns, you'll need to make sure that no appservers are using the old columns first via a nodowntime deployment (I don't know if there are stricter requirements beyond that)15:38
jmljust got a weird error trying to run 'ec2 test': http://pastebin.ubuntu.com/960374/16:30
jmldoes the erstwhile Launchpad team hang out somewhere else these days?16:31
rick_h_sorry, no idea there16:32
rick_h_repeatable error? e.g. not temp bzr connection issue?16:33
jmlI'm trying to repeat it now.16:33
jmljust repeated it.16:33
jmlit happens in the initial connection.16:33
rick_h_k, sec, pulling that branch ( was going to peek at it for review anyway) and I'll test it16:34
cjwatsonjml: public IP problem again?  what does checkip.amazon.com say?16:36
cjwatsonsorry, checkip.amazonaws.com16:36
jmlheh, it might be because...16:36
jmlcjwatson: yeah, that seems to be it.16:37
cjwatsonwhen I was landing a branch from Millbank last week I hardcoded Millbank's external address in ec2test/account.py16:37
jmlcjwatson: I'm actually in a pub tethered to my phone16:37
rick_h_what's the 'public ip problem'?16:38
jmlcjwatson: but checkip says 10.92.29.164 and mumak.net (which I'm ssh'd into) says something else16:38
jmlcjwatson: I'm angry enough to patch lp-dev-utils to have an option for that.16:38
* jml hulks out16:38
rick_h_ruh roh16:38
rick_h_ok, well running fine here so glad it's something special16:39
jmlrick_h_: thanks.16:42
cjwatsonI have no idea why checkip thinks it's clever to honour X-Forwarded-For.16:42
jmlWell, what is honour, if not an occasional rejection of the pragmatic?16:43
cjwatsonWe could use http://whatismyip.akamai.com/ instead16:44
cjwatsonWFM16:45
jmlme to16:46
jmloo16:46
cjwatsonrick_h_: the problem is that amazon's checkip service is stupid and if you're connecting to it through an HTTP proxy it returns the address that the proxy sees (via the X-Forwarded-For HTTP header)16:47
cjwatsonthis is useless if you're trying to find the address amazon will see for you, since proxies are often on private networks16:47
rick_h_cjwatson: ah ok, and this is setup as the address you can ssh from to the new instance?16:47
cjwatsonright16:47
rick_h_gotcha16:47
rick_h_bah, can't edit a review comment after the fact? oh well16:58
rick_h_jml: small note on my end, but yea the others will be better to go over things.16:58
james_wjml, +1 on repurposing commercial16:59
james_wjml, if we strip away the other meanings of it, rename it, and allow anyone to set it if they own the PPA then it works for us, is a pretty minimal change, and IMO reduces the maintainence burden by taking something special-purpose and arcane and makes it pretty obvious16:59
james_wrick_h_, looks like your comment on https://code.launchpad.net/~jml/launchpad/drop-special-commercial-permissions/+merge/104270 got truncated16:59
jmljames_w: yeah. and we've actually got line count credit on this already from deleting getCommercialPPAs16:59
rick_h_james_w: well, I was going to say I'll invite people from the permissions/etc squad but then realized they were already up there and missed hitting delete before submitting16:59
rick_h_and I can't edit it back out, so I fail16:59
james_wrick_h_, cool16:59
* jml has to go soon :(17:04
jmlok. this doesn't seem to actually *work*, but it's a start: https://code.launchpad.net/~jml/lp-dev-utils/public-ip/+merge/10427317:07
jmlI have to go.17:07
=== rick_h_ changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: - | Firefighting: - | Criti
=== rick_h_ changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: - | Firefighting: - | Critical bugs: 3.47*10^2
lifelessflacoste: hiya19:09
flacostehi lifeless19:10
lifelessflacoste: hows the sprint going ?19:14
lifelessbwah... deryck: around ? if so, could you put bug 901892 as the external board # on https://canonical.leankitkanban.com/Boards/View/14028616/10117343319:24
_mup_Bug #901892: bug search cache makes next/prev return old data <bug-columns> <Launchpad itself:Triaged> < https://launchpad.net/bugs/901892 >19:24
deryckhi lifeless19:26
derycklifeless, done19:27
lifelessderyck: thanks19:33
jelmerhi deryck, lifeless19:35
deryckhi jelmer19:35
jelmerlifeless: I'm wondering if it would make sense to kill the code review notification emails; they just seem to be noise at this point. Do you have any opinions on that?19:35
jelmerIIRC you brought it up a while ago, but I can find the relevant log at the moment to confirm.19:36
lifelesscode imports ?19:38
lifelessyes, I proposed removing at minimum all the non-action ones (successes, initial-failures)19:39
lifelessI have no particular opinion about non-initial failures.19:39
jelmerIt's probably easiest to remove both at the same time19:41
jelmerI've stopped paying attention to the non-initial failures, and I haven't seen anybody else use them.19:41
lifelesssure19:58
lifelessmpt: hoya19:58
mpthi lifeless19:58
lifelessso, what questions are you and ev trying to answer with this mtbf thing ?19:58
mptlifeless, for the MTBF specifically, the first question: (1) How reliable is Ubuntu?20:12
mptFor example: Is it more reliable than it was last week? Is it more reliable than the previous release? And how much of a difference would it make if all Ubuntu users had installed all updates?20:13
sinzuiwhat is MTBF? mounted tabernacle bee floggers20:25
lifelessyes20:29
lifelessmpt: those are good questions20:29
lifelessmpt: I have one; how do we define reliable, and is it a definition users would agree with ?20:29
mptlifeless, that introduces the issue that our definition of reliability will expand over time20:30
mptlifeless, in Ubuntu 12.04 it is "programs other than the kernel don't crash"20:31
mptIn Ubuntu 12.10 it may be something like "programs (including the kernel) don't crash, and don't hang for more than 20 seconds"20:31
mptDepending on time and difficulty, it may also include other kinds of errors: package installation failures, debconf prompts, etc20:33
mptSo, one thing we've discussed is keeping track of when we started recording error type, to fairly answer questions like: "Is Ubuntu 13.04 more or less reliable than Ubuntu 12.10, using the same standards that we were using for Ubuntu 12.10?"20:35
mptotherwise each successive release would seem less reliable than the last, even if it's more reliable. :-)20:36
mptlifeless, but there will always be swathes of problems that aren't caught in an automated way, e.g. "this program thinks it copied all these files but it didn't".20:37
mptAnd a user might classify that under "reliability".20:38
mptIf a test suite is the fence at the top of the cliff, Whoopsie and Daisy are the ladies with pens and clipboards standing at the bottom.20:39
lifelessmpt: I like that image20:44
lifelessmpt: so, are you factoring in time in your equations? And how?20:45
mptlifeless, no, we'd completely overlooked it20:48
mptIf by "time" you mean "uptime"20:49
mptThe current formula implicitly assumes that every machine is running 24/720:49
lifelessmpt: perhaps the client could report TSLF in each report, or some approximation thereof.21:03
mptlifeless, I think it will have to, yes. Which means we run the risk that the system clock changed in the intervening period, but oh well.21:06
lifelessmpt: that should be rare though21:06
lifelessmpt: and rare events should be discarded as outliers21:06
mptI suppose system clock adjustments will be roughly normally distributed anyway :-)21:07
david0rkhi all21:19
david0rkanyone on?21:20
david0rknevermind.  I figured out using a bit of common sense and tinkering.   BAZAAR IS AWESOME!22:22
SpamapShrm, I want to make sure I understand. bug #915505 means we can't use bzr builder format 0.4, right?22:23
_mup_Bug #915505: bzr: ERROR: exceptions.AttributeError: 'cStringIO.StringI' object has no attribute 'split' <launchpad-buildd:Triaged> <Open Sesame: libaccounts-glib:New> < https://launchpad.net/bugs/915505 >22:23
* SpamapS is sad, that would let him automate stable releases building into a PPA22:23
bachi lifeless22:29
jelmerSpamapS: yes22:35
SpamapSjelmer: and will LP have to be migrated to precise before that works?22:36
jelmerSpamapS: no, it will just need to be updated to a newer python-debian or we can update bzr-builder to support the older python-debian.22:36
SpamapSah22:36
jelmerSpamapS: the fix for bzr-builder would be trivial, it's mostly the management around it to get it deployed to the builders that's more complicated.22:38
SpamapSWill probably be easier to just automate my packaging branch to have a new debupstream every time I commit to my stable branch22:41
SpamapSbut, thats a pity, as 'latest-tag' is really cool22:42
lifelessbac: hi there22:43
aquariusI have a program, not of my creation, which calls launchpadlib. Launchpadlib, if you've not been authed to use LP, opens up LP in a browser and then blocks, waiting for you to auth the token. Is there some envar or similar I can set so that launchpadlib will just fail that request instead because it's not being run interactively?23:04
cjwatsonaquarius: It uses the webbrowser module in Python's standard library, which honours BROWSER, so you could set that to ... err, something appropriate23:07
aquariusaha! sneaky :)23:07
=== SpamapS_ is now known as SpamapS
cjwatsonNot entirely sure how to make that fail immediately rather than spinning.23:08
StevenKexport BROWSER=/bin/false ?23:08
StevenKThat might be a little mean.23:08
aquariusI shall see if it works :)23:08
cjwatsonI don't think that will work; webbrowser will notice the non-zero exit code and fall back to its next option, as I read it.23:08
cjwatsonBut /bin/true might work.23:09
cjwatsonHopefully you'll get EndUserDeclinedAuthorization.23:09
cjwatsonI don't see another way to control it in an entirely different process.  If you can modify the program then you can pass in a different authorization_engine.23:10
cjwatsonYou might have to write a miniature browser that actually interacts with the URL it's given and presses the Decline button or whatever it's called.23:11
aquariusdoesn't seem to honour BROWSER, afaict :(23:11
wgrantYou won't get a button -- you'll be redirected to the login page.23:11
aquariusah, I'll try the /bin/true trick :)23:11
cjwatsonReally?  It's right there in webbrowser.py.23:11
cjwatsonMm, maybe launchpadlib will just think you haven't got round to logging in and deciding yet. :-(23:12
aquariusthat's what happens23:12
aquariusno browser opens (no problem there), but then lplib just sits about waiting for me to finish authing.23:13
aquariusdarn.23:13
=== jelmer_ is now known as jelmer
aquariusand I can't see how to get out of that code, either23:13
cjwatsonAre you trying to do something headlessly?23:13
aquariusyep23:13
aquariusspecifically, run the test suite for quickly23:13
cjwatsonThat sounds like a bug in the test suite, then.  Surely it shouldn't fail without a Launchpad login.23:14
cjwatson(Or hang.)23:14
aquariusit *is* a bug in the test suite23:14
aquariusbut the test suite's a terrifying mass of shell scripts which I do not have time to entirely rewrite from scratch :P23:15
aquariusso I was hoping for a semi-quick fix to make the tests fail, rather than just hang indefinitely :)23:15
cjwatsonQuickest fix would be to give them a login someho. :-)23:16
cjwatson*somehow23:16
aquariusthat's approach number 2 if I can't find a way to do this, which is fractionally vanishingly more proper a solution :)23:16
aquariusand I don't think this is doable, annoyingly enough. Ah well.23:17
cjwatsonI thought bribing didrocks was the traditional answer to quickly problems, anyway23:17
aquariuss/didrocks/mterry/ in this particular case, and he said "yeah the test suite is weird, feel free to just put in a separate test which is not tied to it" :-)23:18
aquariusso I have plenty of get-out clauses23:18
aquariusI just thought I'd try and do things properly (well, sorta) rather than super-hackily for once, and lo! I am thwarted. there is a lesson here :)23:19
=== elmo_ is now known as elmo
=== StevenK_ is now known as StevenK
=== StevenK changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: StevenK | Firefighting: - | Critical bugs: 3.47*10^2
=== Noldorin__ is now known as Noldorin
=== cjwatson_ is now known as cjwatson

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