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