Change in ovirt-engine[master]: core: change return value of copyImageVDSCommand for better ...

[ofrenkel at redhat.com]

Omer Frenkel has submitted this change and it was merged.

Change subject: core: change return value of copyImageVDSCommand for better logging
......................................................................


core: change return value of copyImageVDSCommand for better logging

currently the return value of copyImageVDSCommand is not initialized,
and is defaulted with emtpy guid.
this empty guid appears in the engine.log as the result of the command,
which is useless and confusing.

Changed the return value to have the vdsm task id,
which is a better guid to log there.

note that the returnVaule of this command is not in use at all,
only the createInfo, so this affect only the log.

Change-Id: I80d830dcdcdfa38a76b08e5399b2f916349b9b8e
Signed-off-by: Omer Frenkel <ofrenkel at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/CopyImageVDSCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Omer Frenkel: Verified
  Maor Lipchuk: Looks good to me, but someone else must approve
  Arik Hadas: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/29609
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I80d830dcdcdfa38a76b08e5399b2f916349b9b8e
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server