Alon Bar-Lev has uploaded a new change for review.

Change subject: packaging: /etc/ovirt-engine should not be owned by engine
......................................................................

packaging: /etc/ovirt-engine should not be owned by engine

engine does not and should not write into it.

Change-Id: Ic24afe853410a56bd27dd613764bb8dd8b18e1c3
Signed-off-by: Alon Bar-Lev <alo...@redhat.com>
---
M packaging/fedora/engine-service.py.in
M packaging/fedora/spec/ovirt-engine.spec.in
2 files changed, 1 insertion(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/12/13412/1

diff --git a/packaging/fedora/engine-service.py.in 
b/packaging/fedora/engine-service.py.in
index 7a992bb..08422eb 100644
--- a/packaging/fedora/engine-service.py.in
+++ b/packaging/fedora/engine-service.py.in
@@ -328,7 +328,6 @@
     checkFile(jbossModulesJar)
 
     # Check the required engine directories and files:
-    checkDirectory(engineEtcDir, uid=engineUid, gid=engineGid)
     checkDirectory(engineLogDir, uid=engineUid, gid=engineGid)
     checkDirectory(engineUsrDir)
     checkDirectory(engineVarDir, uid=engineUid, gid=engineGid)
diff --git a/packaging/fedora/spec/ovirt-engine.spec.in 
b/packaging/fedora/spec/ovirt-engine.spec.in
index 91228ce..d9c9ca9 100644
--- a/packaging/fedora/spec/ovirt-engine.spec.in
+++ b/packaging/fedora/spec/ovirt-engine.spec.in
@@ -423,7 +423,7 @@
 %endif
 
 # Engine configuration files:
-%dir %attr(-, %{engine_user}, %{engine_group}) %{engine_etc}
+%dir %{engine_etc}
 %config(noreplace) %attr(-, %{engine_user}, %{engine_group}) 
%{engine_etc}/engine.conf
 %dir %attr(-, %{engine_user}, %{engine_group}) %{engine_etc}/engine.conf.d
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic24afe853410a56bd27dd613764bb8dd8b18e1c3
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to