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

Change subject: packaging: setup: display only top level setup packages
......................................................................

packaging: setup: display only top level setup packages

currently we print a message with entire transaction content of modified
packages for setup packages.

better to show only setup packages in price of some performance.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1075043
Change-Id: I3a1f27090aa362882047007416aacff1cb5dbc4c
Signed-off-by: Alon Bar-Lev <alo...@redhat.com>
---
M 
packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-common/distro-rpm/packages.py
1 file changed, 6 insertions(+), 12 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/11/25611/1

diff --git 
a/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-common/distro-rpm/packages.py
 
b/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-common/distro-rpm/packages.py
index 15937ed..dae3432 100644
--- 
a/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-common/distro-rpm/packages.py
+++ 
b/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-common/distro-rpm/packages.py
@@ -192,18 +192,12 @@
             sink=self._getSink(),
             disabledPlugins=('versionlock',),
         )
-        with myum.transaction():
-            myum.update(
-                packages=packages,
-            )
-            if myum.buildTransaction():
-                for p in myum.queryTransaction():
-                    self.logger.debug('PACKAGE: [%s] %s' % (
-                        p['operation'],
-                        p['display_name']
-                    ))
-                    if p['operation'] in ('install', 'update'):
-                        update.append(p['name'])
+        for package in packages:
+            with myum.transaction():
+                myum.update(packages=(package,))
+                if myum.buildTransaction():
+                    if myum.queryTransaction():
+                        update.append(package)
 
         return update
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3a1f27090aa362882047007416aacff1cb5dbc4c
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.4
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