Change in vdsm[ovirt-3.4]: sysvinit: Ensure that iscsid is started

ybronhei at redhat.com ybronhei at redhat.com
Tue Jan 28 12:05:57 UTC 2014


Yaniv Bronhaim has submitted this change and it was merged.

Change subject: sysvinit: Ensure that iscsid is started
......................................................................


sysvinit: Ensure that iscsid is started

vdsm is starting the iscsid daemon when it starts, but iscsid "start"
command may silently *not* start the iscsid daemon, because vdsm is
configuring it's nodes to login manually. When iscsid daemon is not
running and vdsm is connecting to a storage server, iscsiadm starts
iscsid daemon. This works when /etc/iscsi/iscsi.conf contains:

    iscsid.startup = /etc/rc.d/init.d/iscsid force-start

This is the default configuration on a fresh install of EL 6, and is
common on developers machines. However when this option is missing,
iscsiadm fails and iscsi storage is not available. According to the
comments on the bug, the missing configuration is common in the field.

This patch starts iscsid using "force-start" command, ensuring that
iscsid daemon is started and removing the dependency on iscsi.startup
configuration. This restores the behavior from commit 0233b5e952 in Nov
2011 until commit 3c0adf791e in May 2013. The behavior was changed to
make systemd-based deployment (which has no "force-start") start iscsid,
but now that we have an independent systemd service, it can be safely
avoided.

This change does not effect the deployment for systemd or upstart.

Change-Id: Idb5c3de4f8dccb1d8344182d5016feabcf86a7df
Relates-To: http://gerrit.ovirt.org/14630
Bug-Url: https://bugzilla.redhat.com/1056948
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/23601
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/23770
Reviewed-by: Allon Mureinik <amureini at redhat.com>
---
M init/sysvinit/vdsmd.init.in
1 file changed, 7 insertions(+), 1 deletion(-)

Approvals:
  Nir Soffer: Verified
  Allon Mureinik: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/23770
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Idb5c3de4f8dccb1d8344182d5016feabcf86a7df
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.4
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list