mark wagner
2008-Mar-26 10:43 UTC
[Ovirt-devel] [Patch] Final change set for developer iscsi setup
-------------- next part -------------- A non-text attachment was scrubbed... Name: iscsi3.patch Type: text/x-patch Size: 1437 bytes Desc: not available URL: <http://listman.redhat.com/archives/ovirt-devel/attachments/20080326/862b539a/attachment.bin>
Chris Lalancette
2008-Mar-26 13:48 UTC
[Ovirt-devel] [Patch] Final change set for developer iscsi setup
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