Change in ovirt-hosted-engine-ha[master]: sharedconf: always read/write the shared configuration as VD...

[msivak at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Rafael Martins <rmartins at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org