Change in ovirt-engine[master]: tools: error in engine-notifierd init script

masayag at redhat.com masayag at redhat.com
Tue Feb 12 21:46:13 UTC 2013


Moti Asayag has submitted this change and it was merged.

Change subject: tools: error in engine-notifierd init script
......................................................................


tools: error in engine-notifierd init script

When starting the notification service, the init script calls
/usr/share/ovirt-engine/bin/engine-notifier.sh ($NOTIFIER_SCRIPT)
through the daemon() function, which can run something as another
user--in this case, user 'ovirt'.

If the init script is run from a directory that 'ovirt' doesn't have
permission to access, a `find` in engine-notifier.sh will print an
error.  Prevent this error by having the init script change to the
ovirt home directory before starting the daemon.

Change-Id: I83622cbb8b39ea5792907e6efc68d824ca4b02da
Bug-Url: https://bugzilla.redhat.com/885446
Signed-off-by: Greg Padgett <gpadgett at redhat.com>
---
M backend/manager/tools/src/main/shell/engine-notifier-service.sh
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Moti Asayag: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I83622cbb8b39ea5792907e6efc68d824ca4b02da
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list