Date: Wed, 09 Oct 2013 13:47:19 -0400 From: Allan Jude <freebsd@allanjude.com> To: "Teske, Devin" <Devin.Teske@fisglobal.com> Cc: "<freebsd-current@freebsd.org>" <freebsd-current@freebsd.org> Subject: Re: [CFT] Patch to bsdinstall to support root-on-ZFS and GELI Message-ID: <525596A7.2090701@allanjude.com> In-Reply-To: <13CA24D6AB415D428143D44749F57D720FC4B3F2@LTCFISWMSGMB21.FNFIS.com> References: <52531295.7090700@allanjude.com> <5254D231.5070803@allanjude.com> <13CA24D6AB415D428143D44749F57D720FC4B3F2@LTCFISWMSGMB21.FNFIS.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 2013-10-09 13:21, Teske, Devin wrote: > On Oct 8, 2013, at 8:49 PM, Allan Jude wrote: > >> On 2013-10-07 15:59, Allan Jude wrote: >>> Devin Teske and I have been working on a big patch to bsdinstall to >>> implement installing on a ZFS pool. It supports both GPT and MBR, the 4k >>> sector gnop trick, and optional GELI encryption. We would like to commit >>> this in time for 10.0-BETA1 so it needs some testing to work out any >>> obvious bugs before we send it off to re@ to get it committed. >>> >>> It includes a single configuration menu that allows you to select all of >>> the required details, including which drives to use (gets details from >>> camcontrol, also includes an inspection utility that presents the >>> detailed output of camcontrol inquiry/identify, and gpart show), what >>> ZFS RAID level to use (taking in to consideration the selected number of >>> drives), GPT/mbr, 4k YES/no, GELI yes/NO, pool name, etc. >>> >>> >>> Additional, it includes some other changes to bsdinstall: >>> 1. Change the default to the 'non-standard keyboard mapping' prompt to no >>> 2. Replace the 3 separate dialogs to configure an ipv4 address with just 1 >>> 3. Remove the dialog asking if you wish to enable crash dumps, this >>> feature has been combined into the regular 'services to enable' dialog >>> and enabled by default >>> >>> >>> You can browse the patches here: >>> http://druidbsd.cvs.sf.net/viewvc/druidbsd/bsdinstall_zfs/ >>> >>> I've built a bootonly.iso (10.0-ALPHA4) to make testing easier, >>> available compressed (48 MB) or uncompressed (211 MB): >>> >>> http://www.allanjude.com/bsd/zfsbootonly_2013-10-06.iso.xz >>> >>> http://www.allanjude.com/bsd/zfsbootonly_2013-10-06.iso >>> >>> >>> We look forward to your feedback >>> >> We've made more improvements, including corporating most all of the >> feedback we've gotten so far >> >> >> Outstanding items: >> 1. Apply the changes to ipv6 config the way we did ipv4 >> 2. improve disk identification (model info and serial # instead of one >> or the other) >> 3. Include a helpful message before the GELI step where you have to >> enter your password many times, the user will be less confused if it is >> explained why they have to enter their password 3 * number of disks times > I'm hopeful that we can script the application of a password that we > first prompt for. > > What tool is prompting for a password? Can we not just provide an answer > on stdin? (e.g., echo "$pass" | tool_that_needs_pass) > It is 'geli create' and 'geli attach'. I am not sure if we want to have the password show up in the process list (obviously in the installer this is less of an issue, but) > >> 4. Validate vdev type choice inside the vdev type menu, and warn the >> user if they have made an invalid selection, so they can add more disks >> or chance their selection, without having to try to start the >> installation first > This will be done with fanciness ;D (read: ... --and-widget --infobox ... and > sundry smartness; retaining as much as possible the ability to do things > out of order but never arise at a point of astonishment). > I don't think we need --and-widget, just in the function where we apply the results of the menu selection, we can add a regular --msgbox telling them that their config won't work, and they need to either select more drives or a different vdev type >> 5. Whatever else you guys find wrong tonight >> >> I generated new test images, and attached the patch (which got REALLY >> big when Devin Teske decided to fix "all of the things": >> > And then I merged "all of the things" into HEAD, so the patch-set shrunk > back to its normal size. Now we have global exit codes which will make > merging of code that is based off of Thomas Dickey's samples easier. I am glad to see all of the good ideas, and plans to make everything wonderful, but my biggest concern is getting this over to re@ so it can get in to 10.0-BETA1, the deadline for which is looming (like, tomorrow I think). As such, I have rolled back the patches to netconfig and netconfig_ipv4 (my stuff to reduce the number of dialogs to configure ipv4, it posed some problems with the possible usage of xdialog, and didn't actually offer an option to 'cancel'). I kept Warren's netconfig wireless patch This leaves the only real outstanding problem the keymap thing. I propose changing it from a yes/no/other to a --menu, and hopefully we can find the bug with the display name, or just make it show the keymap name instead. -- Allan Jude
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?525596A7.2090701>