From 5abb1aa49d93a9bfba1270fa7356e787d520cf28 Mon Sep 17 00:00:00 2001 From: garciadeblas Date: Mon, 8 Nov 2021 10:30:56 +0100 Subject: [PATCH] Fix bug 1736 to assign floating IP addresses only when use_floating_ip set to true Change-Id: I469dbfdf5a52dce4631adc268e3f5d664ed77abe Signed-off-by: garciadeblas --- RO-VIM-gcp/osm_rovim_gcp/vimconn_gcp.py | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/RO-VIM-gcp/osm_rovim_gcp/vimconn_gcp.py b/RO-VIM-gcp/osm_rovim_gcp/vimconn_gcp.py index 73b77bab..ecfdb068 100644 --- a/RO-VIM-gcp/osm_rovim_gcp/vimconn_gcp.py +++ b/RO-VIM-gcp/osm_rovim_gcp/vimconn_gcp.py @@ -926,12 +926,12 @@ class vimconnector(vimconn.VimConnector): ] = "regions/%s/subnetworks/" % self.region + net.get("name") else: net_iface["subnetwork"] = net.get("net_id") - # According to documentation "type" can be only ONE_TO_ONE_NAT and the recomended value for "name" is "External NAT", - # so an external IP will be generated for the instance - # TODO: check if it's possible to allow internet access to the instance with no external IP - net_iface["accessConfigs"] = [ - {"type": "ONE_TO_ONE_NAT", "name": "External NAT"} - ] + # In order to get an external IP address, the key "accessConfigs" must be used + # in the interace. It has to be of type "ONE_TO_ONE_NAT" and name "External NAT" + if net.get("floating_ip", False) or (net["use"] == "mgmt" and self.config.get("use_floating_ip")): + net_iface["accessConfigs"] = [ + {"type": "ONE_TO_ONE_NAT", "name": "External NAT"} + ] network_interfaces.append(net_iface) -- 2.25.1