Reviewed: https://review.openstack.org/308560 Committed: https://git.openstack.org/cgit/openstack/neutron-lbaas/commit/?id=9356005b68a0dd1d1e9853b3933017694bdd473e Submitter: Jenkins Branch: master
commit 9356005b68a0dd1d1e9853b3933017694bdd473e Author: Trevor McCasland <tm2...@att.com> Date: Fri Jul 1 08:33:42 2016 -0500 Enable removal of SSL/TLS reference on listener To remove TLS, a person would update the listener with default_tls_container_ref="" or None and the expected result is for the default_tls_container_ref to be empty but instead it was filled with the current value in the database. This change fixes the if statements that was responsible for this bug. Change-Id: I9c8610998743090297ec6cbc76d766be5c13d624 Closes-Bug: #1569536 ** Changed in: neutron Status: In Progress => Fix Released -- You received this bug notification because you are a member of Yahoo! Engineering Team, which is subscribed to neutron. https://bugs.launchpad.net/bugs/1569536 Title: Neutron-LBaaS v2: Unable to remove SSL/TLS Barbican container reference on listener Status in neutron: Fix Released Bug description: 1. Create TLS Listener (i.e., create listener with valid "default_tls_container_ref" pointed to barbican) 2. Remove TLS by updating Listener with default_tls_container_ref="" or None Result: default_tls_container_ref still contains the original reference Expected: default_tls_container_ref should be empty Log: 2016-04-12 18:33:17,808 22307 INFO [tempest.lib.common.rest_client] Request (TLSListenersTestJSON:test_update_listener_empty_tls_container): 200 PUT http://127.0.0.1:9696/v2.0/lbaas/listeners/91beb010-1ca6-4969-9cd5-93bb2af0ff34 0.948s 2016-04-12 18:33:17,808 22307 DEBUG [tempest.lib.common.rest_client] Request - Headers: {'Content-Type': 'application/json', 'Accept': 'application/json', 'X-Auth-Token': '<omitted>'} Body: {"listener": {"default_tls_container_ref": ""}} Response - Headers: {'status': '200', 'content-length': '465', 'connection': 'close', 'date': 'Tue, 12 Apr 2016 18:33:17 GMT', 'content-type': 'application/json; charset=UTF-8', 'x-openstack-request-id': 'req-e877249b-7bd4-4434-a0d5-325e4b097fdf'} Body: {"listener": {"protocol_port": 443, "protocol": "TERMINATED_HTTPS", "description": "", "default_tls_container_ref": "http://127.0.0.1:9311/v1/containers/4a54ee55-14a1-45ef-8412-03c0d5c6a6b4", "admin_state_up": true, "loadbalancers": [ {"id": "01816a46-d4f5-452b-969c-3852c1716076"} ], "tenant_id": "521684a26d974acea5299a9b9f8d068d", "sni_container_refs": [], "connection_limit": -1, "default_pool_id": null, "id": "91beb010-1ca6-4969-9cd5-93bb2af0ff34", "name": ""}} full log: http://paste.openstack.org/show/493865/ To manage notifications about this bug go to: https://bugs.launchpad.net/neutron/+bug/1569536/+subscriptions -- Mailing list: https://launchpad.net/~yahoo-eng-team Post to : yahoo-eng-team@lists.launchpad.net Unsubscribe : https://launchpad.net/~yahoo-eng-team More help : https://help.launchpad.net/ListHelp