Juan Hernandez has uploaded a new change for review. Change subject: packaging: Fix ownership of deployments and contents ......................................................................
packaging: Fix ownership of deployments and contents A previous commit (9d5af3) changed the .spec so that the backups directory is owned by the setup sub package. This is needed, but we also need to make sure that the deployments and contents directories are still owned by the backend package, otherwise the directories are not created and the service will not start. Change-Id: Ibb4745cfc9898b43a164781fd5866cf2b1fa4f82 Bug-Url: https://bugzilla.redhat.com/876164 Signed-off-by: Juan Hernandez <juan.hernan...@redhat.com> --- M packaging/fedora/spec/ovirt-engine.spec.in 1 file changed, 10 insertions(+), 4 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/69/9369/1 diff --git a/packaging/fedora/spec/ovirt-engine.spec.in b/packaging/fedora/spec/ovirt-engine.spec.in index f98aa42..94d15f8 100644 --- a/packaging/fedora/spec/ovirt-engine.spec.in +++ b/packaging/fedora/spec/ovirt-engine.spec.in @@ -688,10 +688,15 @@ %{_mavenpomdir}/JPP.%{name}-vdsbroker.pom %endif -# Directories for the backup, deployments and contents files need to be -# owned by the engine user because the setup/upgrade wizard or applications -# server need to write to them: +# The state directory is owned by the backend, but not recursively, as +# some of its subdirectories are owned by other packages (backups, for +# example, has to be owned by the setup package): %dir %attr(-, %{engine_user}, %{engine_group}) %{engine_state} + +# Directories for the deployments and content files need to be owned by +# the engine user because the application server needs to write to them: +%attr(-, %{engine_user}, %{engine_group}) %{engine_state}/content +%attr(-, %{engine_user}, %{engine_group}) %{engine_state}/deployments # The backend owns the directory of the ear and most of its contents, # but not all, as most of the web applications are owned by other @@ -732,6 +737,7 @@ # Modules: %{engine_jboss_modules} + %files setup # Links to the main programs: @@ -764,7 +770,7 @@ %dir %{engine_data}/scripts/plugins %{engine_data}/scripts/plugins/example_plugin_000.py* -# Backups directory +# Backups directory: %dir %{engine_state}/backups %files setup-plugin-allinone -- To view, visit http://gerrit.ovirt.org/9369 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ibb4745cfc9898b43a164781fd5866cf2b1fa4f82 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernan...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
