Change in ovirt-hosted-engine-ha[master]: packaging: fixing bad permissions on answer file from the past

Code Review gerrit at ovirt.org
Wed Jun 14 09:58:09 UTC 2017


>From Martin Sivák <msivak at redhat.com>:

Martin Sivák has submitted this change and it was merged.

Change subject: packaging: fixing bad permissions on answer file from the past
......................................................................


packaging: fixing bad permissions on answer file from the past

answers.conf is in the configuration volume on the shared
storage since 3.6 while it was just a local file before.
The 3.5 -> 3.6 upgrade procedure is able to move it to
the shared storage but it runs in ovirt-ha-agent deamon
as vdsm user so answers.conf should be readable by vdsm user.
This patch has been previously applied to
ovirt-hosted-engine-setup but it's more correct to have it here
since the upgrade is performed by ovirt-ha-agent.

Change-Id: Iddc518e4f326bbc4831843e0b980348eb3ebafa3
Signed-off-by: Simone Tiraboschi <stirabos at redhat.com>
---
M ovirt-hosted-engine-ha.spec.in
1 file changed, 8 insertions(+), 0 deletions(-)

Approvals:
  Sandro Bonazzola: Verified; Looks good to me, but someone else must approve
  Martin Sivák: Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/77964
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iddc518e4f326bbc4831843e0b980348eb3ebafa3
Gerrit-PatchSet: 3
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: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list