Ofer Schreiber has uploaded a new change for review.

Change subject: packaging: engine-setup - Don't change selinux context in 
runtime
......................................................................

packaging: engine-setup - Don't change selinux context in runtime

Engine setup use miniyum to perform RPM/YUM based operations, and asks
miniyum to change the selinux context of the whole proccess during it's
intializtion.
This patch removes the selinux context change, as it blocks dbus
messeges issued by firewalld api, and currently not really needed.

Change-Id: I46e04f185c04cc30c8e0b52367d6209716dc0d98
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=904153
Signed-off-by: Ofer Schreiber <[email protected]>
---
M packaging/fedora/setup/engine-setup.py
M packaging/fedora/setup/engine_firewalld.py
2 files changed, 0 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/87/12587/1

diff --git a/packaging/fedora/setup/engine-setup.py 
b/packaging/fedora/setup/engine-setup.py
index 01b539b..b77caac 100755
--- a/packaging/fedora/setup/engine-setup.py
+++ b/packaging/fedora/setup/engine-setup.py
@@ -2121,7 +2121,6 @@
     MiniYum.setup_log_hook(sink=miniyumsink)
     extraLog = open(logFile, "a")
     miniyum = MiniYum(sink=miniyumsink, extraLog=extraLog)
-    miniyum.selinux_role()
     # END: PROCESS-INITIALIZATION
 
     try:
diff --git a/packaging/fedora/setup/engine_firewalld.py 
b/packaging/fedora/setup/engine_firewalld.py
index 4fbcc03..a67768d 100644
--- a/packaging/fedora/setup/engine_firewalld.py
+++ b/packaging/fedora/setup/engine_firewalld.py
@@ -1,5 +1,3 @@
-from gi.repository import GObject
-import sys
 from firewall.client import FirewallClient
 from firewall.errors import *
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I46e04f185c04cc30c8e0b52367d6209716dc0d98
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ofer Schreiber <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to