Change in ovirt-engine[master]: engine: Check if pvsize is not empty-GetDeviceList

Allon Mureinik has submitted this change and it was merged. Change subject: engine: Check if pvsize is not empty-GetDeviceList ...................................................................... engine: Check if pvsize is not empty-GetDeviceList In the GetDeviceListVDSCommand, check if the pvsize entry is not an empty String before trying to parse it into Long. Change-Id: I6d946eddc5f36cdf48fe82adc461b58880d5bdd9 Bug-Url: https://bugzilla.redhat.com/1256027 Signed-off-by: Fred Rolland <frolland@redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/GetDeviceListVDSCommand.java 1 file changed, 9 insertions(+), 5 deletions(-) Approvals: Tal Nisan: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Freddy Rolland: Verified -- To view, visit https://gerrit.ovirt.org/45348 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6d946eddc5f36cdf48fe82adc461b58880d5bdd9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Freddy Rolland <frolland@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Fred Rolland <frolland@redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: automation@ovirt.org
participants (1)
-
amureini@redhat.com