[node-patches] Change in ovirt-node[master]: Allow nested transactions

fabiand at fedoraproject.org fabiand at fedoraproject.org
Tue Feb 25 17:16:27 UTC 2014


Fabian Deutsch has submitted this change and it was merged.

Change subject: Allow nested transactions
......................................................................


Allow nested transactions

Allow Transactions to be nested as transaction elements.
Implementation didn't require the consumer pattern, as the code
was already structured to allow with minimal changes.

Furthermore the inheritance from list has been dropped.

Change-Id: Ia74e1dff22827f0b5fb47ba57eaa72c304998474
Signed-off-by: Ryan Barry <rbarry at redhat.com>
---
M src/ovirt/node/utils/__init__.py
1 file changed, 59 insertions(+), 14 deletions(-)

Approvals:
  Fabian Deutsch: Verified; Looks good to me, approved
  Ryan Barry: Looks good to me, but someone else must approve

Objections:
  oVirt Jenkins CI Server: I would prefer that you didn't submit this



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia74e1dff22827f0b5fb47ba57eaa72c304998474
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the node-patches mailing list