Hello Dan Kenigsberg, Milan Zamazal, I'd like you to do a code review. Please visit
https://gerrit.ovirt.org/49673 to review the following change. Change subject: vm: reformat _EVENT_STRINGS ...................................................................... vm: reformat _EVENT_STRINGS Reformat code to make it easy to extend the known event list. Change-Id: I6e8d577ebcee9e72cf70aab214f9ff42d46fcb9a Format-Only: yes Bug-Url: https://bugzilla.redhat.com/1284564 Signed-off-by: Francesco Romani <from...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/48530 Continuous-Integration: Jenkins CI Reviewed-by: Dan Kenigsberg <dan...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/48789 Reviewed-by: Milan Zamazal <mzama...@redhat.com> --- M vdsm/virt/vm.py 1 file changed, 10 insertions(+), 8 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/73/49673/1 diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py index ba4940c..8bc545a 100644 --- a/vdsm/virt/vm.py +++ b/vdsm/virt/vm.py @@ -184,14 +184,16 @@ # These strings are representing libvirt virDomainEventType values # http://libvirt.org/html/libvirt-libvirt.html#virDomainEventType -_EVENT_STRINGS = ("Defined", - "Undefined", - "Started", - "Suspended", - "Resumed", - "Stopped", - "Shutdown", - "PM-Suspended") +_EVENT_STRINGS = ( + "Defined", + "Undefined", + "Started", + "Suspended", + "Resumed", + "Stopped", + "Shutdown", + "PM-Suspended", +) def eventToString(event): -- To view, visit https://gerrit.ovirt.org/49673 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I6e8d577ebcee9e72cf70aab214f9ff42d46fcb9a Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Milan Zamazal <mzama...@redhat.com> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches