[Engine-patches] Change in ovirt-engine[master]: packaging: upgrade: print yum transaction details

2012-12-30 Thread oschreib
Ofer Schreiber has posted comments on this change.

Change subject: packaging: upgrade: print yum transaction details
..


Patch Set 1:

I see only logging of current transaction, not printing it to the user, is that 
in purpose?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3fb61734f054637e05f4e2658f7b918f057547a6
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Alex Lourie alou...@redhat.com
Gerrit-Reviewer: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Juan Hernandez juan.hernan...@redhat.com
Gerrit-Reviewer: Moran Goldboim mgold...@redhat.com
Gerrit-Reviewer: Ofer Schreiber oschr...@redhat.com
___
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches


[Engine-patches] Change in ovirt-engine[master]: packaging: upgrade: print yum transaction details

2012-12-30 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change.

Change subject: packaging: upgrade: print yum transaction details
..


Patch Set 1:

 I see only logging of current transaction, not printing it to the user, is 
 that in purpose?

Yes.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3fb61734f054637e05f4e2658f7b918f057547a6
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Alex Lourie alou...@redhat.com
Gerrit-Reviewer: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Juan Hernandez juan.hernan...@redhat.com
Gerrit-Reviewer: Moran Goldboim mgold...@redhat.com
Gerrit-Reviewer: Ofer Schreiber oschr...@redhat.com
___
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches


[Engine-patches] Change in ovirt-engine[master]: packaging: upgrade: print yum transaction details

2012-12-30 Thread oschreib
Ofer Schreiber has posted comments on this change.

Change subject: packaging: upgrade: print yum transaction details
..


Patch Set 2: Verified; Looks good to me, approved

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3fb61734f054637e05f4e2658f7b918f057547a6
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Alex Lourie alou...@redhat.com
Gerrit-Reviewer: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Juan Hernandez juan.hernan...@redhat.com
Gerrit-Reviewer: Moran Goldboim mgold...@redhat.com
Gerrit-Reviewer: Ofer Schreiber oschr...@redhat.com
___
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches


[Engine-patches] Change in ovirt-engine[master]: packaging: upgrade: print yum transaction details

2012-12-30 Thread oschreib
Ofer Schreiber has submitted this change and it was merged.

Change subject: packaging: upgrade: print yum transaction details
..


packaging: upgrade: print yum transaction details

Change-Id: I3fb61734f054637e05f4e2658f7b918f057547a6
Signed-off-by: Alon Bar-Lev alo...@redhat.com
---
M packaging/fedora/setup/engine-upgrade.py
1 file changed, 7 insertions(+), 0 deletions(-)

Approvals:
  Ofer Schreiber: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3fb61734f054637e05f4e2658f7b918f057547a6
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Alex Lourie alou...@redhat.com
Gerrit-Reviewer: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Juan Hernandez juan.hernan...@redhat.com
Gerrit-Reviewer: Moran Goldboim mgold...@redhat.com
Gerrit-Reviewer: Ofer Schreiber oschr...@redhat.com
___
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches


[Engine-patches] Change in ovirt-engine[master]: packaging: upgrade: print yum transaction details

2012-12-23 Thread alourie
Alex Lourie has posted comments on this change.

Change subject: packaging: upgrade: print yum transaction details
..


Patch Set 1: Looks good to me, but someone else must approve

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3fb61734f054637e05f4e2658f7b918f057547a6
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Alex Lourie alou...@redhat.com
Gerrit-Reviewer: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Juan Hernandez juan.hernan...@redhat.com
Gerrit-Reviewer: Moran Goldboim mgold...@redhat.com
Gerrit-Reviewer: Ofer Schreiber oschr...@redhat.com
___
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches


[Engine-patches] Change in ovirt-engine[master]: packaging: upgrade: print yum transaction details

2012-12-21 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review.

Change subject: packaging: upgrade: print yum transaction details
..

packaging: upgrade: print yum transaction details

Change-Id: I3fb61734f054637e05f4e2658f7b918f057547a6
Signed-off-by: Alon Bar-Lev alo...@redhat.com
---
M packaging/fedora/setup/engine-upgrade.py
1 file changed, 7 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/09/10309/1

diff --git a/packaging/fedora/setup/engine-upgrade.py 
b/packaging/fedora/setup/engine-upgrade.py
index cdc5c22..ed218c8 100755
--- a/packaging/fedora/setup/engine-upgrade.py
+++ b/packaging/fedora/setup/engine-upgrade.py
@@ -317,6 +317,13 @@
 self._miniyum.update([basedefs.ENGINE_RPM_NAME])
 self.emptyTransaction = not self._miniyum.buildTransaction()
 
+logging.debug('Transaction Summary:')
+for p in self._miniyum.queryTransaction():
+logging.debug('%s - %s' % (
+p['operation'],
+p['display_name']
+))
+
 def update(self):
 self._rpmChange = True
 self._miniyum.processTransaction()


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3fb61734f054637e05f4e2658f7b918f057547a6
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


[Engine-patches] Change in ovirt-engine[master]: packaging: upgrade: print yum transaction details

2012-12-21 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change.

Change subject: packaging: upgrade: print yum transaction details
..


Patch Set 1: Verified

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3fb61734f054637e05f4e2658f7b918f057547a6
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Alex Lourie alou...@redhat.com
Gerrit-Reviewer: Alon Bar-Lev alo...@redhat.com
Gerrit-Reviewer: Juan Hernandez juan.hernan...@redhat.com
Gerrit-Reviewer: Moran Goldboim mgold...@redhat.com
Gerrit-Reviewer: Ofer Schreiber oschr...@redhat.com
___
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches