Ovirt devel - Mar 2009 - [PATCH node] show HAL UDI for the selected disk

HAL UDI includes serial# (if available) and other information,
to help disk identification.

Signed-off-by: Alan Pevec <apevec at redhat.com>
---
 scripts/ovirt-config-storage |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/scripts/ovirt-config-storage b/scripts/ovirt-config-storage
index c92b931..a102164 100755
--- a/scripts/ovirt-config-storage
+++ b/scripts/ovirt-config-storage
@@ -55,7 +55,7 @@ get_selected_drive_size()
     fi
 
     SPACE=$(echo "scale=0; $size / (1024 * 1024)" | bc -l)
-    log "Selected Device: $DRIVE ($SPACE MB)"
+    log "Selected Device: $DRIVE ($SPACE MB) $(basename
"$udi")"
     stop_log
 }
 
@@ -101,10 +101,9 @@ check_partition_sizes()
 }')
 	if [ "$name" == "HostVG" ]; then
 	    udi=$(hal-find-by-property --key block.device --string $device)
-	    name=$(hal-get-property --udi $udi --key storage.serial)
 	    printf "\n"
 	    printf "There appears to already be an installation of oVirt on
another device:\n"
-	    printf "\t$device ($name)\n"
+	    printf "\t$device ($(basename "$udi"))\n"
 	    printf "We cannot proceed until either this device is removed from
the system\n"
 	    printf "or until the volume group is removed.\n"
 	    printf "\nTo re-install the node, please select \"Uninstall
Node\" from the main\n"
-- 
1.6.0.6

On Sat, Mar 14, 2009 at 01:54:35AM +0100, Alan Pevec
wrote:
> HAL UDI includes serial# (if available) and other information,
> to help disk identification.
> 
> Signed-off-by: Alan Pevec <apevec at redhat.com>
ACK. Looks good.

-- 
Darryl L. Pierce, Sr. Software Engineer @ Red Hat, Inc.
Virtual Machine Management - http://www.ovirt.org/
Is fearr Gaeilge bhriste n? B?arla cliste.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL:
<http://listman.redhat.com/archives/ovirt-devel/attachments/20090316/71215dac/attachment.sig>