Change in ovirt-engine[master]: core: adding HostJobInfo/VDSError toString()

From Liron Aravot <laravot@redhat.com>:
Liron Aravot has submitted this change and it was merged. Change subject: core: adding HostJobInfo/VDSError toString() ...................................................................... core: adding HostJobInfo/VDSError toString() This change adds the toString() to the HostJobInfo/VDSError classes. HostJobInfo represents the current status of a job executed by vdsm - returned when executing the GetHostJobsVDSCommand. The results of the execution are logged - adding toString() will make it readable. HostJobInfo has VDSError member, therefore a toString() is being added to it as well. before: FINISH, GetHostJobsVDSCommand, return: {a4c19e06-df55-4ca0-9d7e-3a12fcf59929= org.ovirt.engine.core.common.businessentities.HostJobInfo@3657bbdd} after: FINISH, GetHostJobsVDSCommand, return: {fcd3813b-6783-442e-8659-ab6b2b11879b= HostJobInfo:{id='fcd3813b-6783-442e-8659-ab6b2b11879b', type='storage', description='move_device', status='failed', progress='null', error='VDSError:{code='UnsupportedOperationErr', message='Operation not supported'}'}} Change-Id: I75c1033f573bec4719a8f40f7f0408217c4fbab6 Signed-off-by: Liron Aravot <laravot@redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/HostJobInfo.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VDSError.java 2 files changed, 22 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Freddy Rolland: Looks good to me, but someone else must approve Liron Aravot: Verified -- To view, visit https://gerrit.ovirt.org/68405 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I75c1033f573bec4719a8f40f7f0408217c4fbab6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>
participants (1)
-
Code Review