yahoo-eng-team team mailing list archive

[Eugene Nikanorov <enikanorov@xxxxxxxxxxxx>]

This has been raised multiple times in the past.
Neutron doesn't verify tenant_id against keystone. It's assumed that regular user calls neutron being authenticated.
And admin presumably know what he is doing.

I don't think we need to do anything with this issue, especially in the
scope of such bugs.

** Changed in: neutron
       Status: New => Opinion

** Tags added: lbaas

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1440705

Title:
  Creating listeners  with invalid/empty tenantid is not throwing error

Status in OpenStack Neutron (virtual network service):
  Opinion

Bug description:
  Creating listeners  with invalid/empty tenant is successful (with logging_noop driver).It should throw error during validation.
  Following is tempest test (with logging_noop driver backend) logs:


  neutron_lbaas.tests.tempest.v2.api.test_listeners.ListenersTestJSON.test_create_listener_invalid_empty_tenant_id[negative]
  --------------------------------------------------------------------------------------------------------------------------

  Captured traceback:
  ~~~~~~~~~~~~~~~~~~~
      Traceback (most recent call last):
        File "neutron_lbaas/tests/tempest/v2/api/test_listeners.py", line 89, in test_create_listener_invalid_empty_tenant_id
          tenant_id="")
        File "/opt/stack/neutron-lbaas/.tox/tempest/local/lib/python2.7/site-packages/testtools/testcase.py", line 422, in assertRaises
          self.assertThat(our_callable, matcher)
        File "/opt/stack/neutron-lbaas/.tox/tempest/local/lib/python2.7/site-packages/testtools/testcase.py", line 435, in assertThat
          raise mismatch_error
      testtools.matchers._impl.MismatchError: <bound method type._create_listener of <class 'neutron_lbaas.tests.tempest.v2.api.test_listeners.ListenersTestJSON'>> returned {u'default_pool_id': None, u'protocol': u'HTTP', u'description': u'', u'sni_container_ids': [], u'admin_state_up': True, u'loadbalancers': [{u'id': u'c57117f3-9675-440e-b41a-d3c51bfb1719'}], u'tenant_id': u'', u'default_tls_container_id': None, u'connection_limit': -1, u'protocol_port': 8081, u'id': u'18d515d2-0dcb-4306-8dd8-c45ae1e068a6', u'name': u''}

  
  neutron_lbaas.tests.tempest.v2.api.test_listeners.ListenersTestJSON.test_create_listener_invalid_tenant_id[negative]
  --------------------------------------------------------------------------------------------------------------------

  Captured traceback:
  ~~~~~~~~~~~~~~~~~~~
      Traceback (most recent call last):
        File "neutron_lbaas/tests/tempest/v2/api/test_listeners.py", line 76, in test_create_listener_invalid_tenant_id
          tenant_id="&^%123")
        File "/opt/stack/neutron-lbaas/.tox/tempest/local/lib/python2.7/site-packages/testtools/testcase.py", line 422, in assertRaises
          self.assertThat(our_callable, matcher)
        File "/opt/stack/neutron-lbaas/.tox/tempest/local/lib/python2.7/site-packages/testtools/testcase.py", line 435, in assertThat
          raise mismatch_error
      testtools.matchers._impl.MismatchError: <bound method type._create_listener of <class 'neutron_lbaas.tests.tempest.v2.api.test_listeners.ListenersTestJSON'>> returned {u'default_pool_id': None, u'protocol': u'HTTP', u'description': u'', u'sni_container_ids': [], u'admin_state_up': True, u'loadbalancers': [{u'id': u'c57117f3-9675-440e-b41a-d3c51bfb1719'}], u'tenant_id': u'&^%123', u'default_tls_container_id': None, u'connection_limit': -1, u'protocol_port': 8080, u'id': u'e648907e-758c-44be-be9e-9249adab1071', u'name': u''}

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1440705/+subscriptions