Francesco Romani has posted comments on this change.
Change subject: lib: move the response.py under common
......................................................................
Patch Set 2: Verified+1
Verified together: 63952 63953 63954
This is a simple code move; I tried patched Vdsm performing various simple VM flows (boot/shutdown/migration back and forth/adding a new VM disk) with no issues in sight.
Also 'make rpm' passed without issues.
I think this is good enough for V+1 on master.
--
To view, visit
https://gerrit.ovirt.org/63954
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I6263fb8991a226516ba5d14e909a4a70ea0078d2
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani
fromani@redhat.com
Gerrit-Reviewer: Francesco Romani
fromani@redhat.com
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer
nsoffer@redhat.com
Gerrit-Reviewer: Piotr Kliczewski
piotr.kliczewski@gmail.com
Gerrit-Reviewer: gerrit-hooks
automation@ovirt.org
Gerrit-HasComments: No