Change in vdsm[master]: vdsmd.init: remove validation to reload config

dougsland at redhat.com dougsland at redhat.com
Wed May 16 15:54:49 UTC 2012


Douglas Schilling Landgraf has uploaded a new change for review.

Change subject: vdsmd.init: remove validation to reload config
......................................................................

vdsmd.init: remove validation to reload config

packaged=`/bin/rpm -ql libvirt | /bin/grep libvirtd.upstart | /usr/bin/tail -1`
returns: "/usr/share/doc/libvirt.0.X.Y/libvirtd.upstart"

However, in oVirt Node/RHEV-H we don't have /usr/share/doc dir.

So, when executing:

    /bin/cp -p "$packaged" "$target" || return 1

The above code will always execute 'return 1' since the source
$packaged doesn't exist, making the function exit before start libvirt
daemon via upstart.

Start operation is not often required, let's just reload configuration and start
libvirt daemon.

Change-Id: I7ea03070bf2849e10c74d7921eac9401e2768502
Signed-off-by: Douglas Schilling Landgraf <dougsland at redhat.com>
---
M vdsm/vdsmd.init.in
1 file changed, 1 insertion(+), 10 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/79/4479/1
--
To view, visit http://gerrit.ovirt.org/4479
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7ea03070bf2849e10c74d7921eac9401e2768502
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>


More information about the vdsm-patches mailing list