/srv/irclogs.ubuntu.com/2013/07/04/#ubuntu-quality.txt

=== hggdh_ is now known as hggdh
pittiGood morning03:29
airdronehi, can we do ubuntu touh dev on a desktop running raring with no touch on desktop05:05
jibelgood morning06:46
pittibonjour jibel06:51
jibelbonjour pitti, ça va ?07:01
pittijibel: je vais très bien, merci ! et toi ?07:05
* pitti manges beaucoup de fraises07:05
jibelpitti, ça va bien. Tu as trouvé des fraises en automne ? ;)07:06
pittijibel: c'est l'été !07:06
jibelpitti, haha, when I watch outside, the calendar is lying07:07
pittiil ne pleut plus aujourd'hui07:07
pittiseulment hier07:07
jibelit's like 15°c with rain since last Sunday, happy July07:08
pittiwe had a few days of summer since Saturday, I hope the weekend will be better; we'll go to a wedding07:09
pittinice, just installed current saucy grouper build, it works again07:09
jibel\o/07:09
jibelpitti, have you seen rbasak's proposal of an lxc driver for adt ?07:10
jibelhttps://code.launchpad.net/~racb/ubuntu/saucy/autopkgtest/lxc/+merge/17285607:11
pittino, I just saw your G+ message07:11
jibelI'll make a deeper review, but that's something that should go directly to git07:11
pittilxc_container_name = 'adt-virt-lxc-' + ''.join([random.choice(string.ascii_lowercase) for i in range(6)])07:13
pittihm, I don't understand that bit07:13
pittidon't you specify the container name as argument?07:13
pittioh, that might be a backing file07:14
pittimkstemp!07:14
* pitti follows up in MP07:14
jibelpitti, I think that's because if you don't specify a name lxc-start-ephemeral generate a random name for the new container, and the only way to grab it is parsing stdout07:15
DanChapmanGood Morning all :-)08:12
=== ara_ is now known as ara
=== Daviey_ is now known as Daviey
rbasakjibel, pitti: right. lxc-start-ephemeral's stdout is human-formatted. It would be very tricky to parse. And I could check for existing container names to avoid conflicts, but that would race because of the gap between the check and the call to lxc-start-ephemeral anyway. I could file a bug against lxc to make it machine parseable, and add an autopkgtest task so that it's fixed when lxc supports it if you like?10:11
pittirbasak: sounds good; the race with that isn't very bad, I just wonder if one coudl quickly stat /var/lib/lxc/whatever/$name for that10:12
rbasakpitti: re: copyright, it's Canonical copyright, since I wrote it on company time as part of my work for the Canonical server team. But I can bump the copyright date I suppose. It started off as a copy of adt-virt-schroot anyway.10:12
rbasakpitti: so just wrap it in a while exists /var/lib/lxc/...: regenerate loop?10:12
pittirbasak: ok; I don't mind much the copyright, it just stuck out (2007 is a while ago..)10:12
pittiand  having author names is always good for asking someone about stuff10:13
rbasakAck. I'll update the MP.10:13
pittirbasak: some test -d presumably, yes10:13
pittirbasak: but this is all nitpicking, of course10:13
rbasakSure. I'd rather get it right.10:13
rbasakI appreciate the review.10:13
rbasakNo comment on the quality of the rest of the code base :-/10:14
rbasakGlobal variables everywhere, etc. :-(10:14
rbasakpep8? What's pep8? :-/10:14
rbasakpitti: how I about I use tempfile.mktemp? http://docs.python.org/3/library/tempfile.html#tempfile.mktemp10:23
rbasakpitti: it has a massive warning about a race between choosing a name and creation, but we have that anyway10:24
pittirbasak: no, I didn't try it yet or get into details10:24
pittirbasak: pep8> autopkgtest code is an absolute horror; no amount of pep8ification will rectify that10:24
rbasakIndeed10:24
pittirbasak: I believe that's deprecated10:24
rbasakWhat's deprecated? pep8 or autopkgtest?10:24
pittitempfile.mktemp()10:25
pittirbasak: but with a check whether the machine exists, you coudl as well just try autopkgtest001, autopkgtest002 etc. until you find a free one :)10:25
pittirbasak: but anyway, bikeshedding; I think your approach is quite okay, it just looks funny at first sight10:26
rbasakpitti: mktemp() will do that for me, since every machine that exists has an entry in /var/lib/lxc/10:26
rbasakUsing mktemp() would admittedly reduce the size of the race10:27
pittirbasak: again, mktemp() is deprecated since python2.3, it will cease to exist at some point10:27
rbasakI should have at least left a comment explaining the race10:27
rbasakpitti: MP updated. I'm finding a looping random name generation until I find one that's not taken. I've added a comment about a race, and comments at the top adding copyright and authorship.10:58
DanChapmanpitti, when launching rhytmbox with autopilot i'm getting this output http://pastebin.ubuntu.com/5843179/ is this an autopilot bug?11:15
DanChapmanpitti, vis works fine so I can introspect, just the memory leak part concerned me11:16
pittiDanChapman: I don't get that here -- does that only happen when you run it from autopilot?11:16
pittiactually, I do get it, RB just takes ages to start11:17
DanChapmanpitti yeah, its slow starting11:17
pittisome saucy regression11:17
pittiDanChapman: anyway, I get the same message when not being under autopilot, so you can ignore it for now11:17
DanChapmanpitti, ok thanks for that :-)11:18
jibelrbasak, you could also generate a unique name with uuid.uuid4(). performance is close to the random/range approach11:53
rbasakjibel: I have a dislike of UUIDs. They're useful for some things, but a pain when dealing with by hand.11:55
jibelrbasak, since it's for generating a temporary testing env, does it matter?11:56
rbasakHere, the string doesn't need to be universally unique. Just unique in /var/lib/lxc, so it doesn't have to be that long11:56
rbasakjibel: it matters when developing and testing the driver, since I have to examine/clean up containers by hand when doing that. And high development velocity always helps.11:56
jibelrbasak, okay :)11:57
rbasakeg. mktemp doesn't use UUIDs either :-)11:57
pittiindeed that seems to simplify things11:58
pittino need for a polling loop or this fun loop construction11:58
rbasakThe fun loop construction is pretty much exactly what mktemp does. So I think there's precedent here :-)11:59
DanChapmandidn't mktemp get replaced with mkstemp? or was it mkdtemp? well it was one of them two anyway12:04
rbasakmktemp(3) did, yes. mktemp(1) is still there and not deprecated, AFAIK.12:04
smartboyhwBACK12:28
DanChapmanhey smartboyhw12:28
=== _salem is now known as salem_
smartboyhwDanChapman, SergioMeneses hmm I got two mouse pointers using Mir.13:22
DanChapmansmartboyhw, really? thats wild. Ive not seen that13:23
smartboyhwDanChapman, that's on notebook13:23
smartboyhwWhen I connect to a mouse, both pointers move13:24
smartboyhw:O eh?13:24
* smartboyhw will report a bug soon13:24
DanChapmanstrange!13:24
DanChapmansmartboyhw, are they located in the same place or dofferent areas of the screen? just out of curiosity :-)13:25
DanChapman*different13:25
smartboyhwDanChapman, same13:30
smartboyhwActually, no13:30
smartboyhwOne doesn't clik things13:30
smartboyhw:P13:30
smartboyhwDanChapman, the problem seems like happening occasionall14:05
smartboyhw*occasionally14:05
smartboyhwSometimes it appears, sometimes it doesn't14:05
smartboyhwballoons, do you know of that problem?14:10
smartboyhwGuys, seriously, no comments on the pilot system?14:30
DanChapmansmartboyhw, sorry had to pop to shop. Ran out of coffee!!!! Has anyone filed a similar bug to your mouse prob?14:36
smartboyhwDanChapman, dunno14:36
SergioMenesessmartboyhw, jeje nice!15:05
smartboyhwSergioMeneses, which one are you meaning?15:05
smartboyhwchilicuil, have you looked at the email yet?15:06
SergioMenesessmartboyhw, about the mouse! jeje15:10
SergioMenesessmartboyhw, what email?15:10
smartboyhwSergioMeneses, the email about having testcase pillots15:10
smartboyhwSergioMeneses, that;s not good15:10
SergioMenesessmartboyhw, =/15:11
smartboyhwSergioMeneses, did you get the mail?15:12
chilicuilsmartboyhw: not yet, I'm gonna look at it right now15:14
smartboyhwchilicuil, oh great:)15:14
smartboyhwballoons is OK with the idea15:14
chilicuilsmartboyhw: any hint for what I'm looking for? =P15:14
smartboyhwchilicuil, what do you mean?15:14
chilicuilsmartboyhw: the email about having testcase pillots ?15:15
smartboyhwchilicuil, well that's what I plan to have. The pilots should be testcase admins or Canonical QA people15:15
smartboyhwi.e. you will be one:P15:15
SergioMenesessmartboyhw, yes15:17
smartboyhwSergioMeneses, phew:)15:17
chilicuilsmartboyhw: oh, yep I read it yesterday, I think pilots should be another group, I for instance don't have any clue about autopilot =P15:17
smartboyhwchilicuil, well, maybe we make it autopilot pilots and manual pilots.15:17
smartboyhwIDK15:18
smartboyhwIt's just sheer copying from #ubuntu-devel (sorry devs)15:18
SergioMenesessmartboyhw, that would be awesome15:19
SergioMenesesbtw I still have some testcase bugs :'(15:19
smartboyhwlol15:20
chilicuilsmartboyhw: what are necessary two pilots groups?, AFAIK autopilot is all about automating testcases? is not enough autopilot pilots?, why in the first place we need them?, is not working as it's now?15:23
chilicuilwhy*15:23
=== salem_ is now known as _salem
=== _salem is now known as salem_
=== salem_ is now known as _salem
=== _salem is now known as salem_
* SergioMeneses has the same question 15:31
smartboyhwchilicuil, try to ask in the mailing lists:P15:39
smartboyhwchilicuil, SergioMeneses well it's rather like a system for people to standby if new contributors want to work on new things:P15:44
slickymasterelfy: hi, elfy. As you didn't show up here, I've sent you a PM in the forums. Hope it's ok16:27
elfyslickymaster: just got in from work - the first thing I did was drink tea - the second was reply to you :)16:28
slickymasterelfy: the reason was because yesterday you mentioned that, like me, you are also new to bzr, and I found this mini-tutorial about it16:29
elfyyep16:30
slickymasterMost probably you're already aware of it, but just to to be safe I thought of sending the link16:30
slickymasterelfy: http://doc.bazaar.canonical.com/latest/en/mini-tutorial/16:30
slickymasterelfy: I see you already read it :)16:31
elfy:)16:31
slickymasterelfy: reagrding the settings test, I've done a tiny bit of work on it, last night, but now as a read your answer a doubt pop-up in my mind16:35
slickymasterelfy: I was not supposed to push the changes I've made?16:35
elfynot yet :)16:35
slickymasterwas I not, sorry16:35
elfyslickymaster: you should see a window called elfy in your irc client16:35
slickymasterelfy: I'm not on XChat, I'm on my browser, still at work16:37
elfyyea - have a look across the top of it16:38
elfymaybe a tab or something16:38
slickymasterelfy: :) you're right. there she is16:38
DanChapmanHas anyone got the time to help me setup an autopkgtest for evince using my autopilot test?17:07
DanChapmanpitti, hey what does this mean? /home/dan/auto-package-testing/bin/../etc/config: distro-info: not found17:38
pittiDanChapman: a-p-t uses that to determine the default release if you don't specify one17:39
pittiDanChapman: install the distro-info package for it, or specify one with -r17:39
DanChapmanpitti, great thanks.17:43
=== yofel_ is now known as yofel
phillwHi, I'm just having a 'blonde' moment... what is the name of the system that speeds up boot time which gets triggered each time something like a new kernel is installed? I'm not 100% sure if it is upstart.20:11
Patrickdkbootchart?20:21
Patrickdkoh, you want to speed up20:22
Patrickdkwell, that is not always used, but ureadahead20:22
phillwPatrickdk: thanks, it is ureadahead ! ... I just assume it to be there now (I was there when it 1st was introduced) and had forgotten the little critter's name :)20:27
Patrickdkhmm, back in 9.04 I think20:28
=== netcurli_ is now known as netcurli

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