lifelesswgrant: what do you think? closable?00:03
wgrantlifeless: Needs testing. If it's not closable, then it's a security bug.00:03
* spm needs to fix his losa highlight. "closable" shoudn't be a highlighter....00:03
lifelesshow about closeable?00:03
wgrantlifeless: It's possibly from back in the days when one got notified of duplicates of the master bug if one was subscribed to a duplicate.00:04
lifelesswgrant: yes, its old; thus my question ;)00:04
lifelesswgrant: are you on launchpad-security now?01:08
lifelesswgrant: if not that would explain your confusion when I mentioned the bug before01:08
wgrantlifeless: I have been for a while.01:09
wgrantI get launchpad security mail.01:09
lifelesssinzui: I guess you're off to family time, just saw your reply to my review; if you do want to discuss it now, I'd be delighted02:18
lifelessotherwise, if I don't hear from you I'll reply as much as I can think of via mail and look for you my morning tomorrow02:18
lifelesswhere are my daily oops reports for prod :(02:36
lifelessno landings to devel since tuesday? wtf02:47
pooliehi lifeless02:49
poolieshall we talk now? pots?02:51
lifelessskype is easiest for me02:53
poolielifeless, https://bugs.launchpad.net/bugs/73977202:57
_mup_Bug #739772: Question should be available via the API <Launchpad itself:In Progress by jcsackett> < https://launchpad.net/bugs/739772 >02:57
lifelessjcsackett: you're going to hate me :)03:51
lifelessjcsackett: https://bugs.launchpad.net/launchpad/+bug/74144003:52
_mup_Bug #741440: merge proposal diff only shows once per page load <regression> <Launchpad itself:Triaged> < https://launchpad.net/bugs/741440 >03:52
LPCIBotProject windmill build #96: STILL FAILING in 1 hr 7 min: https://lpci.wedontsleep.org/job/windmill/96/04:16
StevenKWindmill, you make me sad.04:18
StevenKActually, gina's doctest makes me positively suicidal.04:19
StevenKI think I broke it, gina has been running for 11 minutes.04:34
StevenKNo, it's just OOPSing constantly04:35
wgrant:( fake forms04:40
* StevenK burns gina.txt, and then douses it with kerosene.04:44
wgrantWatch out for gina-multiple-arch.txt04:46
wgrantIt may avenge its brother.04:46
StevenK  Ran 1 tests with 1 failures and 0 errors in 18 minutes 25.669 seconds.04:57
lifelessok, time to write more deploy scripts04:57
wgrantHmm. Lots of theme updates.05:10
wgrantNew Ubuntu panel logo, new speaker icons, messaging indicate is now blue...05:10
wgrantWallpaper is... very slightly different.05:10
wgrantIt is almost unnoticably different :/05:12
StevenKThen how did you notice?05:13
LPCIBotYippie, build fixed!05:19
LPCIBotProject devel build #571: FIXED in 5 hr 22 min: https://lpci.wedontsleep.org/job/devel/571/05:19
StevenKUhhh, why is gina's doctest testing Python math?05:27
lifelessbecause twisted numpy accelerator for ssh changes global fnuctions05:29
StevenKThat's so not related.05:32
StevenKlifeless: Can haz review?05:32
StevenKMethinks that means 'No'05:59
lifelessStevenK: check your mail06:25
=== almaisan-away is now known as al-maisan
StevenKlifeless: Thanks06:26
pooliedo mps no longer show the diff inline in the page and only in the popup?06:29
poolieor is this just the problem that the diff is sometimes missing?06:30
wgrantThe diff should always be there.06:34
lifelesspoolie: mps don't have a popup; bug pages referencing mps have a popup06:38
pooliebug search timed out again06:39
poolieok, i think it was just https://bugs.launchpad.net/launchpad/+bug/734629 then06:40
_mup_Bug #734629: empty/useless diff shown for merged proposals <code-review> <confusing-ui> <mail> <Launchpad itself:Triaged> < https://launchpad.net/bugs/734629 >06:40
=== al-maisan is now known as almaisan-away
huwshimilifeless: I have a branch that adds an ajax time log. Here's a screen cap: http://people.canonical.com/~huwshimi/ajax_time_log.ogv07:24
huwshimilifeless: I haven't figured out a way to add a useful identifier to that log yet though07:24
huwshimilifeless: I'd like some suggestions about what other useful info you'd like to see there07:25
lifelesshuwshimi: ----awesome----07:34
lifelesshuwshimi: url07:34
lifelesshuwshimi: and if it oopses the oops id07:34
pooliehuwshimi, wow07:41
huwshimilifeless: Yeah the url is the one I was trying to get before and couldn't. I can get back headers and data though so I should be able to oops info etc.07:47
huwshimilifeless: I'll have a look again at the url07:47
lifelessperhaps we could stash the url in a response header07:47
huwshimilifeless: well I'll try without that first, but it's good to know we can do that07:50
stubBinaryPackageReleaseDownloadCount is updated just over 8 times a second.07:54
lifeless24M rows on qastaging.07:55
stubThat is a batch job or live?07:55
lifelessbatch I believe07:56
stubyer... I was wondering if you can translate that directly to downloads, but I don't think it does07:56
StevenKThat's the job that parses librarian logs, I thought07:57
stubAbout a million downloads counted per day07:59
lifelessstub: hows the dbr looking - have we pushed the numbers down at all recently ?07:59
stubWhich I think means every line counted results in an update. That could be much more efficient if we sacrificed a little RAM.08:00
wgrantstub: We batch the updates.08:03
stublifeless: Roughly the same. Some wins - bug notification has dropped off from chewing ~17% of a core to <1%08:03
wgrantstub: At least it's meant to.08:03
wgrantWe build up a massive dict.08:04
stubwgrant: But the same row might be updated multiple times in the run?08:04
stubwgrant: Or does the dict count everything, then the updates are made at the end?08:05
wgrantstub: It's possible, if the package has been copied.08:05
stubarchivepublisher has dropped off the cpu chewing list too. Was it split into components, or just made awesome?08:07
wgrantstub: Just made a bit less awful.08:07
lifelessstub: when was it doing 17%?08:07
LPCIBotProject windmill build #97: STILL FAILING in 1 hr 9 min: https://lpci.wedontsleep.org/job/windmill/97/08:07
wgrantstub: Each BPRDC should be updated at most once in each batch.08:07
stubhttps://devpad.canonical.com/~lpqateam/dbr/weekly/db-report-2011-03-11-2011-03-18.txt from 2 months ago is what I'm using as a baseline08:07
lifelessstub: thats probably when we had the bad filter code08:07
jtvstub: what say you I lower statistics-logging in LoopTuner to DEBUG2?08:08
stubwgrant: Ok. That makes sense. If it becomes a problem, we could count everything first then do the updates in batches after the count is in.08:08
lifelesswe're going to put the filters back in on monday08:08
stubjtv: I'm not fussed. If it makes things easier to work with, go for it.08:08
wgrantstub: We count everything first, but I guess Storm updates each row separately.08:08
jtvthanks stub08:08
stubwgrant: oic. It is just there are 1000000 rows to update :)08:09
wgrantstub: Heh, not quite that many, I hope.08:09
lifelesswgrant: batch update: insert into temp table (VALUES); select into ...08:10
stubHmm... no... only 250k. So each tuple gets updated 4 times per day. Not sure 100% if that translates to 1 insert and 3 updates.08:11
stubAnyway... not anything to worry about atm....08:11
stubSo 'less aweful' translates to a drop from 70% of a CPU core 2 months ago to no longer being on the list at all.08:13
wgrantstub: Huh, nice.08:14
stubappservers have dropped by 10% too, but that might not be statistically significant.08:15
wgrantstub: ew, distributionmirror shouldn't be that high.08:15
stubwgrant: no, and I think I have open bugs from me bitching about it in the past ;)08:16
wgrantstub: Where are you seeing the archivepublisher drop?08:16
wgrantIt's still on yesterday's, at 70%...08:16
bigjoolsmorning people08:16
wgrantMorning bigjools.08:16
stubAhh crap.... please reverse everything I just said. The tabs where reversed :)08:17
stubSo I guess the old lucille == archivepublisher and no change there08:17
wgrantstub: lucille turned into archivepublisher08:17
wgrantWith no significant change.08:17
stubAnd the appservers are chewing 10% more08:18
stubAnd bug notifications sucks dogs balls.08:18
wgrantIndeed, bugnotification is 40x what it was.08:18
lifelessstub: this is worth a mail to the list +gary -> they want to add the mass filters to prod again; this high utilisation is a red flag for that08:20
jtvwgrant: do we still need commercial-compat.sh?08:21
wgrantjtv: Yes, until Dapper dies in three months.08:21
jtvThree months, egad08:22
bigjoolsrun-parts ...08:22
jtvhi bigjools.  wgrant: it's run at a somewhat awkward point in cron.publish-ftpmaster, and things would become a lot simpler if I could move it up or down just a bit.08:22
jtvSo I was wondering: does it have to be run after the new dist directories are moved into place?08:23
jtvOr alternatively, does it have to be run before the ls-lR.gz files are generated?08:23
wgrantIt should be able to run after ls-lR, I think.08:24
jtvbigjools: seeing 4 streaks of runparts invocations at the moment, 3 of which run twice.  Looking at how that number might be reduced.08:24
wgrantLet me check.08:24
bigjoolsjtv: hmmm we can surely consolidate some08:24
jtvI should hope so08:24
bigjoolsI thought there should only be 208:25
bigjoolsremember that run-parts orders stuff if you want08:25
jtvwgrant: I'm rewriting cron.publish-ftpmaster in python, and the plan is to "outsource" a lot of the work to run-parts directories.08:25
wgrantjtv: Ah.08:25
jtvbigjools: yes, but there's stuff inbetween that doesn't look like it belongs in run-parts.08:25
bigjoolsit's more like adding hooks for non-generic stuff to live in08:25
jtvSuch as moving the dists directory into place: can we move that into run-parts?08:25
bigjoolsno, that's generic08:26
bigjoolsbut we might be able to move things around08:26
wgrantjtv: So, commercial-compat has to run before ls-lR, and after dists08:26
wgrantjtv: Unless you make it work on dists.new instead.08:26
wgrantOr delete partner08:27
bigjoolsit's odd that nobody has had errors about incomplete PPA dists08:27
bigjoolssince we don't do them atomically08:27
wgrantbigjools: buildds do occasionally.08:27
wgrantBut they update infrequently and quickly.08:27
bigjoolsah interesting08:27
bigjoolson another note, isn't it a royal pita to see your ec2 land succeed and then pqm is in testfix!08:28
wgrantYes, someone left it in testfix overnight :(08:28
wgrantIt failed like half an hour after I last checked it, and was failed when I started this morning...08:29
wgrantWhich makes me rather sad.08:29
bigjoolsStevenK: hello08:30
jtvbigjools, wgrant: right now the basic structure looks like http://paste.ubuntu.com/584690/08:30
wgrantjtv: You will ideally be able to move signReleases into publishDistro (for PPAs it's already there)08:31
jtv4 stretches of "runparts" code, which is a bit much.  The smallest change I see that would help with that is to move installDists() down one step.08:31
wgrantWe can also hopefully get rid of copyIndices, because it's stupid.08:31
wgrantWhat's installDists?08:31
jtvMoving signReleases into publishDistro doesn't help.08:31
bigjoolswhy is 4 too much?08:31
jtvwgrant: installDists moves the dists directories into place.08:31
wgrantjtv: So, installDists and removeUncompressedListings and signReleases all go into publishDistro.08:32
jtvbigjools: there's no such thing as "too" much, but maintenance is going to be easier when it's fewer.08:32
jtvBecause it's hard to explain exactly what each of those 4 run-parts directories is going to be fore.08:33
bigjoolsso naming stuff is hard? :)08:33
wgrantjtv: I think there's only one run-parts.08:33
jtvwgrant: what about copyIndices?08:33
wgrantPost-publication stuff.08:33
wgrantjtv: Does that put stuff in dists?08:34
wgrantThat would be a no.08:35
jtvif [ "$LPCONFIG" = "$PRODUCTION_CONFIG" ]; then08:35
jtv    echo "$(date -R): Copying the indices into place."08:35
jtv    rm -f $INDICES/override.*08:35
jtv    cp $OVERRIDEROOT/override.* $INDICES08:35
wgrantSo it doesn't have to run before installdists.08:35
jtvis what it does.08:35
wgrantSo we have this:08:35
wgrantThe end.08:35
wgrant"if not self.options.security" is implemented by the Python script calling run-parts with an env-var which tells them whether it's a security run or not.08:36
jtvWell, it's not quite that simple of course because we need to do most of this twice per script run.08:36
wgrantLong-running scripts can avoid running if the security flag is set.08:36
wgrantWe could also pretend that the concept of a security run doesn't exist.08:37
wgrantjtv: Why must we do it twice?08:37
jtvTo expedite security uploads.08:37
wgrantIs that actually used?08:38
jtvYou're in a better position to answer that than I am.  But it's in the original script and it's in the design for its replacement.08:38
jtvSo you could say it's already automatically used, but we don't know whether we rely on it in any way.08:39
wgrantKeep in mind that this script was written in 2005. Most of the assumptions it makes can probably be brought into question.08:39
wgranteg. dsync kill it with fire.08:39
jtvThat's exactly what I'm doing here: questioning the assumptions.08:39
jtvdsync goes into run-parts where it's not my problem.08:39
jtvSigning of releases is something I'd like to divest myself off as well though.08:40
wgrantWe already have the code to do that.08:40
wgrantBut the primary archive doesn't use it.08:40
bigjoolsI don't want jtv's job complicated with that right now08:41
jtvWe already have the code to divest ourselves of it?  Or we already ahve the code to sign releases?08:41
wgrantjtv: The signing code.08:41
bigjoolsby all means file a bug and do it, but not in this change08:41
jtvI also imagine there's the variable of what GPG key is to be used.08:42
wgrantI think we should be fixing publish-distro before we try to make its wrapper sensible...08:42
bigjoolsI disagree08:42
wgrantThe wrapper can be a lot more sensible if it doesn't duplicate functionality that's already in publish-distro.08:42
jtvWhat I'm doing is at least bringing out the structure in what the old script does, and allows us to re-think these assumptions.08:42
lifelessbigjools: btw - we got the go ahead to combine the soyuz (s)ftp services and uploader into one08:43
jtvAlso, at least now we can run tests.  In temporary directories.  Without global setup and writing to /srv/launchpad.net.08:43
bigjoolslifeless: I am fearful but ok08:44
lifelessbigjools: we can always split it into two again - or optimise08:44
bigjoolslifeless: this will break the "you get an email in a few minutes" thing more often08:44
lifelessbut it will be a simpler initial config, which is important at the moment08:44
bigjoolslifeless: did you ask anyone why they were split like that in the first place?08:45
lifelessbigjools: you asked elmo, I didn't see a reply from him08:45
bigjoolsthat's a no then? :)08:45
lifelessbigjools: if he doesn't know, I suspect noone is left that knows ;)08:45
lifelessbigjools: we can dig further - I've no objection to more info08:47
bigjoolslifeless: I think it would be wise08:47
lifelessbigjools: the main point for me is that we have the stakeholder buy in to do whatever tradeoff make the most sense for now08:47
bigjoolsjust from being burned in the past by seemingly innocuous changes like this08:47
lifelessbigjools: this one seems pretty big to me :) - if we do it it would be staged, easily reversible etc08:48
bigjoolslifeless: did we get a reply from platform?08:48
lifelessbigjools: silence (neither a 'whaaa' nor anything specific)08:48
bigjoolsI think a direct question would be good then08:48
lifelessbigjools: marjo I think, said to me that the mail was forwarded to platform directly08:48
bigjoolsthey may not have seen it08:49
lifelessbigjools: he cc'd me; I think they have seen it - he pulled out the TL;DR summary for us08:51
bigjoolswhat is this tl;dr fad08:52
lifelessbigjools: I'll check with elmo and flacoste and so on when we've finished the single-threaded stack08:52
lifelessbigjools: too long didn't read?08:52
bigjoolsyes I know what it means :)08:52
bigjoolslifeless: anyway, go ahead with the change, I've raised any concerns I may have08:53
adeuringgood morning08:53
lifelessbigjools: I'll drag you into it when we get up to making the uploaders HA during deploys - i don't think any decision needs to be made yet.08:54
lifelessbigjools: I'm just happy we have a broad set of options08:54
bigjoolslifeless: indeed!08:54
jtvwgrant: what does dsync-flist do exactly?08:54
bigjoolsoptions are good08:54
bigjoolsthat's why I run KDE08:54
wgrantjtv: hardlinks common files.08:54
wgrantjtv: It saves roughly a megabyte, I believe.08:55
jtvwgrant: sorry just to be clear, that's a megabyte of L2 CPU cache right?  ;-)08:55
jtvSo… could I just simply kill it, drop it from the script, do away with it, no questions asked?08:56
bigjoolsI doubt anyone would notice it missing08:57
jtvDon't security uploads need to germinate?08:58
jtv(And by the way, triggering mirrors looks like a prime candidate for "&" in the run-parts script)08:59
jtvGermination is not generic to all distros, right?09:00
wgrantMost will probably want it. But they should run the script themselves, since it's fairly specific, and I wish it would get out of our tree.09:00
bigjoolsjtv: mirror trigger is very quick09:00
bigjoolsgermination is distro-specific09:01
bigjoolsthat script is not really owned by LP even though it's in our tree09:01
jtvOh, ISTR back in 2006 we tried to run some of this stuff on a non-Canonical machine and it just froze up trying to connect to each mirror.09:01
jtvSo I sort of remembered it as being slow.09:01
jtvbigjools: what was your verdict on signing?  run-parts or generic?09:05
bigjoolsjtv: run-parts for now since it's less work and won't mess up this change.09:06
jtvWell I'm not 100% sure it's less work.  I see 2 options:09:06
jtv1. Do this in run-parts, and have another script in the same dir that finds and removes the uncompressed Sources/Packages files.09:07
jtv2. Implement it in the script itself, and have one run-parts directory less.  (The removing of uncompressed files could also go into the script, or into the next run-parts stretch)09:08
wgrantbigjools: Oh yeah, it turns out that delayed copies are now much slower than direct copies.09:09
bigjoolswgrant: !09:09
bigjools"delayed" would imply that :)09:09
wgrantSince delayed copies still scale by number of BPRs.09:09
bigjoolsso if you can guarantee that the security team can unembargo a kernel using a direct copy, please remove delayed copies09:10
bigjoolswe can get rid of that re-upload crap too09:10
wgrantCan't yet. Need to move notifications and overrides into direct copies :(09:10
wgrantAnd that, yes.09:10
bigjoolswe can get rid of that anyway09:10
wgrantI'm going to see if doing constant-query overrides is easy.09:10
wgrantIf so, I'll JFDI.09:10
wgrantbigjools: I guess this will also have the convenient effect of stopping copy-package from breaking the archive.09:18
pooliehi all09:18
henningeHi adeuring!09:20
henningeadeuring: your soyuz translations upload branch landed! Good job on finding the right spot. ;-)09:21
adeuringhenninge: thanks :)09:26
henningeadeuring: Unfortunately it seems to be doing a little too much ... :(09:26
adeuringhenninge: really?09:26
henningeadeuring: we still need the *templates* to be uploaded09:26
adeuringhenninge: ah, right09:26
henningeadeuring: so addOrUpdateEntriesFromTarball needs to filter only for pot files.09:27
adeuringhenninge: right09:27
henningeactually, there is a function that checks that09:27
henningeI was just looking for that09:27
henningeadeuring: TranslationImporter.isTemplateName09:29
henningeadeuring: you should be able to add a flag 'only_templates' to addOrUpdateEntriesFromTarball and use that in _isTranslationFile.09:31
adeuringhenninge: right09:31
henningeadeuring: I am sorry but this really needs to be done before this gets rolled out. Let me file a bug for it.09:31
=== almaisan-away is now known as al-maisan
pooliejml, hi?09:37
henningeadeuring: bug 74157109:40
_mup_Bug #741571: Translation templates should always be uploaded from soyuz builds. <regression> <upstream-translations-sharing> <Launchpad itself:Triaged> < https://launchpad.net/bugs/741571 >09:40
bigjoolswgrant: sorry was otp, to which breakage are you referring, for there are many :)09:44
wgrantbigjools: The "oh, you wanted to copy a universe package from a PPA? let me just silently promote that for you" one, mainly.09:44
henningeadeuring: I touched that code, too, because I have removed the translationsharinginfo module. But that branch is just landing.09:53
henningeadeuring: are you busy now or should I fix that?09:54
adeuringhenninge: let's dice ;)09:54
henningeadeuring: I roll a W20 ...09:55
henningeadeuring: I win ;)09:55
adeuringhenninge: 20*19*18... is a huge number09:55
adeuringhenninge: ok, so you'll fix the bug?09:55
henningeadeuring: yeah, I just thought of some other optimization for my last branch that I could include.09:56
adeuringhenninge: cool, thanks09:56
dpmhi jml, did you have the chance to have a look at signing up for a session on Launchpad for Ubuntu AppDeveloperWeek or finding someone else to run it?10:01
wgrantbigjools: Your sync button is buggy.10:27
wgrantbigjools: syncSources copies the latest published source in the archive, which could be, say, a security upload.10:28
wgrantIt does not discriminate by series or pocket.10:28
bigjoolsI expected I'd need to fix it later to sync the versions specified10:30
jtvAny reviewers available?  https://code.launchpad.net/~jtv/launchpad/bug-741585/+merge/5467410:32
bigjoolswgrant is like the eye of Sauron10:33
* jtv tries to see a way that comparison works10:33
jtv—but fails10:33
wgrantbigjools: Sadly not.10:33
bigjoolsI have a vision of an eye sweeping the Launchpad landscape looking for bugs10:34
jtvbigjools, you've been reading too much of that "LotR from Sauron's perspective" stuff10:35
bigjoolsor maybe I feel like Frodo trying to sneak bugs past the eye :)10:36
wgrantOne does not simply land hacks into Mordor.10:38
wgrantStevenK: Do you have a solution to your version issue? I see two possibilities.10:39
jtvbigjools, another Q about publish-ftpmaster: we only need to call process-accepted once, despite the "run once for security, run once for everything" structure, right?10:46
=== al-maisan is now known as almaisan-away
jtvstub, care to review my little logging cleanup?  https://bugs.launchpad.net/launchpad/+bug/74158510:47
_mup_Bug #741585: FakeLogger doesn't quite conform to logging API <Launchpad itself:In Progress by jtv> < https://launchpad.net/bugs/741585 >10:47
jtv1bigjools: if we won't be needing commercial-compat.sh for much longer, maybe I should just hard-code it in and we can drop it from the script later.  It actually makes things simpler.11:02
jtv1(In the long run, that is)11:02
=== jtv1 is now known as jtv
jtvI can just add an "if we're doing ubuntu" guard around it, and add an XXX.11:03
deryckMorning, all.11:04
stubjtv: k11:04
jtvhi deryck11:04
stubjtv: I thought I fixed it so our custom logger class is registered so we shouldn't see any 'standard' loggers?11:07
jtvstub: but by "our custom logger class" you don't mean the FakeLogger, do you?11:08
jtvOh, I see what you mean11:08
jtvyou mean I could just call debug2.11:08
stubThat should work, yes.11:08
jtvIf that's safe, then I'll do it that way.11:08
stubIf it doesn't work, there is a bug somewhere (retrieving a logger through some weird mechanism?), but that could wait for another branch11:09
jtvstub: I take it that debug2 will call self.log(DEBUG2, *x, **y)11:09
jtvI didn't run into a problem with that; I was just being conservative.11:09
jtvRunning tests.11:10
stubActually, according to lp/services/log/loglevels.py it is _log for reasons I don't recall but might be documented in the python reference11:10
stubOr maybe it was the sourcecode...11:11
stubIts registered with the logging module in lp_sitecustomize.py anyway11:11
jtvstub: AttributeError: RootLogger instance has no attribute 'debug2'11:12
stubWith the normal logger or your FakeLogger with debug2() added?11:13
jtvProbably the normal logger.11:13
jtvI don't think my FakeLogger has debug2 though.11:13
jtvThere's nothing overriding the logger in that test.11:14
stubYou might need to add it, or make it inherit from LaunchpadLogger instead of the logging module class.11:14
jtvOr call log() and have it just work.  :)11:14
stubI fixed this so I didn't have to keep seeing log(DEBUGX, 'foo') everywhere ;)11:15
jtvAlso, I don't _think_ the FakeLogger has a debug2 method but I'm not sure11:15
stubYes, you will need to add debug2 to FakeLogger if it isn't there. If it isn't there, it is broken as it isn't a complete mock of the real thing.11:16
jtvI could fix this up, but that would start to look like a rabbit hole and I'd prefer to be unblocked on my main job (which is a scary soyuz script overhaul)11:16
stubYou have a branch that mainly consists of making things less readable, when adding a three line method on FakeLogger would keep the readability and fix a bug.11:18
stubWell.. not mainly. You have only changed two .debug2 callsites to use the old syntax.11:20
jtvBut the problem I just ran into is not the FakeLogger.  So fixing up the FakeLogger wouldn't fix that one; there is yet another bug to fix.11:22
jtvThe logger I seem to be getting (and which doesn't seem to have debug2) is canonical.launchpad.scripts.log.11:25
stubjtv: In what cases do you find you have None as a level?11:25
jtvI'm not aware of any.  Not sure that deserves to be in there.11:26
stubok. Add a comment there mentioning that, or remove it.11:26
jtvI'll remove it.  It's a bit of legacy crud from jml's merging of fake loggers.11:27
jtvAt least one of the old fake loggers printed "log>" instead of a log level.11:27
stubjtv: So it looks like lp_sitecustomize.py installs the custom class, and instances are correctly returned for new loggers but the root logger isn't updated.11:29
jtvI have no idea what you just said.11:29
stub(testing with 'make harness')11:29
stubThe registration of our custom logger class works for everything except the root logger, which is bogus.11:30
jtvSounds like it… what can we do about it?11:30
stubSurprised this hasn't fallen over before.11:30
stubI'll have a quick poke to see if it is an easy fix, or open a bug which you can cite in your branch.11:30
jtvI've added the new log levels.11:32
bigjoolsjtv: sorry just saw your questions11:35
bigjoolsjtv: p-a only needs to run once11:36
jtvRunning it from the script does introduce the PATH question again, of coursee.11:36
jtvOh, different question :)11:36
bigjoolsjtv: embedding commercial compat is ok11:36
bigjoolsbut file a bug of course :)11:37
jtvThat brings us down to 2 run-parts dirs: after publish-distro and at the very end.11:37
bigjoolsrock on11:38
jtvFor the latter, the scripts will be told in some way whether this is a security-only expedited publishing or a proper full one.11:38
bigjoolsso post-publication.d and pre-exit.d ?11:38
jtvI still need to check whether all of the parts break down neatly into archives.11:38
jtvSomething like that, I guess.11:38
jtvbigjools: what about the renaming of dists to dists.new?  That looks like it needn't be repeated either.11:40
jtvI should say: renaming of dists to dists.new plus subsequent rsync from distsroot.11:41
bigjoolsnot sure11:41
stubjtv: So https://pastebin.canonical.com/45170/ looks like the fix, and works with bin/harness. Want to dig the rabbit hole a little deeper and save me making a separate branch?11:45
* jtv waits for that page to load…11:46
* jtv waits for openid…11:48
jtvah, there's my page!11:49
jtvstub: I realize this is piling work onto you, which pains me, but I'd really rather not risk the potential Q/A fallout while I'm doing soyuz feature work.11:51
* stub opens a bug11:51
* jtv gets ready to xxx the bug number11:52
stubBug #74165011:54
_mup_Bug #741650: root logger is not a LaunchpadLogger <Launchpad itself:Triaged> < https://launchpad.net/bugs/741650 >11:54
jmljtv: thumper's merging11:58
stubjtv: Copywrite in logger.py should be 2010-2011, not 201111:58
jtvmy apologies11:58
jtvstub: fixing11:59
=== henninge_ is now known as henninge
henningeadeuring: I am done. Want to review?12:00
henningeadeuring: https://code.launchpad.net/~henninge/launchpad/devel-741571-filter-templates-in-upload/+merge/5468812:00
adeuringhenninge: sure12:00
henningeadeuring: I'll be out to lunch now, you can ask questions later.12:01
henningeadeuring: but I think this is a pretty clear branch, I really enjoyed coding this.12:01
stubjtv: Thats it. r=stub.12:01
jtvstub: thanks—changes pushed, on the way to EC2.12:01
=== henninge is now known as henninge-lunch
stubjtv: For lint issues like the one you mentioned, I usually go with whatever makes lint shutup.12:02
jtvstub: me too, usually.  This is the one exception.  I won't add warnings like that, but I'm hesitant to clean them up.12:03
=== nigelbabu is now known as nigelb
stubjtv: What test are you using for this? I want to test my branch12:13
gary_posterstub, thanks for heads up.  Remind me where the db utilization report is, please?  I'd probably like to talk to you about your concerns too, once I've gotten more familiar with the data.12:14
jtvstub: I simply replaced the log calls in looptuner.py with debug2 calls and then ran looptuner.txt.12:14
stubgary_poster: https://devpad.canonical.com/~lpqateam/dbr/12:14
stubjtv: ta12:14
stubgary_poster: Its not causing problems yet, but it certainly should be looked into since it is a big jump and affects our scalability. Robert asked me to raise it because of the work I believe you are doing.12:16
gary_posterstub, right.  We are doing more work now on the asynchronous side.  It is doing features that have been requested, of course.  Do you have any concrete suggestions on next steps?  The only thing I know of is to try to take a glance at the pertinent code and see if it can be simplified, but that seems fairly vague.12:19
stubgary_poster: If we know what extra work it has started doing in the last two months, then we can tell if the extra load is justified or not. I don't know what it is doing - I just see the percentage.12:21
stubgary_poster: I think Robert's main concern is that we are currently in a performance drive, and shouldn't be landing code that makes performance worse (which it seems we have)12:23
jtvbigjools: do we copy overrides only for the main archive?12:23
gary_posterstub, even asynchronously, for requested features?  How do we know that this is a problem?12:23
bigjoolsjtv: copy?12:23
gary_posterAmusingly to me, two months ago would be about the week when we started working on this, of course. :-)12:23
jtvbigjools:     cp $OVERRIDEROOT/override.* $INDICES12:23
jtv$INDICES is in the main archiveroot12:24
bigjoolsjtv: overrides only apply to main archives12:24
jtvPerfect.  Thanks.12:24
bigjoolsso far anyway :)12:24
gary_posterah it's less than 2 mo though...12:24
jtvWell, not perfect, but then I know what to implement.  :)12:24
stubgary_poster: You would have to discuss features vs. performance drive with lifeless :-) I don't think it is affecting production performance yet, but it has lessened our scalability somewhat.12:25
gary_posterack stub.  ok thanks.12:25
stubgary_poster: It might just be we are missing an index or similar if we can work out what has changed and what we are doing differently now. Or maybe we just have to put up with it, because the extra work is required for some reason.12:26
stubgary_poster: The two months figure I got because today I compared this weeks report with a report from two months ago. It could have spiked any time between then and now.12:27
gary_posterunderstood stub.  I have a suspicion I know the cause, and will investigate and report back on the list.  Thanks again for the heads up.  It started rising in early March.  https://devpad.canonical.com/~lpqateam/dbr/weekly/db-report-2011-03-04-2011-03-11.txt12:27
stub(In fact, first of all I erroneously compared them in reverse and was singing the praises of whoever stopped bugnotifications from chewing on the cpu :) )12:28
stubjtv: https://code.launchpad.net/~stub/launchpad/trivial/+merge/54692 (to try landing tomorrow after your branch)12:43
jtvstub: I know, I'm already looking12:43
jtvstub: does lp_sitecustomize get run by scripts as well?12:44
jtvNot sure it matters,12:44
jtvsince we seem to have some logging magic for scripts anyway that probably already sets things right there12:44
stubjtv: Its buildout magic. Everything in our tree gets it12:45
stub(unless you forget python -S or _pythonpath)12:45
jtvstub: approved12:46
stubta. I guess I can punt it to ec2 land to see what explodes now - if your branch fails, this one will fail too since I merged yours in.12:47
jtvstub: looks like my instance never fully set up, but since my review is also approved, "ec2 land" should just land them both.12:48
jtv(Unless it doesn't want to do that across users)12:48
stubjtv: Actually, I'll wait. Better two separate landings in case mine needs to be backed out.12:48
jtvI should get a chance to send it in tonight.12:48
jmlCan I get a review of this please: https://code.launchpad.net/~jml/launchpad/test-timeout-505913/+merge/5459812:51
=== mrevell is now known as mrevell-lunch
abentleyderyck: chat?13:06
deryckabentley: sure.13:07
deryckabentley: let me head to the mumbler13:07
jml'ec2 test' is complaining about me not having an SSH agent running. But I do.13:18
jtvbigjools: I'm about to head off.  If you look at lp:~jtv/launchpad/bug-55798 at the last dozen lines of lib/lp/soyuz/scripts/publish_ftpmaster.py you'll see the structure I have now, which is thankfully shaping up to be pretty simple.13:19
jmlmaybe I need to set an env var or something13:19
=== jcsackett changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | firefighting: - | On call reviewer: jcsackett | https://code.launchpad.net/launchpad-project/+activereviews
jmlhuh. my keys weren't in it. odd.13:20
wgrantjml: Natty?13:20
jmlwgrant: indeedly13:20
wgrantjml: Unity been crashing?13:21
jmlwgrant: Does a bear... well, you know the rest.13:21
wgrantIt sometimes doesn't restore the environment properly.13:21
=== almaisan-away is now known as al-maisan
jmlwgrant: ok.13:21
jmlanyway, running ssh-add makes everything better. just going to test my branch that fixes the intermittent test_timeout failure13:22
jmlmaybe it'll be done by the time it gets reviewed.13:22
leonardrabentley: if you can take another look at https://code.launchpad.net/~leonardr/launchpad/explicit-versions/+merge/54558, i can get it into ec2 land13:23
leonardrwallyworld, you run tests with bin/test13:28
abentleyleonardr: looking13:31
abentleyleonardr: r=me13:33
gmbjcsackett: Can I get a review of https://code.launchpad.net/~gmb/launchpad/fix-fragile-link-bug-741645/+merge/54698 please? Only 20 lines.13:55
jcsackettgmb: sure thing.13:55
=== henninge-lunch is now known as henninge
jmlabentley: could you please review https://code.launchpad.net/~jml/launchpad/test-timeout-505913/+merge/5459813:59
abentleyjml: sure, but I've got a standup first.13:59
jcsackettgmb: r=me.13:59
gmbThanks jcsackett13:59
jmlabentley: thanks.13:59
jmlwhy am I allowed to change translations settings for empathy (upstream) on lpnet but not qastaging?14:06
jml(https://translations.launchpad.net/empathy vs https://translations.qastaging.launchpad.net/empathy)14:06
=== mrevell-lunch is now known as mrevell
bigjoolsjml, wgrant: I have to ssh-add every session because the stupid script assumes I'm running Gnome desktop14:15
jmlbigjools: "stupid script" == utilities/ec2?14:15
bigjoolsjml: I'm not sure which part exactly, something to do with one of the included modules I expect14:16
bigjoolskeyring wants to use the gnome agent as well, instead of the kwallet14:16
jmlI'm guessing this isn't https://bugs.launchpad.net/launchpad/+bug/57711814:17
_mup_Bug #577118: ec2test barfs if the ssh-agent keys are stored in the wrong order <build-infrastructure> <ec2test> <lp-foundations> <Launchpad itself:Triaged> < https://launchpad.net/bugs/577118 >14:17
bigjoolsdon't think so14:19
benjibigjools: I've added a section to https://dev.launchpad.net/LandingChanges that has instructions for telling keyring to use unencrypted flat files for credential storage so you don't have to type in passwords or fiddle with keyrings14:19
bigjoolsbenji: yep, I saw that thanks.  I don't mind that so much as the problem with the ssh keys14:19
sinzuijcsackett: ping14:20
jcsackettsinzui: good morning.14:20
sinzuijcsackett: do you have time to talk about a branch I am starting: http://pastebin.ubuntu.com/584441/14:20
jcsackettsinzui: sure.14:22
* jcsackett starts mumble14:22
bigjoolsderyck: so, looking at Selenium yet? :)14:29
deryckbigjools: I would be interested in Selenium, definetly. :-)14:31
bigjoolsit was in reference to your last tweet/fb14:32
deryckbigjools: other parts of Canonical are using it now.  Windmill is all but dead, except for us14:32
deryckbigjools: right :-)14:32
bigjoolsyeah, it's not developed any more even, I heard?14:32
deryckright, it's pretty dead, I think14:32
deryckI just don't have an easy path off of Windmill14:32
bigjoolsexcept we have a twitching limb14:32
bigjoolsI'll see if my team can help with this when we get to the feature review stage14:33
deryckbigjools: oh, dude!  That would be wonderful.14:33
bigjoolsdon't get too excited, it's a month away or so :)14:34
deryckheh, fair enough :-)14:34
bigjoolswhen we discussed it we were all of the opinion that we should not flog the dead14:34
deryckI think the important thing is to get it added as an option for these sort of tests and then get an example test going.14:36
deryckand not make the same mistakes again :-)14:36
derycke.g. maybe we should consider running them separtely from our normal tests, decoupled from the Zope test runner14:36
deryckthey have to block in some way to be useful, but exactly what that blocking mechanism is good be looked at, too.14:38
bigjoolsI'd be willing to do that14:38
derycks/good/could/ (for my last one)14:38
bigjoolswe need to look at how many of our landed branches would need that sort of test14:38
bigjoolsworth an experiment!14:38
deryckvery much so!14:39
deryckselenium would, I think, allow us to have real web page integration tests that were more stable, rather than the minimal XHR integration test I'm preaching for now14:39
bigjoolsunfortunately I know next to bugger all about the whole JS environment so I can't be very helpful right now.  I aim to delve into this in more detail soon though.14:39
deryckI'm just thrilled to have someone express an interest :-)14:40
bigjoolsha :)14:40
abentleyjml: what's _logTimeout for?14:43
bigjoolsjames_w: it struck me that there's nothing in the derived distros LEP about considering pockets when working out package differences.  Right now it'll just grab whatever was most recently published but I'm not sure if that will always be what's required.  Any thoughts?14:45
james_wbigjools, could you expand please?14:45
bigjoolsjames_w: ok for example say the most recent upload is to -backports.  It'll show up as a diff but won't say it's from -backports.14:46
bigjoolsyou may not want to sync from backports :)14:46
james_whmm, indeed14:46
bigjoolsso I'm trying to flesh out the use cases here so we can work out what to change to make this work better14:47
james_wmy initial guess is that you would want an item for each pocket that had a new version published14:47
bigjoolsthat's what I thought - you'd end up with multiple rows for the same package showing in the table of diffs14:48
james_w"-security has <this> and -updates has <that>" and you can choose which to sync14:48
=== salgado is now known as salgado-brb
james_wif you chose the later one then the other would go away14:48
abentleyjml: it seems test_timeout_short has a lot in common with test_timeout.  Could we extract the common code to a helper?14:49
james_wit's not quite right, but I don't see a better way14:49
abentleyjml: IIRC, raising an exception meant that the oops would contain a traceback showing what code had timed out.  It seems a shame to lose that.14:52
=== gary-lunch is now known as gary_poster
bigjoolsjames_w: I'll ask a few more people and get some consensus14:56
=== matsubara is now known as matsubara-lunch
bacgary_poster, jcsackett: either of you going to trizpug tonight?15:32
jcsackettbac: i'm not sure. there's a decent chance of 'no'.15:33
gary_posterno. previous commitment, bac (to painting a mural in the baby's room)15:33
bacjcsackett: that's my inclination too15:33
jcsackettbac: in my case, i need to contact one of the other splatspace members. he's technically the point man for the meeting, but is not always on time, and someone has to be there to open doors.15:34
=== al-maisan is now known as almaisan-away
bacjcsackett, gary_poster: on an unrelated note, andrea from lanter is a james beard finalist.15:35
gary_postercool :-)15:35
=== deryck is now known as deryck[lunch]
benjiDoes allhands eat newlines in objective "Progress Notes" for anyone else.  It really hurts the readability of my text.15:48
=== salgado-brb is now known as salgado
=== Ursinha is now known as Ursinha-afk
=== Ursinha-afk is now known as Ursinha
=== deryck[lunch] is now known as deryck
=== Ursinha is now known as Ursinha-lunch
jcsackettbac: confirmed, i won't be there tonight.16:55
* jcsackett is having a braindead day.16:59
adeuring1abentley: my branch lp:~adeuring/launchpad/js-translation now also shows the corect "add packaging" icon17:00
bigjoolsyou can't beat a good refactoring for satisfaction17:03
jmlabentley: have you had a chance to look at that branch?17:03
LPCIBotProject windmill build #98: STILL FAILING in 1 hr 9 min: https://lpci.wedontsleep.org/job/windmill/98/17:33
bigjoolsjml: too late17:49
bigjoolsI already submitted a fix17:50
abentleyjml: I did, and I asked you some questions about it.17:50
jmlbigjools: oops. missed your mail (Gmail put it into a different thread :()17:50
bigjoolsjml: awesome Gmail :)17:51
bigjoolsjml: except PQM just blew me out, grar17:51
jmlabentley: on the channel?17:51
jmlahh yes.17:52
jmlabentley: logTimeout is for making sure the logged oops is a TimeoutError, not ProcessDone(42)17:52
jmlabentley: the tests are fairly short. I guess I could put the logger stuff in setUp and make a custom assertion. Not sure whether that would be better.17:53
jmlabentley: re <abentley> jml: IIRC, raising an exception meant that the oops would contain a traceback showing what code had timed out.  It seems a shame to lose that.17:54
jmlabentley: I agree, it is a shame. I'm not sure we can do that and avoid the race.17:55
* jml tries something17:56
jml(although I don't really know how to verify the traceback17:56
abentleyjml: would it be crazy to log the oops in a signal handler?17:56
jmlabentley: I don't *think* so. The caveats about complexity that we discussed the other day would apply.17:57
jmlabentley: I was thinking of try: raise TimeoutError; finally: os._exit(TIMEOUT_CODE), or something similar.17:58
abentleyjml: I don't understand how that would result in a traceback in the oops.17:58
bigjoolsright, time to make like a shepherd.  Good night all.18:00
jmlabentley: well, probably something more like finally: reactor.callLater(0, os._exit(TIMEOUT_CODE))18:00
abentleyjml: Ah.18:00
jmlerr... reactor.callLater(0, os._exit, TIMEOUT_CODE)18:00
jmlabentley: so the error handling would get a chance to run, and if it doesn't then the next thing that happens is a hard stop.18:01
abentleyjml: works for me.  Works for you?18:01
jmlabentley: if it works at all :)18:01
jmlhmm. the test passes, but I suspect it's logging two TimeoutError oopses.18:04
jml(really hard to actually check for that)18:04
jmlabentley: yeah. it logs two oopses in that case.18:12
abentleyjml: I guess we want that, since the first oops isn't guaranteed to work?18:14
jmlabentley: maybe. I honestly don't know. Two oopses seems less good than one. But maybe the extra information is worth it.18:15
abentleyjml: You could use a different exit status if the exception handler succeeds.18:16
jmlI don't know how to log oopses safely from the signal handler (both because of signal handling but also because logging an oops requires a substantial amount of infrastructure that might not be present, such as parsed config)18:16
jmlabentley: hmm.18:16
jmlabentley: I'll give that a try.18:17
jmlabentley: this is approaching the complexity of the "set some state" solution I discussed the other day, though18:18
=== Ursinha-lunch is now known as Ursinha
jmlabentley: I can't make this work without making a mess.18:24
abentleyjml: I think getting the traceback is pretty important, so I can accept the two-oops solution.18:26
jmlabentley: OK. I'll add some comments.18:26
jmlabentley: http://pastebin.ubuntu.com/584950/18:32
abentleyjml: r=me.18:32
jmlabentley: thanks.18:32
LPCIBotProject windmill build #99: STILL FAILING in 44 min: https://lpci.wedontsleep.org/job/windmill/99/18:45
LPCIBotProject windmill build #100: STILL FAILING in 43 min: https://lpci.wedontsleep.org/job/windmill/100/19:29
leonardrbenji, i'm having a problem receiving zope events. can you help?19:37
benjileonardr: sure, looking now19:37
leonardrlook at EventTestCase in test_webservice.py19:38
benjileonardr: ahh, the zope.component subscriber (the one that takes the super simple events from zope.event and dispatches the interface based events) isn't wired up in the test19:50
benjilet me see how you do that19:50
benjiwell, that was at least part of the problem, but it's still not working19:53
leonardrbenji: i know it's working in webservice.txt, you might look at that20:03
benjileonardr: here you go http://pastebin.ubuntu.com/585015/20:08
leonardrbenji, thanks, will look in just a minute20:08
benjialso, the zope.component.eventtesting.getEvents function would probably simplify your tests a bit (remove the need to register your own handler)20:09
lifeless'sender-time Sent at 4:56 AM (UTC). Current time there: 8:10 PM'20:12
derycklater on, everyone.20:13
leonardrjcsackett: very quick: https://code.launchpad.net/~leonardr/lazr.restful/fix-test-failures/+merge/5477320:14
jcsackettleonardr: looking now.20:14
jcsackettleonardr: r=me20:18
leonardrjcsackett: i'm still getting a failure. it's that damn list of things that need to be sanity-checked20:18
leonardrit needs to be cleared out all the time, not just for those tests20:18
jcsackettleonardr: ok. feel free to ping me after uploading those changes.20:19
leonardrjcsackett: ok, i think i know how to do it20:19
jcsackettleonardr: how does one go about exporting something only into devel?20:41
leonardrjcsackett: for an entry or field, you use as_of="devel"20:41
leonardrfor a named operation, you use @operation_for_version("devel")20:41
leonardrthis is once my branch lands20:41
leonardrah, it just landed, so my advice is accurate20:41
jcsackettbut until your branch lands it's not doable?20:42
jcsackettah, excellent.20:42
leonardrbefore my branch landed it was doable but ugly20:43
lifelessok, logs have updated20:48
lifelessI see 2011-03-24 20:07:25 INFO    Notifying robertc@robertcollins.net about bug 741821.20:48
_mup_Bug #741821: "Mute bug mail" in new bug reports is odd <confusing-ui> <story-better-bug-notification> <Launchpad itself:Triaged> < https://launchpad.net/bugs/741821 >20:48
lifelessand the initial one was20:48
lifeless2011-03-24 15:45:43 INFO    Notifying robertc@robertcollins.net about bug 741821.20:48
_mup_Bug #741821: "Mute bug mail" in new bug reports is odd <confusing-ui> <story-better-bug-notification> <Launchpad itself:Triaged> < https://launchpad.net/bugs/741821 >20:48
lifelessthats 4h20 minutes later20:49
lifelessthe bug activity is20:49
lifeless2011-03-24 15:32:53Andrea Corbellinibugadded bug20:49
lifeless2011-03-24 15:56:44Graham Binnstagsconfusing-ui story-better-bug-notification20:49
* jcsackett nods. that works with what we've seen.20:49
lifeless2011-03-24 15:56:48Graham Binnslaunchpad: statusNewTriaged20:49
lifeless2011-03-24 15:56:52Graham Binnslaunchpad: importanceUndecidedHigh20:49
* gmb needs to stop "Graham" and "Binns" from highlighting20:49
lifelessso 20 minutes would have been reasonable20:50
lifelessthe fact that the interval is 4h20 minutes makes me wonder20:50
lifelessis there a 4 hour grace period on actions being reverted or something ?20:50
lifelessgary_poster: ^20:50
gary_posterI was just talking in ops about all this...20:51
gary_posterlifeless, no20:52
lifelessCHANNEL FAIL20:52
bachi sinzui, you have a minute to chat about menus20:53
jkakarThis is a curious position to be in: You are the owner of this team. You are not currently an active member.20:54
jkakarI guess one never gives up ownership of a team...?20:54
lifelessjkakar: ownership can be handed off20:58
jkakarlifeless: Ah, okay.  That should probably happen in this case.20:59
jkakarlifeless: Aha, found it... thanks for the hint. :)20:59
sinzuibac: I will in 2 minutes21:01
sinzuibac: mumble?21:04
bacsinzui: yes please21:04
=== flacoste changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | firefighting: https://wiki.canonical.com/IncidentReports/2011-03-24-LP-bugmail-extremely-slow-or-offline | On call reviewer: jcsackett | https://code.launchpad.net/launchpad-project/+activereviews
lifelessis anyone fixing the db-devel conflict ?21:11
jcsackettlifeless: is there a new one? bigjools or jml killed one awhile ago.21:11
=== matsubara is now known as matsubara-afk
leonardrjcsackett: ok, take another look. i made the sanity check responsible for cleaning up after itself, and i added code to the test framework to wipe any test that imported a module without running the sanity check21:22
jcsackettleonardr: while the while len > 0: pop? wouldn't just reassigning to [] be faster? the contained objects lose their reference all the same so should still be cleaned up normally...21:27
leonardrjcsackett: i would love to reassign to [], but that doesn't work21:28
lifelessjcsackett: leonardr: cleaning a list ?21:28
lifelessdel list[:]21:28
jcsackettleonardr: ooh, that works nicely.21:28
jcsackettlifeless, rather.21:29
lifelessalso spellable as list[:] = []21:29
lifelessbut I'd use del21:29
leonardrchange made21:29
jcsackettleonardr: does that work in your instance?21:29
leonardri just can't change the reference21:30
jcsackettleonardr: ah, right.21:30
jcsackettwell, with the del, all seems well by me. land it. :-)21:30
sinzuiwgrant:  mumble?22:03
wallyworldleonardr: sorry i missed you this morning - got sucked straight into a prod incident. saw your mp comment. thanks. can you tell me how to run the tests?22:17
leonardrwallyworld: bin/test should work22:17
leonardrif you don't have bin/test you need to run 'python bootstrap.py; bin/buildout'22:17
wallyworldleonardr: i there was no bin/test. i started running buildout etc and it went to download all this zope stuff so i killed it cause i was using my 3g modem at the time22:18
wallyworldi will let it finish running the next time22:19
lifelesssinzui: you wanted to talk about the person merge branch I reviewed22:20
leonardrwallyworld: ok, do this too:22:20
leonardrcreate a ~/.buildout/default.cfg22:20
leonardrmine looks like this22:20
leonardreverything it downloads will be stored so it will only download once22:21
sinzuilifeless: Yes please22:21
leonardrnot for every branch22:21
lifelesssinzui: did you want voice or irc or ... ?22:21
wallyworldleonardr: ah, that's what i need. thanks22:21
sinzuilifeless: I am staring skype22:21
wallyworldleonardr: i'm working on the prod incident atm but will make the changes you suggested and let you know. i should be able to make myself available for a standup tomorrow morning to hopefully finalise stuff before you go22:22
leonardrok, great22:23
leonardrtalk to you then22:23
=== jcsackett changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | firefighting: https://wiki.canonical.com/IncidentReports/2011-03-24-LP-bugmail-extremely-slow-or-offline | On call reviewer: - | https://code.launchpad.net/launchpad-project/+activereviews
wgrantOoh, explicit API versioning.22:39
lifelesssinzui: mailed.23:14

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