/srv/irclogs.ubuntu.com/2012/02/06/#launchpad-dev.txt

wgrantwallyworld_, StevenK: You've a revision each to QA.00:33
wgrantThen we can unbreak JS :)00:33
StevenKwgrant: I'm distracted by Apache and convoy, but also still trying out how to QA it00:37
StevenKSetEnv doesn't hit os.environ :-(00:37
wallyworld_wgrant: do you know of any private branches on qas?00:38
StevenKMake one00:39
wgrantwallyworld_: All of lp-production-configs00:39
wallyworld_right, thanks00:39
wallyworld_wgrant: qa is done00:45
* StevenK kicks Apache, and looks at his QA.00:49
wgrantwallyworld_: Thank01:06
wgrants01:06
wallyworld_np01:06
StevenKMine is done too01:07
StevenKwallyworld_: Jon marked my combo-url branch as Needs Information :-(01:07
wgrantGreat. Now we just need the DB user setup.01:07
* wallyworld_ takes a look01:08
wallyworld_StevenK: i think a fair point is raised? rf already does sudo stuff to install apache, so it could do the new stuff as well01:13
StevenKwallyworld_: You can NOT re-run rf-setup after it has run once01:14
StevenKWe need to handle existing setups as well as new ones01:14
wallyworld_sure, but for new installs01:14
rick_hStevenK: right, but for those starting from now on it could/should be there right?01:14
wallyworld_+1 onwhat rick said01:14
rick_hStevenK: so it seemed like a small missing automated update for the branch to be "complete"01:14
StevenKNew installs will just work with rf-setup01:14
rick_hI was hoping he'd ok it with the request for that added, but didn't01:15
StevenKCurrent installs need to install mod-wsgi, and run 'sudo make copy-apache-config' if they want to play with the loader01:15
wallyworld_yes, and he was suggesting that be done in rf wasn't he?01:15
StevenKIn rf-setup?01:15
wallyworld_yes01:16
StevenKLike I said, we can't?01:16
StevenKIt is already handled in rf-setup01:16
wallyworld_ok, so was the needs info request redundant?01:16
wallyworld_if it's already all done01:16
StevenKI have no idea, and you've confused me.01:16
rick_hrf-setup already install mod_wsgi and creates the symlink directory for the convoy root?01:16
StevenKrf-setup install mod_wsgi, and if I'm reading the makefile correctly, it will create /srv/launchpad.dev too01:17
wallyworld_i think the above is what jon was askin for, if not done already01:17
rick_hStevenK: so the answer is that it already does that. I don't think we saw it that way reading the diff01:18
StevenKrick_h: Right -- it requires some knowledge of what rf-setup does01:18
wallyworld_so, bottom line is that rf-setup needs to prepare a clean system to be able to run the combo loader. if it can do that,  the needs info is redundant01:18
rick_hwallyworld_: +101:18
wallyworld_and we also need to the tools/make target(s)/process doc to be able to add the required symlink/whatever to an existing system01:20
StevenKwallyworld_: I was going to announce that on launchpad-dev once combo-url lands01:20
wallyworld_yeah, as i thought you would, just rounding out the requirements so we all understand01:21
wallyworld_or agree01:21
StevenKSigh. WSGI sucks.01:24
StevenK"Environment variables are too hard, so you need to deal with it in this dict01:24
StevenKSo to get convoy working with overriddable roots, I had to copy the inner function in combo_app :-(01:42
StevenKJust returning the app didn't work, since I can't pass the environ and start_response into the inner function in combo_app.01:43
StevenKAnd this, kids, is why nested functions are bad.01:43
rick_hStevenK: huh?01:44
rick_hStevenK: if you wsgi layer another app on top of the convoy wsgi app and chage convoy to pull from there no worky?01:44
rick_hor we don't want to change convoy for that, it would suck01:44
StevenKrick_h: WSGI layer?01:44
rick_hsec, let me pull it up01:45
StevenKrick_h: Using def application(...): return combo_app(root) no worky01:45
rick_hStevenK: right right01:46
rick_hStevenK: hmm, so I think what we'd have to do is move the app aprt out of inside of combo_app() in convoy'01:48
rick_hand then we'd wrap that ourselves with our own wrapper01:48
rick_hthe problem is that convoy doesn't expose the raw wsgi app without the root already set01:49
StevenKRight01:49
rick_hthat's ok thoguh, should be a small patch to convoy to get it so we can import app vs combo_app01:49
rick_hand then we'd have raw access to setting root as we see fit ok01:49
StevenKSo we *could* change convoy to in the inside app call to set root from the environ if it's None01:50
StevenKrick_h: But changing it so the inner function becomes an exportable one sounds fine too01:54
rick_hStevenK: right, but I'd rather do something like tihs: https://pastebin.canonical.com/59462/01:55
rick_hwhich is a completely off the cuff 2min take at it01:55
rick_hthen we'd basically change our .wsgi files to do our own combo_app with our own values and can change them as we see fit01:55
StevenKRight01:56
StevenKWhy partial() ?01:56
rick_hjust because it keeps their signature all the same01:56
rick_hI'm sure combo_app could be turned into some sort of decorator but that would require me to hit more docs01:56
StevenKreturn partial(.. ? :-)01:56
rick_hI jsut mean that I had to add extra kwargs to app() so that their combo_app thing works the same01:57
rick_hand those values need to get passed through, since we're returning a function, not a result you'd either have to decorate it or partial it or something01:57
rick_hbut if I'm off let me know, I'm 3 glasses of wine into the night before bed so grain of salt and all that jazz01:58
StevenKI was hoping I could just use os.environ, but I'm not that lucky01:58
rick_hStevenK: well our wrapper could once the app is exposed on its own01:58
rick_hbut right, we're going to need to do another patch to convoy to get it there01:59
StevenKWe'll need to patch it before it hits qas01:59
rick_hok, well I can do that tomorrow morning if you want.01:59
StevenKBecause as it stands, the version in our PPA will only look in /var/tmp/convoy01:59
rick_hI'm working on updating the tets and such, but that requires your convoy patch to land first so I'm stacking branches like there's no tomorrow02:00
rick_hStevenK: right02:00
StevenKHaha02:00
StevenKrick_h: I'm happy to work off your pastebin02:00
rick_hStevenK: k, if it falls apart shoot me an email and I'll pick it up come morning02:00
StevenKrick_h: Okay, thanks!02:00
wgrantstub: In PL/pgSQL I can INSERT from a ROWTYPE variable, but I don't seem to be able to UPDATE all fields without naming them explicitly.02:59
wgrantIs there any benefit to an UPDATE over a DELETE and INSERT?02:59
stubwgrant: syntax sucks there02:59
* stub ponders02:59
wgrantI guess in 9.2 there might be (what with immutable columns and all that), but I suspect now there's no difference.02:59
stubwgrant: Two statements get run - two lookups etc. The end result on disk will be the same.03:05
stubUnless the only change is to unindexed columns03:05
wgrantHm?03:06
wgrantWon't the UPDATE create a new tuple regardless?03:06
wgrantand therefore poke the indexes the same way03:06
wgrant?03:06
stubSo if you update a row, and don't touch indexed columns, and the new row can be written to the same page, the indexes don't need to be rewritten.03:07
wgrantAh03:07
wgrantAnyway, I guess I'll just name the columns explicitly :/03:07
stubYer. You can use dynamic SQL, but that solution sucks worse than the problem03:08
wgrantYep03:08
stubOr plpython, but that will be slower due to the type casting that needs to go on03:08
wgrantplpython is never the solution :)03:11
stubLots of improvements have gone into plpython in 9.103:13
wgrantYes, but it's still terrible.03:14
lifelessstub: I thought you could write to indexed columns and only the affect indices must change (and they may end up with same-page updates too, under appropriate circumstances)03:19
stub"UPDATEs and DELETEs leave dead tuples behind, as do           failed INSERTs. Previously only           VACUUM could reclaim space taken           by dead tuples. With HOT dead tuple space can be           automatically reclaimed at the time of INSERT or UPDATE           if no changes are made to indexed columns. This allows           for more consistent performance. Also, HOT avoids adding duplicate index           entries."03:23
stub^^^ was what I was thinking off, pulled from the 8.3 changelog03:23
lifelesshmm, I should dig up the code and check the conditions. It makes sense to me that the same core logic would allow a wider ranger of conditions.03:24
stubI'm not sure about the optimization you mention03:24
lifelessanyhoo, I'm not really here today ;)03:25
rick_hStevenK: ping03:56
StevenKrick_h: Hai03:57
StevenKrick_h: Aren't you supposed to be sleeping03:57
rick_hStevenK: so stupid ?, why do we need the ENV based root for convoy?03:57
rick_hStevenK: yea, but I can't stop thinking about this, I don't thinkt he answer I gave you will help03:57
StevenKHaha03:57
rick_hand then I got wondering why we're doing this anyway03:57
StevenKrick_h: I just put up an MP: https://code.launchpad.net/~stevenk/convoy/exportable-app/+merge/91601 . That should answer your question03:58
rick_hbecause if we update convoy we can set the root to /srv/.../ right?03:58
rick_hStevenK: right, but when are we going to have to change the root in the wsgi config in apache?03:59
rick_hthat's the part I'm missing03:59
StevenKrick_h: So, asuka runs both staging and qastaging03:59
StevenKrick_h: So it will have convoy installed, and the WSGI file in /usr/share/convoy/convoy.wsgi03:59
rick_hand they're not the same rev/updated together?04:00
StevenKrick_h: staging uses db-stable and qas uses stable04:00
rick_he.g. is there a reason they can't share the same combo loader?04:00
StevenKAnd no, they aren't updated together04:00
rick_hStevenK: ah right, ok...well @#$#@$04:00
lifelessthey can share the same combo loader as long as:04:00
rick_hnvm then, carry on. Thanks for updating me04:00
StevenKrick_h: So we need a way to inject a different root directory04:00
lifeless - we never ever do an incompatible change to it04:00
lifeless - we do something to accomodate the different revno sequence between db-stable and stable04:01
lifelessthe former point is probably pretty easy, as incompatible changes will screw deploys anyhow04:01
rick_hStevenK: ok, so this change will do what you need? You've tested this?04:01
lifelessthe latter point is harder04:01
StevenKrick_h: I've tested both cases of combo_app() and _application()04:01
rick_hStevenK: because we're still just building a callable we return and don't actively change the root here are we?04:01
rick_hStevenK: right, but I mean when you use _application in LPs .wsgi file it will listen to the ENV path as you wnt?04:02
StevenKdef application(environ, start_response):04:02
StevenK    root = environ.get('CONVOY_ROOT', '/var/tmp/convoy')04:02
StevenK    return _application(environ, start_response, root=root)04:02
StevenKOur WSGI file with that change04:02
rick_hStevenK: right, but that's called once when apache starts up the wsgi app04:02
rick_hStevenK: so whatever one is the latest one wins? How is this passing two different roots?04:03
rick_hStevenK: because we're saying one apache .wsgi file is serving both servers right?04:03
StevenKrick_h: The environment variable in environ will be different for both staging and qas04:03
* rick_h should go to bed, probably missing something easy04:03
rick_hStevenK: ok, so in apache are there one or two convoy servers running?04:03
StevenKThere will be two04:04
rick_hStevenK: so why couldn't we pass the root kwarg to the old combp_app thing?04:04
rick_hStevenK: ah I get it, the environ is where you're getting it from04:05
StevenKRight04:05
rick_hStevenK: so you need the app to be up first, ok, sinking in now.04:05
StevenKAnd Apache's SetEnv does not hit os.environ04:05
rick_hI was thinking a system ENV variable like os.environ04:05
rick_hStevenK: right right, remember hitting that before now that you mention it04:05
StevenKrick_h: Clear as mud?04:05
rick_hok, the world makes sense again, I can go back to sleep. Thanks for walking me through it04:06
StevenKrick_h: I should probably thank you for the patch in the MP.04:06
StevenKMy guilt may not let me claim credit. :-P04:06
rick_hStevenK: heh, it's ok. Consider it a freebie :P04:06
StevenKrick_h: I've been in free software since 2000 -- I can't not mention you. :-P04:07
StevenKIt's just not done.04:07
rick_hanyway, see you all again in 6hrs ish04:07
StevenKlifeless: Can I nail bug 805546 shut?04:16
_mup_Bug #805546: persontransferjob does not have a unique oops prefix <Launchpad itself:Triaged> < https://launchpad.net/bugs/805546 >04:16
wgrantwallyworld_, StevenK: The AJAX log is still broken on production, despite the LPJS thing being deployed.04:16
StevenK:-(04:17
wallyworld_is that all that look broken?04:34
wgrantNo idea.04:36
stubAnd all my qa done for me, ta muchly :)04:42
StevenKHaha04:47
StevenKwgrant: Wish I knew how to debug it04:58
StevenKwallyworld_: Can haz update of the kanban board?04:59
StevenKSince wgrant has gone and done the one bit I was about to05:00
wallyworld_done. didn't realise you were so anal about it :-P05:00
nigelblol05:16
* StevenK sighs at bugs including OOPSes that have been pruned05:17
lifelessStevenK: does it generate good oops ids now?05:18
StevenKlifeless: PTJ? I have no idea. I thought we didn't use prefixes for oopsii at all now?05:19
wgrantWe really must get LaunchpadScript to inject a sensible reporter.05:19
lifelessStevenK: if there is no oops prefix configured for it in the configs, then it should get a sensible one at runtime05:29
lifelessas long as a) the base config its using is sane (e.g. has 'production') and b) it attempts an override, which I think the job runners all do05:29
StevenKlifeless: The bug was reported when it would oops with REPORTIFSEEN05:30
StevenKCan haz review? https://code.launchpad.net/~stevenk/launchpad/invalid-mp-api/+merge/9160705:39
StevenKstub: ^ You're on the hook as OCR.05:46
stubThat will teach me for starting on time.05:47
StevenKIt's a very short MP05:47
stubI'll just finish typing up this db review05:47
stubStevenK: r=stub05:53
StevenKstub: Thanks!05:57
StevenKwgrant: The CommercialSubscription query is ~12ms on DF, and ~4ms when I create the index on CS.date_expires06:12
wgrantStevenK: It's a small table atm, but still worth the index06:16
wgrantcan has explain with and without?06:16
StevenKwgrant: http://pastebin.ubuntu.com/831004/ -- but it doesn't seem to use the index?06:19
wgrantAh, so the 12ms was cold?06:22
wgrantAnd the 4ms reasonably hot?06:22
StevenKPerhaps06:23
wgrantAnyway, looks like there's so few rows that it doesn't even try the index, so let's not bother.06:24
wgrantWe'll have other problems once the table starts getting big.06:25
StevenKOh, right, Postgress will just decide to not touch the index?06:25
wgrantRight. The whole table probably fits on just a page or two.06:26
wgrantUsing the index would double the reads.06:26
wgrant relpages06:26
wgrant----------06:26
wgrant        706:26
wgrantBut similar argument.06:26
wgrantThe index is 4 pages, so yeah, I think the planner's decision is reasonably correct.06:27
wgrantstub: I wonder if a sequence column is missing from specificationworkitem06:33
wgrantThey presently seem to like to keep them roughly in implementation order.06:33
wgrantBut perhaps not.06:33
wgrantHm07:13
wgrantWhy is everything segfaulting...07:13
StevenKEverything?07:14
stubwgrant: I've added a comment re: sequence column.07:19
wgrantStevenK: Well, tail still worked, but pretty much everything else I could find segfaulted in libc6 startup07:27
wgrantI think x-x-v-ati was doing bad thing.07:27
StevenKYet again another reason that fglrx is a pox.07:28
nigelbpox?07:33
nigelbPOS?07:33
wgrantStevenK: Hmm? x-x-v-ati isn't fglrx07:34
lifelessnigelb: http://en.wiktionary.org/wiki/pox07:36
nigelboh.lol.07:45
=== almaisan-away is now known as al-maisan
=== jtv is now known as jtv-C8H10N4O2
=== jtv-C8H10N4O2 is now known as jtv
=== al-maisan is now known as almaisan-away
=== almaisan-away is now known as al-maisan
adeuringgood morning09:05
=== al-maisan is now known as almaisan-away
rick_hmorning11:37
jelmerhey rick_h11:38
nigelbrick_h: I find you online at the start of my day and end of my day.  Do you sleep at all? :)11:40
rick_hnigelb: of course, during your day :)11:40
nigelbheh11:40
rick_hStevenK: still around? Is your combo-url branch updated and ready for review again?12:15
=== almaisan-away is now known as al-maisan
StevenKrick_h: It is, yes. You can also prod sidnei to review my convoy branch.12:54
rick_hStevenK: will do, ok I'll mark that branch as needs review. Thanks12:54
nigelb3513:41
rick_hanyone know a trick to download all translation .pot files for a project/lang at once? https://pastebin.canonical.com/59476/ is the question13:46
rick_hlp-translations-tools just seems to do multi uploads13:46
rick_hand all the wiki info is about downloading one .pot at a time13:46
=== matsubara is now known as matsubara-lunch
rick_hadeuring: deryck gone through RT, new projects, POT translations, and some spam monitoring14:01
deryckrick_h, ack, cool.14:05
rick_hderyck: this is the first one, I wanted to go over the new templates for testing and make sure that they looked good to you: https://code.launchpad.net/~rharding/launchpad/combo_yui_tests/+merge/9147814:40
rick_hso that's the updated templates and a couple of modules changed over to it14:40
rick_hderyck: and this is the next big changeset https://code.launchpad.net/~rharding/launchpad/combo_yui_tests214:40
rick_hwith #3 in the works as well14:40
deryckrick_h, ack, will look now14:40
rick_hnp, heading afk14:40
jcsackettsinzui: saw your notes on my MP; i've pushed up changes addressing them.14:53
sinzuiI have approved14:54
sinzuijcsackett, about the date in the XXX: https://dev.launchpad.net/PolicyandProcess/XXXPolicy14:55
=== al-maisan is now known as almaisan-away
jcsackettsinzui: yeah, i went looking for that after your comment.14:55
sinzuiI think the policy was added after we could not tell what sabdfl, kiko, and stevea meant in their comments14:55
jcsackettsinzui: make sense. i've updated the wiki page too, as the date example for the python was another ambiguous case, and only became clear when you read the TAL example.14:58
sinzuithanks14:58
jcsackettbasically, when dealing with multiple country date formats, numeric dates are problematic. :-P14:58
jcsackettsinzui: do you have time to talk about bug 741234? i'm contemplating tackling that next.15:08
_mup_Bug #741234: product:+code-index merge proposal queries do not show private bugs visible by assignment <disclosure> <easy> <Launchpad itself:Triaged> < https://launchpad.net/bugs/741234 >15:08
sinzuijcsackett, I do15:09
=== matsubara-lunch is now known as matsubara
sinzuijcsackett, confirm you do not see this: https://bugs.qastaging.launchpad.net/gdp/+bug/86458715:19
_mup_Bug #864587: compiz crashed with SIGSEGV in PrivateWindow::configureFrame() <amd64> <apport-crash> <compiz-0.9> <oneiric> <running-unity> <ubuntu> <compiz (Ubuntu):New> < https://launchpad.net/bugs/864587 >15:19
sinzuiand look at this : https://code.qastaging.launchpad.net/gdp15:20
lifelessnigelb: so when did you want this presentation on lp thingy?15:28
nigelblifeless: heh, it was last week.15:29
nigelbI completely forgot.15:30
lifelessah well :)15:30
nigelb^-^15:30
abentleyderyck: my QA's giving some really weird results.  Can we chat?15:40
deryckabentley, sure.  let me just get coffee. was just about to do that.  and then we can chat.15:43
abentleyderyck: cool15:43
rick_habentley: deryck adeuring heads up, back and numb wheee15:51
abentleyrick_h: hehe.15:51
deryckrick_h, oba-kba15:51
deryckabentley, ready now…. mumble or hangout?15:52
abentleyderyck: mumble.15:52
rick_hjcsackett: heads up, StevenK updating the MP for the convoy stuff: https://code.launchpad.net/~stevenk/launchpad/combo-url/+merge/9120315:53
sinzuidanhg, I am going to attempt a last minute change the interactive mockup. Many of the tests I am writing for you require a new project. I think we want a start page where you choose a young or old project so that the data looks right for the task we are testing16:01
danhghey sinzui16:02
danhgOK sure16:02
sinzuidanhg, I think i want to change the names of the people and teams too because the nonsense is a barrier to knowing you have done the right thing. eg football teams and politicians do not on you mind when you are looking for project-related groups16:03
danhgIf it it is difficult to implement these changes, we could start some of the tests off with an ''imagine that this is a new project'' narrative to put testers into the right frame of mind.16:04
danhgIf it can be done though, that's great16:04
danhgSure, the comedy names make it seem 'less real'16:05
sinzuidanhg, okay. I will set a limit of 2 hours to do this.16:08
danhgOK. Thanks sinzui. I'll be working most of this evening so we'll be able to talk things over later than normal.16:10
jcsackettrick_h: i've replied, thanks for the heads up.16:36
rick_hjcsackett: thanks16:36
=== beuno is now known as beuno-lunch
=== beuno-lunch is now known as beuno
jcsackettsinzui (or anyone): we're supposed to be removing remove_security_proxy_and_shout_at_engineer, right? i have stumbled across a use of it and am wondering if i should replace it.17:13
sinzuijcsackett, we can remove them. It is okay to remove the proxy to work with the object, but that naked object should not be return to an ignorant callsite17:19
* jcsackett nods.17:20
jcsackettok, that's what i thought.17:20
jcsackettso when it's just a onetime use to manipulate something in a test, we're good.17:20
sinzuiabsolutely17:20
jcsackettfantastic.17:20
lifelessflacoste: taking cynthia for a bit; not sure when I'll be back to keyboard, I should be on skype though if you want to ping me for our call17:27
jcsackettsinzui: this test look like it's setting up the scenario we discussed? b/c it's passing, and given our test and our assumptions it shouldn't. http://pastebin.ubuntu.com/831640/17:49
jcsacketti've poked at it a bit, and i think either i'm missing something or an assumption is wrong.17:50
=== mpt_ is now known as mpt
sinzuijcsackett, please wait while I stop the bank from contriving a scheme to make me default on my mortgage17:53
jcsackettsinzui: sure, that sounds like a priority.17:55
=== mpt_ is now known as mpt
sinzuijcsackett, I ca talk now18:41
=== matsubara is now known as matsubara-afk
abentleyderyck: changing the feature flags didn't solve it.  Copying the remote feature flags to my dev instance didn't solve it.18:46
abentleyI mean Copying the remote feature flags to my dev instance didn't reproduce it.18:46
jcsackettsinzui: excellent. i was briefly away, but i'm back now.18:48
deryckabentley, hmmm, ok, very weird.19:00
deryckabentley, I've got a call now and I can look deeper after that.19:00
jcsackettsinzui: http://pastebin.ubuntu.com/831740/19:08
sinzuijcsackett, http://pastebin.ubuntu.com/831747/19:16
abentleyderyck: I think it's from dupe detection.19:17
sinzuijcsackett, do you see the linked bu on https://code.qastaging.launchpad.net/~sinzui/gdp/trunk-1/+merge/7765119:19
sinzuijcsackett, https://code.qastaging.launchpad.net/gdp/+activereviews19:29
sinzuijcsackett, https://code.qastaging.launchpad.net/gdp/19:30
sinzuijcsackett, https://bugs.launchpad.net/launchpad/+bug/72378319:34
_mup_Bug #723783: distribution and project group index pages have wrong header <confusing-ui> <distributions> <projectgroups> <trivial> <Launchpad itself:Triaged> < https://launchpad.net/bugs/723783 >19:34
deryckabentley, ok, interesting.19:43
deryckabentley, duplicate comment detection?19:43
abentleyderyck: Yes, I think I saw something about that in the bug message handling.19:51
deryckabentley, that kind of makes sense.  So nothing wrong with your work per se, just the kinds of comments you posted as tests being mistaken as dupe comments?19:52
abentleyderyck: That's right.  Not detected locally, because a unique subject, author, bugtask & bug is used by default.19:53
deryckabentley, right, makes sense19:53
deryckabentley, glad you tracked it down.19:53
* deryck goes offline for school pick up, back soon20:09
abentleyderyck, rick_h: did interrupt duties: bugs, RT, questions.20:47
=== matsubara-afk is now known as matsubara
lifelesswgrant: hey21:54
lifelesswgrant: whats the status of the heat plumbing21:54
wgrantlifeless: I was planning on waiting for a few days of insufficient complaints before ripping it out.21:59
lifelesswgrant: doit22:00
lifelessor at least, lets get the branch ready to troll22:00
wgrantlifeless: Disable the max heat update code, and the aging job?22:02
wgrants/Disable/Delete/22:02
lifelessyeah, and remove the flag22:02
wgrantlifeless: Do you know what's happening with removing the old bug listing code?22:07
lifelessderyck: ^22:07
lifelessderyck: also I imagine its too late today; shall we talk tomorrow?22:07
derycklifeless, yeah, let's chat tomorrow.22:09
deryckand we haven't made plans for removing the old code yet.22:10
deryckhow about "while we're on maintenance"22:10
wgrantThere's a regression which means we're still using the old code on one of the feeds.22:11
deryckah right.22:11
deryckmeant to add that to our board actually22:11
deryckour board is kind of full right now though22:12
wgrantHeh22:12
wgrantlifeless: Like blueprints? :)22:24
wgrantAlthough Linaro seems to be trying to prevent it :(22:24
StevenKrick_h: Still here?22:28
lifelesswgrant: blueprints is very heavily used; there hasn't been a real proposition to *remove* it, ever - but there is one to fold it into the rest of the system better (issuetracker)22:32
StevenKlifeless: Please file bugs about use of sourcecode22:34
lifelessStevenK: please reply to the thread and say 'yay'.22:35
StevenKBah, then I need to find the mail again22:40
StevenKI will when I dig my way out of my warthogs folder22:40
poolielifeless, re policy22:59
pooliei do wonder if there ought to be some at least rough calculation of bugs from a particular type of change22:59
pooliei guess people do this alreday with "X is dangerous" or "Y could close many bugs"23:00
lifelesspoolie: that might be nice too; as jml says though, these things require knowing the future (or at least trying to predict) - so they are intrinsically harder than knowing the now.23:02
pooliewell, that's what i'm trying to get away from23:04
pooliei'm talking about looking only at currently open bugs23:04
pooliefor example,23:05
pooliewe know removal of a moderately-used feature tends to cause about A amount of noise/complaint23:05
pooliewe also know having semi-working hg imports causes B amount of noise23:05
pooliei can't work out how to make this very concrete though23:05
lifelessinteresting to think about though23:06
lifelessno reason we can't evolve the policy as better rules are created / become feasible23:06
mabacA general question for anyone; I have a merge proposal that is approved. Should I do something more to land the change or will that happen automatically?23:08
lifelessyou need an LP core dev to land it for you23:10
lifelessthe person that approved often will, but its best to nag (gently) here :)23:10
StevenKmabac: I'm happy to land for it you, please link me the MP.23:10
mabacStevenK, thanks. it's https://code.launchpad.net/~mabac/launchpad/login-raises-non-string/+merge/9126523:13
mabaclifeless, thanks, that's kind of what I was going for :)23:13
StevenKmabac: Please set a commit message on the MP.23:14
StevenKI'll toss it at ec2, just to be safe.23:14
james_wis https://lp-oops.canonical.com/?oopsid=OOPS-4cc7d8d466ed2a4569ebb00fa667f23e just me because of the number of branches my teams own?23:16
james_whi mabac!23:16
StevenKmabac: Too slow. :-)23:16
mabacStevenK, sorry about that. it's added23:16
mabacStevenK, hmmpf ;)23:16
mabacjames_w, hi!23:16
james_wmabac, how is connect so far?23:16
mabacjames_w, it's great. we miss you of course :)23:17
lifelessjames_w: I don't know :P23:17
lifelessjames_w: but I'll have a look.  You should be able to see it too...23:17
james_wI did look, and that's my guess23:18
lifelessjames_w: do you still own all of ubuntu?23:18
james_wyeah23:18
james_windirectly23:18
james_w~ubuntu-branches, which I am a member of23:18
lifelessthats probably it23:18
james_wI can't see any code indexes, which is mighty annoying23:18
StevenKmabac: That branch is hitting an ec2 instance to run tests first. I'm not expecting any failures, but you should get a mail in about four hours from it.23:18
james_wI was wondering if it was just me, as that would explain why it wasn't fixed yet23:19
mabacStevenK, cool, thanks. it's not the most intrusive change I can imagine. good for me to learn the process before I get to propose any more significant changes.23:20
james_wmabac, say hi to everyone23:20
lifelessjames_w: there is or was a bug about it23:23
lifelessjames_w: its nasty business to fix23:23
mabacjames_w, the Infra team waves back from our hacking room!23:23

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