Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Ignore blank domains ...................................................................... restapi: Ignore blank domains Currently the the value of the "domain" column of the "vm_init" table can be null or an empty string, both meaning that the VM doesn't have a domain assigned. But when the value of the column is an empty string it will be included in the RESTAPI representation as follows: <domain> <name></name> </domain> This is confusing for callers, and in particular it triggers wrong behaviour in the Python SDK. To avoid this issue this patch changes the RESTAPI so that it won't generate the "domain" element if the domain name is a blank string. Change-Id: Ic2dc13b1dabc08e38def31297148f135ec299ec9 Bug-Url: https://bugzilla.redhat.com/1113571 Signed-off-by: Juan Hernandez <juan.hernandez@redhat.com> --- M backend/manager/modules/restapi/types/pom.xml M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/TemplateMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 3 files changed, 8 insertions(+), 5 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Juan Hernandez: Verified -- To view, visit http://gerrit.ovirt.org/30548 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic2dc13b1dabc08e38def31297148f135ec299ec9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez@redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez@redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi@redhat.com> Gerrit-Reviewer: automation@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server