Change in ovirt-hosted-engine-ha[master]: sharedconf: always read/write the shared configuration as VD...
Martin Sivák has submitted this change and it was merged.
Change subject: sharedconf: always read/write the shared configuration as VDSM user
......................................................................
sharedconf: always read/write the shared configuration as VDSM user
The shared conf lib is in use also by ovirt-hosted-engine-setup
which is executed as root. On NFS shared storage, root could be
inhibited to write so ensure we are always writing as VDSM user.
Removing latest dependency on heconflib from
from ovirt-hosted-engine-setup
Change-Id: I973c1423b7f07b37014d2c1f688f28f730a7eddc
Bug-Url: https://bugzilla.redhat.com/1254671
Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com>
---
M ovirt_hosted_engine_ha/env/config.py
M ovirt_hosted_engine_ha/lib/heconflib.py
2 files changed, 16 insertions(+), 3 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, but someone else must approve
Martin Sivák: Looks good to me, approved
Simone Tiraboschi: Verified
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/45812
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I973c1423b7f07b37014d2c1f688f28f730a7eddc
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org