diff --git a/acos_client/tests/unit/v30/test_slb_virtual_port.py b/acos_client/tests/unit/v30/test_slb_virtual_port.py index 231e18ad..52e01edc 100644 --- a/acos_client/tests/unit/v30/test_slb_virtual_port.py +++ b/acos_client/tests/unit/v30/test_slb_virtual_port.py @@ -89,10 +89,10 @@ def test_virtual_port_create_with_params(self): 'ha-conn-mirror': 1, 'no-dest-nat': 1, 'conn-limit': 50000, - 'tcp_template': 'test_tcp_template', + 'template-tcp': 'test_tcp_template', 'template-persist-cookie': 'test_c_pers_template', 'template-persist-source-ip': 'test_s_pers_template', - 'udp_template': 'test_udp_template', + 'template-udp': 'test_udp_template', 'use-rcv-hop-for-resp': 1 } } @@ -191,10 +191,10 @@ def test_virtual_port_create_with_templates(self, mocked_get): 'port-number': 80, 'protocol': 'http', 'service-group': 'pool1', - 'tcp_template': 'test_tcp_template', + 'template-tcp': 'test_tcp_template', 'template-persist-cookie': 'test_c_pers_template', 'template-persist-source-ip': 'test_s_pers_template', - 'udp_template': 'test_udp_template', + 'template-udp': 'test_udp_template', 'template-virtual-port': 'template_vp', 'template-policy': 'template_pl' } @@ -211,10 +211,10 @@ def test_virtual_port_create_with_templates(self, mocked_get): 'port-number': 80, 'protocol': 'http', 'service-group': 'pool1', - 'tcp_template': 'test_tcp_template', + 'template-tcp': 'test_tcp_template', 'template-persist-cookie': 'test_c_pers_template', 'template-persist-source-ip': 'test_s_pers_template', - 'udp_template': 'test_udp_template', + 'template-udp': 'test_udp_template', 'template-virtual-port': 'template_vp', 'template-tcp': 'template_tcp', 'template-policy': 'template_pl' @@ -267,10 +267,10 @@ def test_virtual_port_create_with_partial_templates(self, mocked_get): 'port-number': 80, 'protocol': 'http', 'service-group': 'pool1', - 'tcp_template': 'test_tcp_template', + 'template-tcp': 'test_tcp_template', 'template-persist-cookie': 'test_c_pers_template', 'template-persist-source-ip': 'test_s_pers_template', - 'udp_template': 'test_udp_template', + 'template-udp': 'test_udp_template', 'template-virtual-port': 'template_vp' } } @@ -286,10 +286,10 @@ def test_virtual_port_create_with_partial_templates(self, mocked_get): 'port-number': 80, 'protocol': 'http', 'service-group': 'pool1', - 'tcp_template': 'test_tcp_template', + 'template-tcp': 'test_tcp_template', 'template-persist-cookie': 'test_c_pers_template', 'template-persist-source-ip': 'test_s_pers_template', - 'udp_template': 'test_udp_template', + 'template-udp': 'test_udp_template', 'template-virtual-port': 'template_vp', 'template-tcp': None, 'template-policy': None, @@ -342,10 +342,10 @@ def test_virtual_port_update_with_params(self, mocked_get): 'service-group': 'pool1', 'ha-conn-mirror': 1, 'conn-limit': 50000, - 'tcp_template': 'test_tcp_template', + 'template-tcp': 'test_tcp_template', 'template-persist-cookie': 'test_c_pers_template', 'template-persist-source-ip': 'test_s_pers_template', - 'udp_template': 'test_udp_template', + 'template-udp': 'test_udp_template', 'use-rcv-hop-for-resp': 1, } } @@ -399,10 +399,10 @@ def test_virtual_port_update_with_templates(self, mocked_get): 'service-group': 'pool1', 'ha-conn-mirror': 1, 'conn-limit': 50000, - 'tcp_template': 'test_tcp_template', + 'template-tcp': 'test_tcp_template', 'template-persist-cookie': 'test_c_pers_template', 'template-persist-source-ip': 'test_s_pers_template', - 'udp_template': 'test_udp_template', + 'template-udp': 'test_udp_template', 'template-virtual-port': 'template_vp' } } @@ -422,10 +422,10 @@ def test_virtual_port_update_with_templates(self, mocked_get): 'service-group': 'pool1', 'ha-conn-mirror': 1, 'conn-limit': 50000, - 'tcp_template': 'test_tcp_template', + 'template-tcp': 'test_tcp_template', 'template-persist-cookie': 'test_c_pers_template', 'template-persist-source-ip': 'test_s_pers_template', - 'udp_template': 'test_udp_template', + 'template-udp': 'test_udp_template', 'template-virtual-port': 'template_vp' } } diff --git a/acos_client/v30/slb/virtual_port.py b/acos_client/v30/slb/virtual_port.py index 9113d2e2..cda652b0 100644 --- a/acos_client/v30/slb/virtual_port.py +++ b/acos_client/v30/slb/virtual_port.py @@ -112,7 +112,6 @@ def _set( elif virtual_port_templates.get('template-virtual-port-shared'): params['port']['template-virtual-port-shared'] = virtual_port_templates['template-virtual-port-shared'] params['port']['shared-partition-virtual-port-template'] = True - if protocol in ['http', 'https']: if virtual_port_templates.get('template-http'): params['port']['template-http'] = virtual_port_templates['template-http'] @@ -125,7 +124,6 @@ def _set( elif virtual_port_templates.get('template-tcp-shared'): params['port']['template-tcp-shared'] = virtual_port_templates['template-tcp-shared'] params['port']['shared-partition-tcp'] = True - if virtual_port_templates.get('template-policy'): params['port']['template-policy'] = virtual_port_templates['template-policy'] elif virtual_port_templates.get('template-policy-shared'): @@ -142,9 +140,9 @@ def _set( if source_nat_pool and len(source_nat_pool) > 0: params['port']['pool'] = source_nat_pool if tcp_template: - params['port']['tcp_template'] = tcp_template + params['port']['template-tcp'] = tcp_template if udp_template: - params['port']['udp_template'] = udp_template + params['port']['template-udp'] = udp_template server_ssl_tmpl = kwargs.get("template_server_ssl", None) client_ssl_tmpl = kwargs.get("template_client_ssl")