Francesco Romani has posted comments on this change.

Change subject: lib: move define.py under common
......................................................................


Patch Set 1: 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/63953
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I634d3bac718fe0615a2aca951ec47b976967ada1
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to