/srv/irclogs.ubuntu.com/2010/08/03/#launchpad-dev.txt

jelmerthumper: Will do. Thanks.00:01
mwhudsonjelmer: man, that mapdbs fix had me seriously head-scratching :-)00:03
mwhudsoni hope it's not necessary any more00:03
jelmermwhudson: heh, I bet00:04
jelmermwhudson: it's still used actually00:04
jelmerthough we'll hopefully be able to use bzr index files with git pack files for the cache hopefully00:04
mwhudsonok, maybe more of a fix than deleting the tearDown is required00:04
jelmer(that should also be the point at which we can have a "bzr serve --git" that works with decent performance)00:05
jelmerthat reminds me.. any news on bzr:// support, or is it all recipes these days in the code team ? :-)00:06
thumperjelmer: all recipes00:06
lifelessjelmer: bzr:// support - for anonymous access to branches ?00:07
jelmerlifeless: yeah00:07
lifelessjelmer: I'd rather we just did bzr on http00:07
lifelessjelmer: less setup, better support in firewalled situations00:08
jelmerlifeless: either would be awesome00:11
lifelesswgrant: btw00:16
lifelesswgrant: private PPA log file tailing. That permits an attack on launchpad.net via content insertion, no ?00:16
wgrantlifeless: Howso? logtail should be escaped by the template.00:16
lifelessah00:17
lifelessI was thinking put js in it and let the streamorview thing do its sniffing00:17
wgrantAnd the downloable build log should be text/plain.00:17
wgrantAh, no, there's no JS there.00:17
wgrantJust boring old refreshing.00:17
lifelessand the build log is the full thing, right ?00:17
wgrantIt is.00:17
wgrantThere's probably a vulnerability there somewhere.00:18
lifelesswgrant: you were saying that setting cd:attachment will break streaming logs00:21
lifelesswgrant: but if its being escaped in the appserver, it won't have any immediate impact, will it ?00:21
wgrantlifeless: Not streaming logs. Viewing of logs in the browser.00:21
wgrantAt the moment I can click on a build log link and not have to download it.00:21
wgrantThis is unrelated to logtail.00:21
lifelessok00:21
lifelessanyhow, I've filed a bug, if you have ideas about how to do things, please dive in.00:23
wgrantI think the existing build log streaming might be vulnerable in IE, since IE likes to sniff content rather than obeying content types.00:23
lifelessheh00:26
wgrantHow much SQL time is there on OOPS-1675ED4744?00:40
lifelessis that oops code correct ?00:42
mwhudsonwgrant: did it just get created?00:42
wgrantMaybe it just missed the sync.00:42
lifelessSQL time: 4761 ms00:44
lifelessNon-sql time: 13888 ms00:44
lifelessTotal time: 18649 ms00:44
lifelessStatement Count: 5900:44
wgrantRight, that's about what I thought.00:45
wgrantWTF is the non-SQL time, I wonder.00:45
thumperwgrant: python execution time,00:46
thumperstorm object creation time...00:46
lifelesswe can get a profile of it for you00:46
lifelessif you ask a losa00:46
wgrantI know, but that's an awfully large amount of time considering the few queries.00:46
lifelesspasting you some stuff00:47
lifelesslosa ping01:49
spmyo01:50
lifelessI'd like to get a profile from staging per favour01:50
spmsure01:50
lifelessso, in theory I wait, you say 'go ahead' then I say 'done'.01:51
lifelessin practice, I'm guessing this is the first time ever and you have no idea whats about to happen :P01:51
spmthat'd be spot on01:51
lifelessand the difference between theory and practice.. :)01:51
lifelessso in the config file for staging01:51
lifelesson staging01:51
lifelessthere is a profiling section01:51
spmI can infer what you want; but not quite sure exactly what you want to achieve, and the how of.01:51
lifelesswith a key profile_requests (IIRC)01:51
lifelessyou need to turn that on and bounce the app server01:52
spmhuh. I didn't know that!01:52
lifelesstada01:52
spmthis is the app server? (to confirm) vs code*01:52
lifelesswe're going to make this less manual01:52
lifelessappserver01:52
lifelesshttps://api.staging.launchpad.net/beta/~ubuntu-dev/participants is the URL I want to profile01:52
lifelessspm: for context, see maris's email to launchpad-dev at the end of the epic01:53
lifelessspm: so, 2 weeks backish. it describes the process on a developer machine.01:53
spmhrm. we have no 'profiling section' or 'profile_requests' is the existing configs; can you give me a quick dump of the lines to add?01:54
spms/is/in/01:54
lifeless[profiling]01:54
lifelessprofile_requests: True01:55
spmhmm. complex.01:55
lifeless(see configs/schema-lazr.conf for missing things like this)01:55
lifelessyou can also set profile_dir:01:55
lifelessif you want them to go to a /var dir or some such - the default is '.'01:55
spmwhat!?!? srsly!?!? you want me to rtfm!?!?! I'm a *sysadmin*!!!01:55
lifelessoh, and lastly, we should also set01:56
lifeless*in that section*01:56
lifelesserror_dir: and oops_prefix:01:56
lifelessbecause it will generate an oops to go with, so that we get the DB trace etc01:56
spmahh kk01:56
lifelesshttps://pastebin.canonical.com/35303/01:57
lifelessto bring it all together.01:57
spmnot such a big deal here'n'now; but might be worth getting this formally landed into the prod configs - staging section only. ??01:57
lifelessyeah01:57
lifelessif you tell me the stuff to put in it :)01:58
spmstaging-lazr.conf ?01:58
spmoh. right. lala eparse01:58
lifelessI can wait if you need a coffee :)01:59
spm[profiling]01:59
spmprofile_requests: True01:59
spmerror_dir: /srv/staging.launchpad.net/staging-logs/profiling01:59
spmprofile_dir: /srv/staging.launchpad.net/staging-logs/profiling01:59
spmoops_prefix: PROFILE01:59
spmdo I what...01:59
spmlosa_needs_coffee: ZOMGTRUE01:59
spmrestarting the app server...02:00
spmKah Boom.02:00
spmhttps://pastebin.canonical.com/35304/02:00
lifelessHAH02:00
lifelessdelete the error_dir key02:01
spmkk02:01
lifelessoh02:01
spmsame error.02:01
lifelessdelete the oops prefix too02:01
lifelessthe schema had a confusing bit, it got me good02:01
spm:-)02:01
spmthat's happy02:02
spmsee that's what rtfm does for you. confuses. much better to just make s**t up02:02
lifelessis it up and running ?02:02
spmshouldbe... stopped doing stuff; giove it a few secs to "settle"02:03
spmand responding to nagios checks; go for it02:03
lifelessok, I just got a request through.. turn it off again02:03
lifelessthen we'll grab the profile file and oops and I can nod off and examine it02:04
spmhaha. what a guess. that directory - completely unlooked for; already existed.02:04
spm[profiling]02:04
spmprofile_requests: False02:04
spmprofile_dir: /srv/staging.launchpad.net/staging-logs/profiling02:04
spmsyncing logs...02:04
lifelessbzr+ssh://bazaar.launchpad.net/~lifeless/lp-production-configs/timeouts has it in it now, but don't merge it - see special rollout notes02:05
spmis restarted02:05
lifelessso in that directory02:07
lifelessthere should be a bunch of files02:07
lifelessand02:07
lifelesspossibly some oops's there, or otherwise they will be in the regular oops dir02:07
spmsodium:/srv/launchpad.net-logs/staging/asuka/profiling/02:08
lifelessspm: you can delete all the 2008 files ;)02:09
lifelesshmm, but I can't see the participation one I would expect. time to grep em al I think02:10
spmI was thinking that :-)02:10
lifelesswgrant: https://bugs.edge.launchpad.net/ubuntu/+bug/602360 is a bug you might find interesting - soyuz query02:12
_mup_Bug #602360: timeout on source package bug filing page <timeout> <Launchpad Bugs:Triaged> <Ubuntu:Invalid> <https://launchpad.net/bugs/602360>02:12
lifelessspm: ah, its one of ScopedCollection:CollectionResource02:12
wgrantlifeless: That plan makes me cringe.02:13
wgrantIt can't possibly need all of those tables.02:13
lifelesscurtis said, in a similar bug02:14
lifelesshttps://bugs.edge.launchpad.net/launchpad-answers/+bug/60803702:14
_mup_Bug #608037: timeouts in Question:+edit <oops> <timeout> <Launchpad Answers:Triaged> <https://launchpad.net/bugs/608037>02:14
lifelessspm: actually, can you please nuke the 2008* files ;)02:21
spmsure; done on asuka, will rm from sodium02:21
spmgone02:22
lifelessthanks02:22
lifelessspm: also, can you look for oopses +- 1 minute from when I said 'done'02:24
lifelessspm: and grep them for the url .../participation02:24
lifelessthere should be 2 I think02:24
lifelessI just need the OOPS ids.02:24
spm /participants ?02:26
lifelessyes02:26
spm03796.S56 03826.S95 are the files. go up one dir on sodium from the logs and into 2010-08-0302:27
* spm regrets cat'ing those out...02:27
lifeless:P02:28
spmOOPS-1676S95 OOPS-1676S5602:28
lifelessthanks!02:28
lifelesssweet - https://lp-oops.canonical.com/oops.py/?oopsid=1676S9502:29
lifeless1.8 seconds sql, 4.6 seconds python02:29
lifeless430 sql requests02:29
lifelessalso02:31
lifelesshaha:  WHERE Person.name = %s AND Person.merged IS NULL ORDER BY person_sort_key(Person.displayname, Person.name)02:32
lifelessthats a little odd02:32
lifelessexact match, oh but lets order it please.02:32
spm*430* SQL requests! eek!02:33
lifelessiz potato programming02:33
lifelessnow, I need to find some folk with detailed knowledge of the storm glue02:34
spmas in like Mr Potatohead? just more bits stuck together?02:34
lifeless2010-08-03_02:03:40.261-ScopedCollection:CollectionResource-OOPS-1676S95-Dummy-2.prof is the profile file if you're interested02:35
spmI shall have a look02:36
thumperlifeless: what's up?02:38
* thumper knows glue well02:38
lifelessthumper: hiya02:42
lifelesshave a look at  https://lp-oops.canonical.com/oops.py/?oopsid=1676S9502:42
lifelessits a slow API call02:42
thumperlifeless: we could go to voice if it helps02:42
lifelessthat might be a good bootstrap exercise02:43
lifelesssodium:/srv/launchpad.net-logs/staging/asuka/profiling/2010-08-03_02:03:40.261-ScopedCollection:CollectionResource-OOPS-1676S95-Dummy-2.prof is the kcachegrind profile02:52
lifelessthumper: https://api.edge.launchpad.net/beta/~ubuntu-dev/participants - open it in your browser03:02
lifelessthumper: http://www.amazon.com/Patterns-Enterprise-Application-Architecture-Martin/dp/0321127420/ref=sr_1_1?ie=UTF8&s=books&qid=1280802410&sr=8-103:27
lifelessbbiab, chores to run03:51
=== Ursinha-afk is now known as Ursinha
=== Ursinha is now known as Ursinha-afk
pooliehappy performance day!04:50
* thumper done for nwo06:47
thumpernow06:47
=== almaisan-away is now known as al-maisan
lifelessthumper: so, another question - can you point me at an example of testing the API / making a browser request - outside of a doctest ?07:27
bebo_mzcan any one help me in this W: GPG error: http://ppa.launchpad.net lucid Release: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY 449F83829320B41C07:56
lifelessyou might have better luck in #launchpad08:09
adeuringgood morning08:31
lifelessgood morning08:41
didrocksjml: hey, how are you?09:02
adeuringlifeless: could you please explain a bit what the security issue is in my MP for lp:~adeuring/launchpad/bug-39674-update-retricted-flag-of-private-bugattachments ?09:08
mrevellMorning09:15
jkakarIf someone has time to review a trivial Storm branch, please check this out, it adds an is_empty method to SQLObjectResultSet: https://code.edge.launchpad.net/~jkakar/storm/sqlobject-is-empty/+merge/3156509:19
adeuringwhat should we for a buildbot failure "xvfb-run: error: Xvfb failed to start"?09:19
lifelessadeuring: hi09:31
adeuringhi lifeless09:31
lifelessadeuring: its the whole arc of work - I filed a dedicated bug about it09:31
adeuringlifeless: so, do you mean the ProxiedLFA stuff?09:32
lifelessyes09:32
lifelessbug attachments are user controlled content09:32
lifelessif they are served in a domain, they can run scripts to access anything else in the domain - other bugs, private lists, etc, and submit the content to other servers09:33
adeuringlifeless: right, but I don#t see the issue yet... anyway, can you tell me the bug number?09:33
lifelessits why the librarian is on launchpadlibrarian.net rather than part of the appservers09:33
lifeless61277909:34
lifelessadeuring: the issue is, that anyone can file a private bug, with an attach script, which will (for instance) steal the users oauth cookie and forward it to a hostile site09:35
adeuringlifeless: yes, but that applies to attachments of bugs too...09:36
lifelessno it doesn't09:36
lifelessbecause they are on a different domain09:36
adeuringah, right, forgot that09:36
bigjoolslifeless: see my comment on your librarian MP09:36
lifelessbigjools: https://code.edge.launchpad.net/~lifeless/launchpad/private-librarian/+merge/31020 ?09:38
lifelessbigjools: or the oops one ?09:38
lifelessI can't see a comment on the private-librarian MP09:38
bigjoolslifeless: 3150809:38
lifelessfull url please ?09:39
lifelessbigjools: I've already put it on the wiki page a week ago ;)09:40
bigjoolslifeless: teh cool09:43
* thumper fixes the merge conflict09:45
lifeless\o/ https://bugs.edge.launchpad.net/ubuntu/+filebug is live with my tweaks09:46
adeuringlifeless: ok, I get it now. So, any estimate when you can land your librarian branch?09:47
lifelessadeuring: well my branch doesn't really help09:48
lifelessit has the same concern09:48
lifelessadeuring: I think what we can sensibly do for now is set cd:attachment on untrusted restricted objects.09:48
lifelessadeuring: this may require a new field on LFA, *or* a new view for how they are traversed.09:49
adeuringlifeless: right, that's the other option. I just wondered if this is worth the effort since your work would fix the problem as well09:49
* thumper fixes stable -> db-devel conflicts09:49
lifelessadeuring: so my work will move the private content off the launchpad main domain09:50
lifelessadeuring: but it will still let private content attack other private content09:50
adeuringlifeless: right09:50
lifelessadeuring: so I think we should *also* take steps to make it less risky09:50
adeuringlifeless: makes sense. A variant of ProxiedLFA looks reasonable09:51
stublifeless: Why doesn't your branch help? I thought with your work, restricted files would be served on the public port on its seperate domain.09:53
stuboic09:53
lifelessstub: it helps, but its not the full story09:53
stubBut there is nothing to attack, as no cookies are set.09:53
lifelessstub: sadly there is09:53
stubstealing URL history or something?09:53
stubI guess we could use a wildcard and serve files from https://abctoken.launchpadlibrarian.net/123/foo.txt instead of https://launchpadlibrarian.net/123/foo.txt?token=abctoken09:55
lifelessstub: window A can access the DOM of window B09:55
lifelessthus the urls (and tokens) for other private content, and then submit that url encoded to $attckingstite09:56
lifelessstub: yes, wildcard dns is what I'm thinking will work.09:56
lifeless(Window A and window B have to be on the same domain for the above attack to work)09:56
lifelessstub: I was thinking https://contenthash.launchpadlibrarian.net/foo.txt?token=abctoken or something like that09:57
stubIf you use the LFA.id, https://f123.launchpadlibrarian.net/foo.txt instead of the existing https://launchpadlibrarian.net/123/foo.txt, and allows us to rewrite existing URLs.09:59
lifelessstub: its rather late for me, but I'm totally keen on whatever permutation will work best for us09:59
lifelessstub: so if you'd like to shove that into the MP discussion on https://code.edge.launchpad.net/~lifeless/launchpad/private-librarian/+merge/31020 with any hints about what glue I'll need to piece it together, that would be cool.10:00
stubI think you want spiv or jml or someone who has worked with twisted http for the hints. I can add this discussion though.10:01
lifelessthe twisted server internals I am fine on10:01
lifelessits more things like the dns setup, apache/squid changes etc - just a bookmark of the gotchas10:01
lifelessalso, yay, bug filing searches seem solidly < 10 seconds to me now, on edge.10:03
bigjoolslifeless and stub: can you bless the db patch and give me a number please: https://code.edge.launchpad.net/~julian-edwards/launchpad/buildd-failure-counting/+merge/3155610:03
lifelesswhat TZ is deryck in ?10:05
lifelessah, us.10:10
jkakarbigjools: Are you pumped?!  Ready to have a great time!!!  https://code.edge.launchpad.net/~jkakar/storm/sqlobject-is-empty/+merge/3156510:14
bigjoolsjkakar: \o/10:20
bigjoolsjkakar: I think we should also fix the bug about removing the ordering on .any()10:21
bigjoolslifeless is on a speed war10:21
lifeless\o/ phwoar10:22
jkakarbigjools: It's fixed in another branch and al-maisan had agreed to review it!10:25
jkakarOnce that's done it'll land, hopefully today.10:25
* al-maisan just approved that branch10:25
bigjools\o/10:27
bigjoolsjkakar: when will it be released?10:27
lifelessmpt: hai10:29
mptoh hai10:29
jkakarbigjools: As a package, you mean?10:29
jkakaral-maisan: Thanks!10:29
lifelessmpt: I'm all excited - bugs.edge.launchpad.net/ubuntu/+filebug10:29
lifelessmpt: try it. Marvel at the speed.10:29
lifeless[actually, it still has lots of room to improve, but I'm easily pleased]10:29
bigjoolsjkakar: we use eggs for buildout10:30
jkakarbigjools: So you need a tarball, right?10:31
bigjoolsI guess so, I'm not that familiar with that stuff10:31
jkakarbigjools: Or I guess an egg, but we don't create those when we make releases as far as I know.10:31
jkakarbigjools: What would it take to start using the python-storm package?10:31
mptlifeless, "lifeless is excited at the speed" returns suggestions in <2 seconds. "Ubuntu sucks" returns suggestions in ~13 seconds.10:31
mptPretty sure that's an improvement. :-)10:31
jkakarAlso, fwiw, we use buildout and have it checkout lp:storm which works quite well, since we're always on trunk.10:32
lifelessmpt: yes, sadly 2 term searches will still be slow until we get a new search layer.10:32
bigjoolsjkakar: no idea!  I don't know the pros/cons of that versus what we do, someone from Foundations would know though.10:32
bigjoolsjkakar: interesting, I'll get gary to talk to you10:32
jkakarbigjools: Get him to talk to sidnei. :)10:33
lifelessmpt: as in, 2-term searches will be the slowest, if you graph speed vs term count10:33
bigjoolsjkakar: o710:33
bigjoolswgrant: howdy.  I don't suppose you've QA verified any of your fixes this cycle?11:40
lifelessstub: so, is a 'dba' bug tag ok with you?11:41
stubSure, but I need to work out how to get GMail to flag them. I seem to require push methodologies :)11:42
wgrantbigjools: My stuff is just about impossible for me to QA.11:42
bigjoolsI figured :(11:42
wgrantSorry.11:43
lifelessstub: ok, well I'll start marking up :)11:44
lifelessstub: would an rss feed work ?11:45
stubI always forget to check my RSS feeds :)11:46
lifelesshah, ok11:46
lifeless[me too]11:46
lifelessderyck: oh hai12:01
derycklifeless, hey, man.12:01
wgrantSo, I have a whole lot branches that need to be reviewed, just about no reviewers in my timezone, there's a lack of European OCRs, and American OCRs tend to knock them off the queue if you're not around.12:06
jelmerwgrant: I'm OCR tomorrow, should have some time to review them then.12:09
wgrantjelmer: Thanks.12:09
lifelesswgrant: also, I can review them all tomorrow.12:10
lifelesswgrant: I wasn't today because performance day!12:10
jelmerhmm, actually12:15
jelmerjtv: are you OCR as well tomorrow? Perhaps I should do Thursday this week as noodles is on holidays.12:16
jtvjelmer: yes, I'm on tomorrow12:16
jtvjelmer: I'll finish around mid-day for you though12:16
jtvWho removed installKarmaRecorder from TestCase?12:17
jtvOh crap... the entire KarmaRecorder is gone.12:21
jmljtv, https://code.launchpad.net/~henninge/launchpad/bug-612583-test-errors/+merge/31546 is the only mention of KarmaRecorder in my mailbox12:22
jtvjml: we must not have been using it for a while.12:22
jtvjml: hmm... that mp says it was introduced in the recife branch.  I don't remember that; I thought it was older.  Thanks.12:23
jmljtv, np12:23
wgrantlifeless: Thanks.12:30
wgrantThey're mostly small.12:30
=== mrevell is now known as mrevell-lunch
* lifeless -> sleep12:36
=== mrevell-lunch is now known as mrevell
=== matsubara-afk is now known as matsubara
=== Ursinha-afk is now known as Ursinha
didrocksjml: hey, how are you?13:49
jmldidrocks, hello, I'm well thanks.13:58
didrocksjml: I didn't have the time to talk to you before because of GUADEC last week and didn't see you at the sprint apart from the first day, did you have some time to have a look at the gpg/ssh/ppa api?13:59
jmldidrocks, no, sorry. it's on my list though. I think the next step is creating a new permission level14:00
jmlwhich maybe I can convince someone else to do14:00
didrocksjml: let's hope so, no hurry for now. I'll ask for a feature freeze exception on the Quickly side. Thanks :)14:01
=== salgado is now known as salgado-lunch
=== al-maisan is now known as almaisan-away
jelmermars: Hi15:56
jelmermars: I've build a new EC2 image and would like to bless that as the new official image. As far as I can tell I've done everything mentioned on the wiki, but "ec2 test" locally is still using the old version.15:57
=== deryck is now known as deryck[lunch]
jelmermars: I've added myself to the list of valid AMI owners, and made my image public.15:58
jelmermars: Is there anything I might have forgotten?15:59
bigjoolsjml: thanks for the extra twisted tip BTW, that's sneaky16:02
jmlbigjools: np. as I said, perfectly valid in tests and in some extremely rare real code cases.16:03
marsjelmer, gary_poster might know if the image is ready to go16:05
marsI haven't actually built one myself yet, but gary_poster and mwhudson have16:05
marsjelmer, ack, gary_poster is not around right now, but he should be back before your EOD I would suspect16:06
gary_posterI am, I am :-)16:06
jelmermars: Ok, thanks!16:06
jelmerah! Hi Gary16:06
gary_posterjelmer, hi. public image and valid AMI owner: sounds right to me16:06
gary_posterdo you want me to see if I can see your image on the management console or something?16:07
jelmergary_poster: It'd be great if you can check, although it is marked as public here.16:08
gary_posterk16:08
jelmerami-217f9448 is the ID16:08
jelmergary_poster: Sorry, unping. I've found the issue.16:10
gary_posterjelmer: cool.  fwiw, I couldn't find it16:10
gary_posterI mean, the ami16:10
jelmerthe branch I was submitting didn't have myself added to the list of valid AMI owner IDs yet.16:11
gary_posterah :-)16:11
jelmergary_poster: sorry for the trouble, thanks for thinking along :-)16:12
gary_posternp :-)16:12
james_wcould someone tell me what a reasonable check for calling removeSecurityProxy is?16:15
gary_posterjames_w: "reasonable check"?16:16
james_wexactly16:16
james_wUnreasonableRemoveSecurityProxyWarning: Called removeSecurityProxy(<PackageUpload at 0xda8906c>) without a check if this is reasonable.16:16
gary_postersorry, was intending to convey the idea that I didn't understand that part of the question16:16
gary_posteroh16:16
james_wneither do I16:16
gary_posterabel knows about that16:17
gary_posterhe isn't around16:17
gary_posterderyck[lunch] knows about that...16:17
gary_posterof course, I'd grep for the error and see if the code path explained the intent16:17
gary_posterI'm supposed to be doing several other things so I'll leave that to you, I'm afraid16:18
james_wit appears to be rather than fixing the fallout caused by returning proxied objects16:18
* james_w replaces it with removeSecurityProxy, I can't see any reason why that would be wrong16:21
jmljames_w, hey16:24
james_whi jml16:24
jmljames_w, what that error means is that adeuring changed the code to use rSP but isn't sure whether it was appropriate16:24
jmljames_w, so if it is appropriate to use rSP (which IMO basically it is in all cases in a test except when you're testing security) then change the call to removeSecurityProxy16:25
james_wright, this is in SoyuzTestPublisher, so if there is application code using it then we have bigger issues16:25
jmlthe alternative is to log in as an appropriate user16:25
adeuringjml, james_w: yes's what I meant: I simply added these calls mecahnically ust to ensure that tests do not fail due to added security proxies16:26
jmladeuring, thanks.16:27
jmlmaybe there's a better way of phrasing the warning...16:27
=== beuno is now known as beuno-lunch
=== matsubara is now known as matsubara-lunch
rockstarbigjools, is there an open bug about getting new build hardware?  I'd love to mark this bug as a dupe: https://bugs.edge.launchpad.net/launchpad-code/+bug/61217717:51
_mup_Bug #612177: Built daily? Really? <recipe> <Launchpad Bazaar Integration:Triaged> <https://launchpad.net/bugs/612177>17:51
bigjoolsrockstar: not on soyuz, no.17:52
rockstarbigjools, should there be?17:55
bigjoolsrockstar: it's not really a bug17:55
bigjoolsit's an RT17:55
bigjoolshowever, we are working on some performance improvements but given the number of jobs versus number of builders, we're never going to win17:56
bigjoolsrockstar: search for the "buildfarm" and "buildd-manager" tags and see if any fit for a dupe17:56
=== salgado-lunch is now known as salgado
=== beuno-lunch is now known as beuno
=== matsubara-lunch is now known as matsubara
* rockstar waits forever for dogfood...18:55
jmlabentley, if I wanted to fix bug 608114, would I need to change anything other than Diff.fromTrees in lp.code.model.diff?19:24
_mup_Bug #608114: Show modified class/method names in the diff of the merge proposals <code-review> <Launchpad Bazaar Integration:Triaged> <https://launchpad.net/bugs/608114>19:24
abentleyjml, that's a request to stop using bzr to perform diffs and start using /usr/bin/diff.19:25
jmlabentley, does bzr switch to using external diff only when external diff options are provided?19:26
abentleyjml, yes.19:26
jmlabentley, is there any reason we shouldn't be using external diff?19:27
abentleyjml, bzr produces better diffs.19:27
jmlabentley, I see.19:28
abentleyjml, and bzr produces different diffs.  So people will see differences between the bzr output they produces locally and the diffs that Launchpad generates.19:29
jmlabentley, I don't think last one is necessarily bad.19:29
jmlabentley, producing worse diffs is enough to make me not want to go for the easy fix for the bug, though.19:30
abentleyjml, variation breeds confusion.19:30
jmlamong other things19:31
jmlI wish more things were easier.19:31
benjiwords to live by19:31
jmlabentley, ok, thanks. I've commented on the bug, including a patch that fixes it in the bad way19:36
jmlabentley, I just saw https://bugs.edge.launchpad.net/testrepository/+bug/611706 -- do you think something like the following would be readable?19:52
_mup_Bug #611706: Confusing output <Testrepository:New> <https://launchpad.net/bugs/611706>19:52
jmlid=0, tests=21, failures=4 skips=119:52
jmlerr... imagine there's a comma after the '4'19:52
abentleyjml, yes, I think that would help me read it properly.19:53
jmlabentley, thanks. I'll submit a patch.19:56
jmljames_w, are you still being bitten bug 597060?20:43
_mup_Bug #597060: testr run should support limiting ids <testr-run> <Testrepository:New> <https://launchpad.net/bugs/597060>20:43
james_wjml: yes, in that I've learnt not to do it and so don't use testr as much20:45
jmlhmm :(20:45
jmljames_w: maybe I don't understand it then.20:46
james_wRob wants to solve it by working on the bug called something like "$IDOPTION needs some thought"20:46
james_wjml: I just want to be able to do the parallel of "test run lp.code". With $IDLIST this runs the everything, and then the code tests the second time.20:46
james_w(not that it does that on LP, as LP uses IDOPTION, so I don't know what happens)20:47
jmlahh ok20:49
jmljames_w, the source of my confusion is that I was unaware of how flexible .testr.conf files really are20:49
marsjml, quick silly question: how do I run tribunal from a fresh checkout?  The README doesn't say how.21:02
jmlmars, ./bin/tribunal21:02
jmlmars, ./bin/tribunal-subunit, actually21:03
marshmm, no bin/ in the lp:tribunal checkout21:03
jmlmars, my local copy appears to be out of date21:03
marsusefully out of date :)21:04
jmlmars, ./tribunal-subunit, then21:04
marsjml, ok, thanks.21:04
jml(as an aside, Launchpad is a bit weird in that it's the only project I know that has binaries in bin/ that aren't intended for distribution)21:05
marsI run OpenKomodo mainline, and they have a project-specific bin/ for developers21:06
marsThe project is built from Python and JS wrappers over Mozilla XUL, so it's a fairly large project, too21:07
marsok, so "~/src/tribunal/tribunal-subunit ." works21:10
marsit picks up the .testrepository dir automagically21:10
lifelessmoin21:47
jmllifeless, hi21:47
lifelesshi21:47
jmllifeless, I'd love to talk with you about all the stuff I just did to testrepository, but I also want to escape my computer.21:48
lifelesswell21:48
lifelessyou could ring me from your noncomputer21:48
jmlnah, same thing. thanks though.21:49
jmllifeless, none of it's urgent, so I think I'll just sign off.21:49
jmlhave a good day21:49
* jml off21:49
lifelessok, ciao21:49
=== lifeless changed the topic of #launchpad-dev to: Launchpad Development Channel | Week 3 of 10.08 | PQM will be closing 22:00 UTC Friday | firefighting: - | https://dev.launchpad.net/ | Get the code: https://dev.launchpad.net/Getting | On-call review in irc://irc.freenode.net/#launchpad-reviews | Use http://paste.ubuntu.com/ for pastes
james_wI've just found two objects that don't correctly implement their interface, I wonder how many more there are21:55
lifelessN21:59
lifeless:P21:59
wgrantjames_w: Lots of classes have verifyObject called on them.21:59
wgrantBut I guess some do not.21:59
wgrantmars: Thanks for the reviews.21:59
wgrantmars: yes, lots of those doctests should be unit tests instead, but that basically requires a rewrite of a large portion of the Soyuz test suite.22:00
thumpermorning22:00
wgrantThat's going to take a while.22:00
james_wa "fun" task is going to be signing an upload during the tests so that an ubuntu series isn't hardcoded in the test data22:02
james_wthough perhaps the best way to do that is to wait until that day that we remove the sampledata and use the factory to create a series with a matching name22:02
wgrantThat was going to be my suggestion.22:02
marswgrant, yep, I know.  Julian and I had a discussion about that very problem earlier today.22:03
wgrantmars: I dissect them where I can, but there's a lot left...22:03
thumperdb-devel branch failing with testMemcachedWorking22:03
thumperanyone know of recent changes in that area?22:03
marsthumper, yep22:03
marslooking22:04
thumperta22:04
marsHurray for test suite instability!22:04
marsthumper, losas to fix both, I'll tell them.22:05
thumpermars: what do you mean?22:06
marsthumper, both builders are dead on test infrastructure failures22:06
thumpermars: jelmer is putting in an RT for the devel failure, old ec2 image I'm told22:07
thumpermars: not sure about the db-devel failure22:07
marsthumper, ah, that might do it22:08
thumpermars: jelmer was going to roll-back the last commit (that needs the new package)22:08
marsyep22:09
marsmight not be that simple22:09
marsTom updated that machine to Lucid, which means it's off EC2 now.  If that is true, then a new image won't help22:09
=== matsubara is now known as matsubara-afk
=== salgado is now known as salgado-afk
=== Ursinha is now known as Ursinha-brb
marslifeless, ping, did the --load-list option for the zope testrunner get landed or released upstream?23:37
mwhudson morning23:50
=== Ursinha-brb is now known as Ursinha-afk
lifelessmars: I don't know23:57
marsok23:58

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