/srv/irclogs.ubuntu.com/2008/01/17/#ubuntu-installer.txt

agoevand any other bug you want to go through tonight?00:01
tjaaltoncould I request a new d-i against -4 kernel?08:23
cjwatsontjaalton: sure, on its way09:18
CIA-22debian-installer: cjwatson * r869 ubuntu/ (10 files in 4 dirs): * Move to 2.6.24-4 kernels.09:18
tjaaltoncjwatson: thanks!09:18
CIA-22debian-installer: cjwatson * r870 ubuntu/debian/changelog: releasing version 20070308ubuntu2609:20
=== cjwatson_ is now known as cjwatson
=== cjwatson_ is now known as cjwatson
xivulonhi evand14:54
xivulonany q over remaining patches?14:54
evandxivulon: I'll let you know, but none currently.14:54
xivulonwhat's the status of bug #173659 and bug #151579?14:58
ubotuLaunchpad bug 173659 in wubi "Add /host/boot to fstab" [Medium,Fix committed] https://launchpad.net/bugs/17365914:58
ubotuLaunchpad bug 151579 in wubi "umountfs must check whether a mountpoint contains a loopmounted root file" [High,Fix committed] https://launchpad.net/bugs/15157914:58
evandI'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
xivulonevand 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
ubotuLaunchpad 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/17786815:01
evandAre you sure that's the right bug?15:02
xivulonThat mostly involves merging https://code.launchpad.net/~ubuntu-installer/lupin/hardy15:02
xivulonehm nope15:02
evandheh15:02
xivulonI meant bug #14479815:02
ubotuLaunchpad bug 144798 in casper "Merge lupin functionality + add external hooks" [Wishlist,Confirmed] https://launchpad.net/bugs/14479815:02
cjwatsonevand: it looks horrendously complicated and much harder than my suggestion, but it's your call15:03
xivuloncopied the wrong line15:03
evandok, I'll take some more time to look at that then.15:03
xivuloncjwatson what was your proposal on that?15:03
xivulonI can recode if you give some guidance15:04
cjwatsonsome derivative of: sed -n '/^rootfs/,$p' /proc/mounts15:04
cjwatsonmessing about with line numbers is almost always an overcomplex option in Unix15:04
xivuloncjwatson one thing is not clear to me15:04
xivulondo we want to skip all mountpoints above root, or all mountpoints above rootfs?15:05
xivulonMy understanding was the first case15:05
cjwatsonoh, hmm, yeah, should be root. adjust the regexp accordingly15:08
cjwatson/^\/[^ ]* \//15:09
cjwatson/^\/[^ ]* \//15:09
cjwatson/^\/[^ ]* \/ /15:09
cjwatsonmy 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:09
xivulonHeh I am not a bash esthetist, getting things to work is good enough for me :P15:10
xivulonsh15:10
cjwatsonI think it's sort of important in the shutdown procedure to call as few external commands as possible, for efficiency15:12
xivulonI agree15:12
xivulonnow I get the regex15:15
xivulonI like it15:15
xivulonyep much better http://paste.ubuntu-nl.org/52272/15:33
xivulonevand ^15:33
cjwatsonlooks good15:35
xivuloncjwatson, what is the sed command to print the top half of the file as opposed to the bottom half?15:36
xivulonneeded in umountroot15:36
xivulonin fact not needed15:36
cjwatson0,/regex/15:36
xivulonah was trying $p,/regex/15:36
xivulonwas close15:36
cjwatsonwell, sed -n '0,/regex/p' to be a little more verbose15:36
cjwatsonalternatively sed '/regex/q' tells sed to quit as soon as it encounters /regex/ (before printing that line)15:37
cjwatsonyou can do similar things in awk too of course, sed just fits my brain better15:37
xivuloncjwatson, evand in bug #151579 last comment is also a patch for umounroot, can you please review that too?15:38
ubotuLaunchpad bug 151579 in wubi "umountfs must check whether a mountpoint contains a loopmounted root file" [High,Fix committed] https://launchpad.net/bugs/15157915:38
evandwill do15:39
xivulonI 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/515:40
ubotuLaunchpad bug 151579 in wubi "umountfs must check whether a mountpoint contains a loopmounted root file" [High,Fix committed]15:40
xivulonevand re lupin/#144798 https://code.launchpad.net/~ubuntu-installer/lupin/hardy, I am only intrested in the casper package, you can skip lupin-helpers alltogether16:04
evandnoted16:04
xivulonlupin/casper by the way might become a separate package, called maybe custom-installation16:05
xivulonwith 2 packages, 1 for casper and 1 for the alternate initrd16:05
xivulonthat basically now provides external hooks to modify the installation using files on local hard disks16:06
xivulonwhat's your view on that?16:07
evandhrm, no complaints.  But I'm more concerned with getting this done as we have it set up at the moment.16:08
xivulonre casper, you can simply merge the old lupin/casper with the new one16:09
xivulonsuggestion only moves lupin/hardy/casper to a different launchpad project, + adds code for the alternate initrd to provide equivalent hooks16:10
evandcjwatson, 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:13
xivulonlooks good to me16:14
xivulonthanks a lot16:16
cjwatsonevand: hmm, <(...) is a bashism16:19
cjwatsonthat's a /bin/sh script so it'll break16:19
cjwatsonmaybe use a temporary file if that's possible at that stage?16:19
evandhrm16:19
evandso:16:22
evand        sed -n '/^\/[^ ]* \/ /,$p' /proc/mounts > tempmounts16:22
evand        exec 9<&0 < tempmounts16:22
cjwatsonwell, put it in a known-safe temporary directory (*not* /tmp; /var/run maybe?)16:22
evandok, indeed I wasn't sure where to put it and was just going to ask16:23
xivulonskip_mountpoints=$(sed -n '0,/^\/[^ ]* \/ /p' /proc/mounts)16:23
xivulonmaybe16:23
xivulonhttp://paste.ubuntu-nl.org/52281/16:36
xivulonevand, cjwatson ^16:36
xivulonthe above those not use a file and I believe is also safer code16:37
xivulonif for instance a mountpoint appears below root using the same device of a mountpoint above root will also be skept16:37
xivulonwhich is the correct behaviour IMHO16:37
cjwatsonerr ... no, it's still important to umount such a device even if it happens to be mounted multiple times16:41
xivulonThis is required when you have mountbinds for instance,16:41
cjwatsonquite the opposite16:41
xivulondon't agree16:41
cjwatsonyou need to unmount the bind mount16:41
cjwatsonyou don't need to unmount the thing it's a bind-mount of16:41
xivulontrue16:41
cjwatsontherefore it is not correct to fail to unmount something just because the same device appears before root16:42
xivulonremembered wrongly that it would umount the device16:42
xivulonbut it does umount -f -r -d $REG_MTPTS16:42
cjwatsonyes, that's fine16:43
xivulonyep16:43
cjwatsonyou can't umount a device, anyway16:43
cjwatsonthe umount system call takes a mountpoint16:43
xivulon umount [-dflnrv] dir | device16:44
cjwatsonthat is not a system call16:44
xivulonah16:44
cjwatsonthat is a command16:44
cjwatsonman 2 umount16:44
xivulonbut doesn't umountfs use the command?16:45
xivulonanyway moot point16:46
cjwatsonif you pass a device node to umount(8), it will simply look up the device node in mtab and figure out the mountpoint16:46
cjwatsontherefore it's not really unmounting by device, it's just a convenience16:47
cjwatsonmy point is that you cannot tell the kernel "unmount everything mounted on this device"16:47
cjwatsonthough 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 mounted16:48
xivulonhttp://paste.ubuntu-nl.org/52284/16:50
xivulondon't uses a file, but requires cut and grep16:50
cjwatsonthat'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:53
xivulontrue16:54
xivulonthat's what I call a corner case16:55
cjwatsonbut you seem to be putting serious effort into breaking this corner case when it's not necessary16:55
cjwatsonmaking this corner case work is *simpler*16:55
cjwatsonEvan's code above will take care of it just fine16:56
xivulonone less then16:56
xivulonevand you mentioned you were also working on #173659 any comment there? I'll be leaving in about 30m17:00
evandnothing to report yet, I'm tied up with this sysvinit stuff at the moment.17:01
xivulonsure17:02
evandargh, xivulon's /host/boot to fstab patches fail for ubuntu inside ubuntu as you'll always have a boot directory inside /host.20:25
=== mtp_ is now known as matttp
majikinshi22:35
majikinscan someone help me with raid1 and lvm?22:35
majikinsI've setup via a howto on youtube22:36
majikinstesting to see if failover works22:36
majikinsdisconnect one disk from the raid1 set and kernel loads but just gets stuck on either usb module load on cdrom22:37
majikinson either disk22:37
majikinsanyone?22:39
Thugacationyes?22:39
majikinscool22:39
majikinshi - hope you can help22:39
Thugacationprobably not22:39
majikinsmaybe u can suggest where to go?22:40
majikinsI've installed ubuntu server with raid1 and lvm22:40
majikinsraid1 part does not quite work22:40
majikinsdisconnect either disk of raid1 array and the system does not fully boot22:41
cjwatsonI think you need the kernel people rather than here22:46
cjwatson#ubuntu-kernel22:46
majikinscool - thank you22:47
CIA-22oem-config: cjwatson * r397 oem-config/ (5 files in 4 dirs): * Activate appropriate input methods when changing language (LP: #181857).22:47
cjwatson^- fear the nasty code22:48
CIA-22oem-config: cjwatson * r398 oem-config/lib/im_switch.py: remove stray debugging code22:48
cjwatsonevand: 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 first22:50
xivulonevand any bug you want to go through tonight?23:08
Thugacationif you use wubi-installer to install linux to a folder on a windows os, doesnt that mean linux will run off of windows23:10
xivulonnope23:10
xivulonyou can delete c:\windows and still boot wubi23:11
Thugacationoooh... ahhh23:11
xivulonyou cannot remove the c: partition though23:11
Thugacationi see23:12
cjwatsonThugacation: it's just running from the Windows filesystem, that's all23:12
cjwatsonthe Windows operating system is not running or otherwise involved23:12
Thugacationyeah i had a brainfart for a second23:12
Thugacationlike windows is in c:\windows and wubi would be c:\wubi or something23:12
xivulonbasically a linux sees a file as a whole hard disk23:12

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