Dan Kenigsberg has posted comments on this change.

Change subject: virt: add and use vm.acpiReboot() method
......................................................................


Patch Set 6: Code-Review-1

(3 comments)

only one nit, and English fixes.

https://gerrit.ovirt.org/#/c/44923/6//COMMIT_MSG
Commit Message:

Line 13: access
accesses


Line 14: in
impolite


https://gerrit.ovirt.org/#/c/44923/6/tests/vmfakelib.py
File tests/vmfakelib.py:

Line 264: @recorded
unused


-- 
To view, visit https://gerrit.ovirt.org/44923
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5d6dd3a81cd4ad981a839849a4a2914e35a53fa6
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to