From owner-freebsd-current@FreeBSD.ORG Mon Oct 21 16:22:40 2013 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id 28CFDA74 for ; Mon, 21 Oct 2013 16:22:40 +0000 (UTC) (envelope-from johan@bridgenet.se) Received: from smtp-gw21.han.skanova.net (smtp-gw21.han.skanova.net [81.236.55.21]) by mx1.freebsd.org (Postfix) with ESMTP id A417A2307 for ; Mon, 21 Oct 2013 16:22:39 +0000 (UTC) Received: from mail.bridgenet.se (212.181.212.146) by smtp-gw21.han.skanova.net (8.5.133) id 52551723005813EB for freebsd-current@freebsd.org; Mon, 21 Oct 2013 18:22:38 +0200 Received: from mail.bridgenet.se (unknown [10.0.0.5]) by mail.bridgenet.se (Postfix) with ESMTP id 9E78B18AFE for ; Mon, 21 Oct 2013 18:22:38 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=bridgenet.se; h= content-transfer-encoding:content-type:content-type:in-reply-to :references:subject:subject:to:mime-version:user-agent:from:from :date:date:message-id; s=dkim; t=1382372556; x=1383236557; bh=Qr 3y2bOAiiJd448sstkqJh/Jyc/TPUCypFNmg+k1Dio=; b=XUwjyFCXKTYEdIxWmX gPUnr1hL7SLYEZCHXy0F4UH40S6HOwlMwRHwIiYmkp/NoWa7yQhIVR8KswAbxNbC 3+b1q3j6LXjHxzT9Gy2XufyMC9WCDt+hEWWvd64bMmVd+QPFqIwO+xzZY83Glot8 kr+ncc9sWYfD5Cb8JRBl31sy0= X-Virus-Scanned: amavisd-new at mail.bridgenet.se Received: from mail.bridgenet.se ([10.0.0.5]) by mail.bridgenet.se (mail.bridgenet.se [10.0.0.5]) (amavisd-new, port 10024) with ESMTP id WIomOzDahFGW for ; Mon, 21 Oct 2013 18:22:36 +0200 (CEST) Received: from dhcp-208-143.arn.redhat.com (unknown [192.168.0.129]) by mail.bridgenet.se (Postfix) with ESMTPSA id 5D31218AE7; Mon, 21 Oct 2013 18:22:26 +0200 (CEST) Message-ID: <526554C1.5040102@bridgenet.se> Date: Mon, 21 Oct 2013 18:22:25 +0200 From: Johan Broman User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.0 MIME-Version: 1.0 To: Allan Jude , Devin Teske Subject: Re: [CFT] Patch to bsdinstall to support root-on-ZFS and GELI References: <52629DA7.7090103@bridgenet.se> <5262A3EE.4050600@allanjude.com> <5262A5D0.3050604@bridgenet.se> <5262A69D.5070601@allanjude.com> <13CA24D6AB415D428143D44749F57D720FC7E9DE@LTCFISWMSGMB21.FNFIS.com> <13CA24D6AB415D428143D44749F57D720FC7EAEC@LTCFISWMSGMB21.FNFIS.com> <5262B304.3090605@allanjude.com> <5262BC56.4050004@bridgenet.se> <13CA24D6AB415D428143D44749F57D720FC7F916@LTCFISWMSGMB21.FNFIS.com> <52654C0C.80702@bridgenet.se> <52654D47.6020106@allanjude.com> <13CA24D6AB415D428143D44749F57D720FC8FE7F@LTCFISWMSGMB21.FNFIS.com> <52655118.6010909@allanjude.com> In-Reply-To: <52655118.6010909@allanjude.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: " Current" , "Teske, Devin" X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 21 Oct 2013 16:22:40 -0000 On 21/10/13 18:06, Allan Jude wrote: > On 2013-10-21 12:04, Teske, Devin wrote: >> On Oct 21, 2013, at 8:50 AM, Allan Jude wrote: >> >>> On 2013-10-21 11:45, Johan Broman wrote: >>>> Hi! >>>> >>>> Sorry for the delayed answer. I've patched zfsboot and rebuilt the >>>> release. I now get an error message that ada2 can't be used, which is >>>> correct. Good stuff! :) >>>> >>>> ( I've recreated the test environment using a KVM guest with four SATA >>>> drives instead of the server I was using. I makes it easier to test >>>> stuff. ) >>>> >>>> Here's the screenshot: >>>> >>>> https://urldefense.proofpoint.com/v1/url?u=http://212.181.212.146/bsdinstall/Screenshot_2013-10-21.png&k=%2FbkpAUdJWZuiTILCq%2FFnQg%3D%3D%0A&r=Mrjs6vR4%2Faj2Ns9%2FssHJjg%3D%3D%0A&m=B6jLX5vSxIfvyks3HH55lYWtfhRBpGZ3nVA65M%2FgmXM%3D%0A&s=f6fc4ec7c9d8b9486e897156dac5af7c3f64e3fae746dcf6c66ad91564a8ce99 >>>> >>>> >>>> Maybe one should be unable to select drives that are part of a graid >>>> in the first place? Or is that out-of-scope for bsdinstall at this >>>> point? (As I guess that requires too many changes/new lines) >>>> >>>> >>>> Cheers >>>> Johan >>>> >>>> >>>> On 19/10/13 22:20, Teske, Devin wrote: >>>>> On Oct 19, 2013, at 10:07 AM, Johan Broman wrote: >>>>> >>>>>> I recreated the graid mirror on ada2 and ada3 and reran the >>>>>> installation. I'm unable to scroll the msgbox using PgDn or arrow >>>>>> keys. There is no indication that the action failed and I'm returned >>>>>> to the ZFS setup screen if I hit OK. >>>>>> >>>>>> I have screen shots (taken with my phone) of the msgbox and "ps >>>>>> auxwww" output. Let me know what kind of debug info you would like. >>>>>> I've put the screen shots here: >>>>>> >>>>>> https://urldefense.proofpoint.com/v1/url?u=http://212.181.212.146/bsdinstall&k=%2FbkpAUdJWZuiTILCq%2FFnQg%3D%3D%0A&r=Mrjs6vR4%2Faj2Ns9%2FssHJjg%3D%3D%0A&m=B6jLX5vSxIfvyks3HH55lYWtfhRBpGZ3nVA65M%2FgmXM%3D%0A&s=6322000e13ed155bda748698c4a0d54c9de7c29f5566affe202d7c5a29917cd1 >>>>> I've added a patch to fix debugging in the zfsboot script... >>>>> >>>>> https://urldefense.proofpoint.com/v1/url?u=http://druidbsd.cvs.sf.net/viewvc/druidbsd/bsdinstall_zfs/&k=%2FbkpAUdJWZuiTILCq%2FFnQg%3D%3D%0A&r=Mrjs6vR4%2Faj2Ns9%2FssHJjg%3D%3D%0A&m=B6jLX5vSxIfvyks3HH55lYWtfhRBpGZ3nVA65M%2FgmXM%3D%0A&s=1dc96a8b5450d27fe8210b361c9ed736ccd448f78df6aabe170bb80e31bca6d9 >>>>> >>>>> Feedback welcome. >>>>> >>>>> Johan,... >>>>> >>>>> Can you see if the patch sheds some better light as to what's failing? >>>>> >>>>> The patch won't fix the problem, but it should give us an accurate error >>>>> message so that we can learn what precisely is returning an error >>>>> status. >>>>> >>>>> Thanks in advance. >>>>> >>> I do notice that Devin's manually prefixing the error message with the >>> tool name, is partially redundancy when the tool does it it self, but we >>> can't always be sure it will do that. >>> >> The next patchset will fix that. >> >> I'm dropping the tool name from the msgbox contents and putting it in >> the title (e.g., '"Error: gpart") that way... even if the tool spits out its own >> name (or not), we'll know what exactly what was going on by looking >> at the title. >> >> >>> the graid thing is rather hard to detect, especially when it is a >>> faulted array that doesn't even appear in graid status etc. >>> >> I believe the idea behind the script is that whatever you tell it to use will >> be destroyed. >> >> Allan, maybe perhaps we could add some code that attempts to dis- >> assemble a graid to make the disk usable? >> >> Johan, what would you be more apt to expect? That it killed your graid >> or that it gave an error? (/me thinks what the recourse to the error might >> entail -- going to partedit?) > Your recourse would be switching to the shell (control+alt+f4) and > destroying the graid. > > I am a little hesitant to go destroying graids unprompted. If we had the > geom.confxml parsing, we might be able to detect it and ask the user > what to do > In this case I was unaware of the fact that the old disks I was using had previously been used in a graid. So I just wanted it to destroy it and move on :) On the other hand, Allan has a valid point. The user does get a warning (the "last chance" warning), but maybe raising and additional warning if a graid is detected is a good thing? My opinion - one warning saying (quite clearly) that the disks will be destroyed should be enough :) /my 5 cents :) Cheers Johan