Francesco Romani has posted comments on this change.
Change subject: libvirtconnection: Replace assert with AssertionError
......................................................................
Patch Set 1: Code-Review+1
I agree with the rationale behind this patch, but I'm wondering if we should make use of an even more specific exception or not.
AssertionError could be good enough for starter, hence +1
--
To view, visit
http://gerrit.ovirt.org/34364
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Icf1564f81f4c1fbf77ccaff6d93c047a02d946da
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer
nsoffer@redhat.com
Gerrit-Reviewer: Dan Kenigsberg
danken@redhat.com
Gerrit-Reviewer: Francesco Romani
fromani@redhat.com
Gerrit-Reviewer: Maor Lipchuk
mlipchuk@redhat.com
Gerrit-Reviewer: Saggi Mizrahi
smizrahi@redhat.com
Gerrit-Reviewer: Vinzenz Feenstra
vfeenstr@redhat.com
Gerrit-Reviewer: automation@ovirt.org
Gerrit-HasComments: No