Change in ovirt-engine[master]: core: Return empty list if no VM numa nodes are present

[rgolan at redhat.com]

Roy Golan has submitted this change and it was merged.

Change subject: core: Return empty list if no VM numa nodes are present
......................................................................


core: Return empty list if no VM numa nodes are present

Let AddVmNumaNodesCommand and VM always return an empty list instead of
null if no numa configuration is present to avoid confusing null checks.

Bug-Url: https://bugzilla.redhat.com/1279178
Bug-Url: https://bugzilla.redhat.com/1177259
Change-Id: I095e245227aed5e5c8f6d342d2a8af17b936eb00
Signed-off-by: Roman Mohr <rmohr at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/numa/vm/AbstractVmNumaNodeCommand.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VmNumaNodeOperationParameters.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmBase.java
3 files changed, 12 insertions(+), 8 deletions(-)

Approvals:
  Martin Sivák: Looks good to me, but someone else must approve
  Roman Mohr: Verified
  Jenkins CI: Passed CI tests
  Tomer Saban: Looks good to me, but someone else must approve
  Roy Golan: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/48719
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I095e245227aed5e5c8f6d342d2a8af17b936eb00
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tomer Saban <tsaban at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>