Francesco Romani has posted comments on this change.
Change subject: stats: report detailed VM down status
......................................................................
Patch Set 12:
I hate to burn a +2, but I had to. Patch Set v12:
* rebased
* fixed a typo which defeated the last change to derive the exitMessage from the exitReason for the common case
* added test to catch the error above
* cleaned out the tests properly using permutations
Verification: verified no regression for creation/shutdown, migration, QEMU killed outside vdsm (kill -KILL...)
--
To view, visit
http://gerrit.ovirt.org/22631
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8d7064fe79d1cd34499fbb32ed0644757cbe05dd
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani
fromani@redhat.com
Gerrit-Reviewer: Allon Mureinik
amureini@redhat.com
Gerrit-Reviewer: Antoni Segura Puimedon
asegurap@redhat.com
Gerrit-Reviewer: Dan Kenigsberg
danken@redhat.com
Gerrit-Reviewer: Federico Simoncelli
fsimonce@redhat.com
Gerrit-Reviewer: Francesco Romani
fromani@redhat.com
Gerrit-Reviewer: Michal Skrivanek
michal.skrivanek@redhat.com
Gerrit-Reviewer: Saggi Mizrahi
smizrahi@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim
ybronhei@redhat.com
Gerrit-Reviewer: automation@ovirt.org
Gerrit-HasComments: No