[00:01] <ago> evand any other bug you want to go through tonight?
[08:23] <tjaalton> could I request a new d-i against -4 kernel?
[09:18] <cjwatson> tjaalton: sure, on its way
[09:18] <CIA-22> debian-installer: cjwatson * r869 ubuntu/ (10 files in 4 dirs): * Move to 2.6.24-4 kernels.
[09:18] <tjaalton> cjwatson: thanks!
[09:20] <CIA-22> debian-installer: cjwatson * r870 ubuntu/debian/changelog: releasing version 20070308ubuntu26
[14:54] <xivulon> hi evand
[14:54] <xivulon> any q over remaining patches?
[14:54] <evand> xivulon: I'll let you know, but none currently.
[14:58] <xivulon> what's the status of bug #173659 and bug #151579?
[14:58] <ubotu> Launchpad bug 173659 in wubi "Add /host/boot to fstab" [Medium,Fix committed] https://launchpad.net/bugs/173659
[14:58] <ubotu> Launchpad bug 151579 in wubi "umountfs must check whether a mountpoint contains a loopmounted root file" [High,Fix committed] https://launchpad.net/bugs/151579
[15:01] <evand> I'll take care of the former next.  As for the latter, cjwatson, does http://paste.ubuntu-nl.org/52195/ look ok to you or would you prefer the solution you had originally suggested?
[15:01] <xivulon> evand can you also look into bug #177868? Without the new hooks is very difficult for me to do releases (have to support 2 hooks)
[15:01] <ubotu> Launchpad bug 177868 in wubi "When loopfiles are used mkfs has to target the file and not the containing device" [Low,Fix committed] https://launchpad.net/bugs/177868
[15:02] <evand> Are you sure that's the right bug?
[15:02] <xivulon> That mostly involves merging https://code.launchpad.net/~ubuntu-installer/lupin/hardy
[15:02] <xivulon> ehm nope
[15:02] <evand> heh
[15:02] <xivulon> I meant bug #144798
[15:02] <ubotu> Launchpad bug 144798 in casper "Merge lupin functionality + add external hooks" [Wishlist,Confirmed] https://launchpad.net/bugs/144798
[15:03] <cjwatson> evand: it looks horrendously complicated and much harder than my suggestion, but it's your call
[15:03] <xivulon> copied the wrong line
[15:03] <evand> ok, I'll take some more time to look at that then.
[15:03] <xivulon> cjwatson what was your proposal on that?
[15:04] <xivulon> I can recode if you give some guidance
[15:04] <cjwatson> some derivative of: sed -n '/^rootfs/,$p' /proc/mounts
[15:04] <cjwatson> messing about with line numbers is almost always an overcomplex option in Unix
[15:04] <xivulon> cjwatson one thing is not clear to me
[15:05] <xivulon> do we want to skip all mountpoints above root, or all mountpoints above rootfs?
[15:05] <xivulon> My understanding was the first case
[15:08] <cjwatson> oh, hmm, yeah, should be root. adjust the regexp accordingly
[15:09] <cjwatson> /^\/[^ ]* \//
[15:09] <cjwatson> /^\/[^ ]* \//
[15:09] <cjwatson> /^\/[^ ]* \/ /
[15:09] <cjwatson> my point is just to avoid having to read through the file multiple times, work out line numbers, use lots of commands - an elegance thing :)
[15:10] <xivulon> Heh I am not a bash esthetist, getting things to work is good enough for me :P
[15:10] <xivulon> sh
[15:12] <cjwatson> I think it's sort of important in the shutdown procedure to call as few external commands as possible, for efficiency
[15:12] <xivulon> I agree
[15:15] <xivulon> now I get the regex
[15:15] <xivulon> I like it
[15:33] <xivulon> yep much better http://paste.ubuntu-nl.org/52272/
[15:33] <xivulon> evand ^
[15:35] <cjwatson> looks good
[15:36] <xivulon> cjwatson, what is the sed command to print the top half of the file as opposed to the bottom half?
[15:36] <xivulon> needed in umountroot
[15:36] <xivulon> in fact not needed
[15:36] <cjwatson> 0,/regex/
[15:36] <xivulon> ah was trying $p,/regex/
[15:36] <xivulon> was close
[15:36] <cjwatson> well, sed -n '0,/regex/p' to be a little more verbose
[15:37] <cjwatson> alternatively sed '/regex/q' tells sed to quit as soon as it encounters /regex/ (before printing that line)
[15:37] <cjwatson> you can do similar things in awk too of course, sed just fits my brain better
[15:38] <xivulon> cjwatson, evand in bug #151579 last comment is also a patch for umounroot, can you please review that too?
[15:38] <ubotu> Launchpad bug 151579 in wubi "umountfs must check whether a mountpoint contains a loopmounted root file" [High,Fix committed] https://launchpad.net/bugs/151579
[15:39] <evand> will do
[15:40] <xivulon> I have added colin suggestion as a comment, so the other one to check is now comment n-1 https://bugs.launchpad.net/ubuntu/+source/sysvinit/+bug/151579/comments/5
[15:40] <ubotu> Launchpad bug 151579 in wubi "umountfs must check whether a mountpoint contains a loopmounted root file" [High,Fix committed]
[16:04] <xivulon> evand re lupin/#144798 https://code.launchpad.net/~ubuntu-installer/lupin/hardy, I am only intrested in the casper package, you can skip lupin-helpers alltogether
[16:04] <evand> noted
[16:05] <xivulon> lupin/casper by the way might become a separate package, called maybe custom-installation
[16:05] <xivulon> with 2 packages, 1 for casper and 1 for the alternate initrd
[16:06] <xivulon> that basically now provides external hooks to modify the installation using files on local hard disks
[16:07] <xivulon> what's your view on that?
[16:08] <evand> hrm, no complaints.  But I'm more concerned with getting this done as we have it set up at the moment.
[16:09] <xivulon> re casper, you can simply merge the old lupin/casper with the new one
[16:10] <xivulon> suggestion only moves lupin/hardy/casper to a different launchpad project, + adds code for the alternate initrd to provide equivalent hooks
[16:13] <evand> cjwatson, xivulon: May I ask that you both just eye this over one last time before I send it to the buildds? http://pastebin.ubuntu.com/3623/
[16:14] <xivulon> looks good to me
[16:16] <xivulon> thanks a lot
[16:19] <cjwatson> evand: hmm, <(...) is a bashism
[16:19] <cjwatson> that's a /bin/sh script so it'll break
[16:19] <cjwatson> maybe use a temporary file if that's possible at that stage?
[16:19] <evand> hrm
[16:22] <evand> so:
[16:22] <evand>         sed -n '/^\/[^ ]* \/ /,$p' /proc/mounts > tempmounts
[16:22] <evand>         exec 9<&0 < tempmounts
[16:22] <cjwatson> well, put it in a known-safe temporary directory (*not* /tmp; /var/run maybe?)
[16:23] <evand> ok, indeed I wasn't sure where to put it and was just going to ask
[16:23] <xivulon> skip_mountpoints=$(sed -n '0,/^\/[^ ]* \/ /p' /proc/mounts)
[16:23] <xivulon> maybe
[16:36] <xivulon> http://paste.ubuntu-nl.org/52281/
[16:36] <xivulon> evand, cjwatson ^
[16:37] <xivulon> the above those not use a file and I believe is also safer code
[16:37] <xivulon> if for instance a mountpoint appears below root using the same device of a mountpoint above root will also be skept
[16:37] <xivulon> which is the correct behaviour IMHO
[16:41] <cjwatson> err ... no, it's still important to umount such a device even if it happens to be mounted multiple times
[16:41] <xivulon> This is required when you have mountbinds for instance,
[16:41] <cjwatson> quite the opposite
[16:41] <xivulon> don't agree
[16:41] <cjwatson> you need to unmount the bind mount
[16:41] <cjwatson> you don't need to unmount the thing it's a bind-mount of
[16:41] <xivulon> true
[16:42] <cjwatson> therefore it is not correct to fail to unmount something just because the same device appears before root
[16:42] <xivulon> remembered wrongly that it would umount the device
[16:42] <xivulon> but it does umount -f -r -d $REG_MTPTS
[16:43] <cjwatson> yes, that's fine
[16:43] <xivulon> yep
[16:43] <cjwatson> you can't umount a device, anyway
[16:43] <cjwatson> the umount system call takes a mountpoint
[16:44] <xivulon>  umount [-dflnrv] dir | device
[16:44] <cjwatson> that is not a system call
[16:44] <xivulon> ah
[16:44] <cjwatson> that is a command
[16:44] <cjwatson> man 2 umount
[16:45] <xivulon> but doesn't umountfs use the command?
[16:46] <xivulon> anyway moot point
[16:46] <cjwatson> if you pass a device node to umount(8), it will simply look up the device node in mtab and figure out the mountpoint
[16:47] <cjwatson> therefore it's not really unmounting by device, it's just a convenience
[16:47] <cjwatson> my point is that you cannot tell the kernel "unmount everything mounted on this device"
[16:48] <cjwatson> though it's true that if you pass a device to umount(8) it will try to unmount it from all the places where it's mounted
[16:50] <xivulon> http://paste.ubuntu-nl.org/52284/
[16:50] <xivulon> don't uses a file, but requires cut and grep
[16:53] <cjwatson> that's incorrect. If /usr was mounted for some reason before the real / was mounted, and then a new /usr mounted over the top, the new /usr *must* still be unmounted during umountfs.
[16:54] <xivulon> true
[16:55] <xivulon> that's what I call a corner case
[16:55] <cjwatson> but you seem to be putting serious effort into breaking this corner case when it's not necessary
[16:55] <cjwatson> making this corner case work is *simpler*
[16:56] <cjwatson> Evan's code above will take care of it just fine
[16:56] <xivulon> one less then
[17:00] <xivulon> evand you mentioned you were also working on #173659 any comment there? I'll be leaving in about 30m
[17:01] <evand> nothing to report yet, I'm tied up with this sysvinit stuff at the moment.
[17:02] <xivulon> sure
[20:25] <evand> argh, xivulon's /host/boot to fstab patches fail for ubuntu inside ubuntu as you'll always have a boot directory inside /host.
[22:35] <majikins> hi
[22:35] <majikins> can someone help me with raid1 and lvm?
[22:36] <majikins> I've setup via a howto on youtube
[22:36] <majikins> testing to see if failover works
[22:37] <majikins> disconnect one disk from the raid1 set and kernel loads but just gets stuck on either usb module load on cdrom
[22:37] <majikins> on either disk
[22:39] <majikins> anyone?
[22:39] <Thugacation> yes?
[22:39] <majikins> cool
[22:39] <majikins> hi - hope you can help
[22:39] <Thugacation> probably not
[22:40] <majikins> maybe u can suggest where to go?
[22:40] <majikins> I've installed ubuntu server with raid1 and lvm
[22:40] <majikins> raid1 part does not quite work
[22:41] <majikins> disconnect either disk of raid1 array and the system does not fully boot
[22:46] <cjwatson> I think you need the kernel people rather than here
[22:46] <cjwatson> #ubuntu-kernel
[22:47] <majikins> cool - thank you
[22:47] <CIA-22> oem-config: cjwatson * r397 oem-config/ (5 files in 4 dirs): * Activate appropriate input methods when changing language (LP: #181857).
[22:48] <cjwatson> ^- fear the nasty code
[22:48] <CIA-22> oem-config: cjwatson * r398 oem-config/lib/im_switch.py: remove stray debugging code
[22:50] <cjwatson> evand: I think we might want something very similar to oem-config r397 in ubiquity at some point, though I'll want to test it more first
[23:08] <xivulon> evand any bug you want to go through tonight?
[23:10] <Thugacation> if you use wubi-installer to install linux to a folder on a windows os, doesnt that mean linux will run off of windows
[23:10] <xivulon> nope
[23:11] <xivulon> you can delete c:\windows and still boot wubi
[23:11] <Thugacation> oooh... ahhh
[23:11] <xivulon> you cannot remove the c: partition though
[23:12] <Thugacation> i see
[23:12] <cjwatson> Thugacation: it's just running from the Windows filesystem, that's all
[23:12] <cjwatson> the Windows operating system is not running or otherwise involved
[23:12] <Thugacation> yeah i had a brainfart for a second
[23:12] <Thugacation> like windows is in c:\windows and wubi would be c:\wubi or something
[23:12] <xivulon> basically a linux sees a file as a whole hard disk