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