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

Change subject: packagers: yum: disable self if not active
......................................................................

packagers: yum: disable self if not active

this breaks offline packager if exists, introduced at:
2b4f0bdccf9 since 2013-16-07, not sure how it passed all QA cycles.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1112370
Change-Id: I2ddd0ac38828bc0536026df465fc17557ac20ce7
Signed-off-by: Alon Bar-Lev <[email protected]>
---
M ChangeLog
M src/plugins/otopi/packagers/yumpackager.py
2 files changed, 10 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/otopi refs/changes/90/29090/1

diff --git a/ChangeLog b/ChangeLog
index 99d6bfa..447eeda 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -3,6 +3,8 @@
 ????-??-?? - Version 1.2.2
 
  * config: warn if config append is missing, rhbz#1092397.
+ * packagers: yum: disable self if other packager installed,
+   rhbz#1112370.
 
 2014-05-02 - Version 1.2.1
 
diff --git a/src/plugins/otopi/packagers/yumpackager.py 
b/src/plugins/otopi/packagers/yumpackager.py
index 13ea591..ae1ed73 100644
--- a/src/plugins/otopi/packagers/yumpackager.py
+++ b/src/plugins/otopi/packagers/yumpackager.py
@@ -208,6 +208,14 @@
 
     @plugin.event(
         stage=plugin.Stages.STAGE_SETUP,
+        priority=plugin.Stages.PRIORITY_HIGH-1,
+        condition=lambda self: self._enabled,
+    )
+    def _setup_existence(self):
+        self._enabled = self.packager == self
+
+    @plugin.event(
+        stage=plugin.Stages.STAGE_SETUP,
         priority=plugin.Stages.PRIORITY_HIGH,
         condition=lambda self: self._enabled,
     )


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2ddd0ac38828bc0536026df465fc17557ac20ce7
Gerrit-PatchSet: 1
Gerrit-Project: otopi
Gerrit-Branch: otopi-1.2
Gerrit-Owner: Alon Bar-Lev <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to