Moti Asayag has uploaded a new change for review. Change subject: engine: Skip empty bridge mappings ......................................................................
engine: Skip empty bridge mappings If the user hasn't provided bridge mappings, there is no need to pass it to ovirt host deploy. Change-Id: Iaf2337d837ddb456f1c6bb70e2aec96394d2a2ea Bug-Url: https://bugzilla.redhat.com/1086789 Signed-off-by: Moti Asayag <masa...@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InstallVdsInternalCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/95/27295/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InstallVdsInternalCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InstallVdsInternalCommand.java index a8f55d5..be9aae0 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InstallVdsInternalCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InstallVdsInternalCommand.java @@ -94,7 +94,7 @@ if (provider.getType() == ProviderType.OPENSTACK_NETWORK) { OpenstackNetworkProviderProperties agentProperties = (OpenstackNetworkProviderProperties) provider.getAdditionalProperties(); - if (parameters.getNetworkMappings() != null) { + if (StringUtils.isNotBlank(parameters.getNetworkMappings())) { agentProperties.getAgentConfiguration().setNetworkMappings( parameters.getNetworkMappings() ); -- To view, visit http://gerrit.ovirt.org/27295 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Iaf2337d837ddb456f1c6bb70e2aec96394d2a2ea Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masa...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
