[Ovirt-devel] [Patch] Final change set for developer iscsi setup

[Chris Lalancette]

mark wagner wrote:
> 
> ------------------------------------------------------------------------
> diff --git a/wui-appliance/common-pkgs.ks b/wui-appliance/common-pkgs.ks
> index 15c47aa..af2766f 100644
> --- a/wui-appliance/common-pkgs.ks
> +++ b/wui-appliance/common-pkgs.ks
> @@ -21,6 +21,7 @@ ipa-admintools
>  xinetd
>  libvirt
>  cyrus-sasl-gssapi
> +scsi-target-utils
>  iscsi-initiator-utils
>  collectd
>  ruby-libvirt
> @@ -45,4 +46,4 @@ livecd-tools
>  -fetchmail
>  -slrn
>  -cadaver
> --mutt
> \ No newline at end of file
> +-mutt
> diff --git a/wui-appliance/devel-post.ks b/wui-appliance/devel-post.ks
> index 3ce6239..c925204 100644
> --- a/wui-appliance/devel-post.ks
> +++ b/wui-appliance/devel-post.ks
> @@ -230,3 +230,23 @@ esac
>  EOF
>  chmod +x /etc/init.d/ovirt-app-first-run
>  /sbin/chkconfig ovirt-app-first-run on
> +
> +# Setup the iscsi stuff to be ready on each boot.  Since tgtadm does not use a config file
> +# append what we need to the rc.local file.  Note that this for the developers version only. 
> +
> +cat >> /etc/rc.d/rc.local << \EOF
> +#
> +# Set up the fake iscsi targets
> +tgtadm --lld iscsi --op new --mode target --tid 1 -T ovirtpriv:storage
> +#
> +# Now associate them to the LVs
> +# 
> +tgtadm --lld iscsi --op new --mode logicalunit --tid 1 --lun 1 -b /dev/VolGroup00/iSCSI1
> +tgtadm --lld iscsi --op new --mode logicalunit --tid 1 --lun 2 -b /dev/VolGroup00/iSCSI2
> +tgtadm --lld iscsi --op new --mode logicalunit --tid 1 --lun 3 -b /dev/VolGroup00/iSCSI3
> +# 
> +# Now make them available
> +#
> +tgtadm --lld iscsi --op bind --mode target --tid 1 -I ALL
> +EOF
> +

That's more like it.  I think you missed the changes to wui-devel-i386.ks and
wui-devel-x86_64.ks in this latest patch, but I took those from earlier patches.
 Committed to tip in git now (but note that I had to modify the patch somewhat,
since things have moved around in tip now).  We can cherry-pick it back onto the
release branch if people want, too.

Thanks!
Chris Lalancette