=== meetingology` is now known as meetingology === Ursinha is now known as Ursinha-afk === Ursinhal is now known as Ursinha === _salem is now known as salem_ [02:40] balloons: Have you slept yet? [02:42] smartboyhw, sleeping now.. good night [02:44] balloons: Good night! :P === salem_ is now known as _salem === _salem is now known as salem_ === salem_ is now known as _salem === _salem is now known as salem_ === salem_ is now known as _salem [06:18] good morning [06:29] pitti, you were right about loop devices, I created 1000 yesterday without problems, I don't know why I had this limit in mind, maybe linux 1.2.x times ... :) [06:35] jibel: with /dev/loop-control? [06:35] jibel: as I tried with mount -o loop, and that indeed doesn't seem to request new loop devices if the existing ones run out [06:37] pitti, with mknod [06:38] ah, that works, too :) [06:39] my hope is that a less ancient util-linux will actually ask loop-control for more by itself === smartboyhw_ is now known as smartboyhw [07:09] didrocks, I moving prepare_fs from the pre-mount hook to a pre-start hook, then will unmount manually in a post-stop hook, see if it fixes the umount issue [07:10] jibel: oh, nice idea! [07:10] I'll be able to confirm if needed [07:10] * didrocks continues on the archive thingy [08:21] didrocks, pitti I fixed the loop devices leak in otto by moving all the mounts to a pre-start script and unmounting in a post-stop [08:22] jibel: oh [08:22] jibel: I have a post-stop as well [08:22] furthermount I use umount.aufs instead of umount to fix a problem with permanent links between the delta and the rootfs after the umountr [08:22] jibel: please push so that I reconcile :p [08:22] didrocks, np, it is just a function [08:22] jibel: ok, I handled the copy of those scripts in a better way as well [08:23] I'm archive, finish testing the restore now ;) [08:23] archiving* [08:26] didrocks, oh, and now lxc-start works without pre-mounting the iso [08:29] jibel: excellent! [08:29] didrocks, r59 [08:30] jibel: ok, reconciling with this :) [08:31] didrocks, I dropped all the checks that verifies a squashfs exists on the iso, and they should be moved to the pre-start scripts [08:32] jibel: hum, maybe my merge will be hard then :p [08:32] * didrocks tries [08:33] jibel: rock! [08:33] jibel: that gets rid of the atexit lazy unmount, too? [08:33] pitti, yes [08:34] really nice [08:34] jibel: so in general, we want to push as many smarts as possible into the pre/post-mount script, right? [08:35] as there they have the potential for getting partial upstream adoption [08:35] jibel: instead of you redoing iso_mount="/run/otto/iso/$(echo $IMAGE | tr '/' '_')" which double the logic, mind if I put that as a parameter? [08:35] pitti, I think so, we should stay as close as possible of native lxc commands, what do you think? [08:35] didrocks, go [08:35] jibel: yes, my feeling as well [08:36] didrocks: ah, I used that tr (in python, formerly) to allow running several instances in parallel and still knowing which image is which [08:37] pitti: sure, that's fine, it's just the logic is in two places now :) otto and the post-stop script [08:38] I'm passing it as a parameter to post-stop [08:39] didrocks: perhaps we should move the mount point into the /var/lib/lxc/ directory? [08:39] and call it /iso [08:39] that's even more clear, and it doesn't need parameterization [08:39] similarly to /squashfs [08:39] why didn't I think of this yesterday.. [08:39] pitti: but we'll be able to mount the same iso in two containers? [08:39] sure [08:39] you can loop-mount an image as many times as you like [08:40] pitti: ok, let me first get back and reconcicle with jibel last changes [08:40] let's see afterward to mount within the container [08:40] the "mountpoint" check was just to avoid mounting an iso over the same mount point twice (it's possible, but might lead to refcount issues0 [08:40] but /var/lib/lxc/container/iso seems more elegant to me [08:40] jibel: argh, you failed me [08:40] jibel: as you mount now in pre-mount… [08:40] I can't get access to the iso infos [08:41] so all my consistency check is not possible :/ [08:41] (like if you want to keep the delta, and so on…) [08:41] ah, good point [08:42] jibel just removed my 2 last hours of work :p [08:46] didrocks, you just need to extract .disk/info from the iso isn't it ? [08:47] didrocks, alternatively, you can mount in otto and check in the pre-start if it's already mounted [08:47] .disk/info, README.diskdefines and the dist directory [08:47] this way you can keep the logic in otto and lxc-start will still work [08:47] jibel: let's do this [08:52] didrocks, I found the problem with autopilot and zeitgeist [08:52] didrocks, .local mode is set to 600 [08:55] didrocks, I think it's because it exists on the ISO, so we need to override it [08:56] jibel: ah… makes sense [08:56] jibel: hum, I wonder if my bzr rebase didn't failed on some on your file [08:56] like I don't see the pre-start.sh [08:57] hum, there is none… [08:57] (on trunk) [08:57] * didrocks digs where mounting the iso is [08:58] didrocks, r60 [08:58] jibel: ok, I was at r59 :p [08:58] You'd see it if I added it [08:58] I need to rerebase [09:01] didrocks, did you add support for proxies and custom uris in sources.list ? [09:05] jibel: not yet [09:06] jibel: I'm fighting to get something working again, it's weird, I'm mounting in otto, but then, it seems pre-start sees it as unmounted and can't mount it again… [09:06] ah, was a transient issue it seems (like it happens sometimes if I don't reboot) [09:07] no, after a manual run, of lxc-start, it's failing [09:07] * didrocks should have pushed first :p [09:41] didrocks, https://bugs.launchpad.net/ubuntu/+source/lxc/+bug/1181136 [09:41] Launchpad bug 1181136 in lxc (Ubuntu) "Empty log file when a container is started with the API" [Undecided,New] === xgoo is now known as Guest58857 [10:21] didrocks, http://paste.ubuntu.com/5673638/ [10:32] pitti: ok, FYI, apparmor was the guilty one from our discussion :) [10:32] pitti: pushed, if you use otto, you should be able to save/restore old run [10:32] (also, check for image consistency is done) [10:39] didrocks: oh, which discussion? [10:40] pitti: was that discussion above about mounting not working ^ [10:40] didrocks: nice! [10:41] oh, that === smartboyhw_ is now known as smartboyhw [11:43] didrocks, finally on the host: sudo ln -s /etc/apparmor.d/usr.bin.lxc-start /etc/apparmor.d/disable/; sudo /etc/init.d/apparmor reload [11:58] jibel: ok, we need to write that in a procedure :) [12:59] phillw: posted and stuck it now [13:03] elfy: Did I miss anything? [13:06] elfy: thanks! [13:32] jibel, you seen these firefox failures in saucy? [13:32] https://jenkins.qa.ubuntu.com/job/saucy-adt-firefox/26/ARCH=i386,label=adt/ [13:32] not sure what is going on with that one [13:33] also, https://jenkins.qa.ubuntu.com/job/saucy-adt-firefox/23/ARCH=i386,label=adt/ looks like it's using the source package from proposed, but the old binaries in the release pocket [13:33] oh, actually [13:33] chrisccoulson, yes I have seen that failure for lot of different packages yesterday, it doesn't really make sense to me. [13:34] actually, the second failure makes sense now. it actually wasn't published in proposed when that ran [13:34] which you already know about :) [13:35] jibel, did you see my last few comments in #ubuntu-desktop? it seems seb uploaded a broken fontconfig which caused a lot of firefox tests to fail [13:35] which is good ;) [13:35] smartboyhw: not that I know of [13:35] chrisccoulson, hm, this is possible only for PPAs, for -proposed we really start the test when the binary hits the archive. That's really weird. [13:36] yeah, that is a bit weird [13:36] chrisccoulson, yeah, it's great when tests find bugs :) [13:36] but the second error is definitely due to running the test scripts from the new source with the old binaries [13:36] as it's showing a file missing that was only introduced in to the latest upload === francisco is now known as Guest99200 [14:11] xnox, are you still about? === _salem is now known as salem_ === francisco is now known as Guest36497 === francisco is now known as Guest85075 === salem_ is now known as _salem