port_security_enabled is used even when the extension is not available

Bug #1638487 reported by YAMAMOTO Takashi on 2016-11-02
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
tempest
High
Unassigned

Bug Description

recent changes in tempest and devstack makes it uses port security attributes
even if it isn't available.
it broke deployments without the extension.

eg. midonet v1 jobs
http://logs.openstack.org/25/392425/1/check/gate-tempest-dsvm-networking-midonet-v1/ffeb1c2/

Traceback (most recent call last):
  File "tempest/test.py", line 100, in wrapper
    return f(self, *func_args, **func_kwargs)
  File "tempest/scenario/test_network_advanced_server_ops.py", line 140, in test_server_connectivity_pause_unpause
    server = self._setup_server(keypair)
  File "tempest/scenario/test_network_advanced_server_ops.py", line 57, in _setup_server
    network, subnet, router = self.create_networks()
  File "tempest/scenario/manager.py", line 1189, in create_networks
    port_security_enabled=port_security_enabled)
  File "tempest/scenario/manager.py", line 686, in _create_network
    result = networks_client.create_network(**network_kwargs)
  File "tempest/lib/services/network/networks_client.py", line 27, in create_network
    return self.create_resource(uri, post_data)
  File "tempest/lib/services/network/base.py", line 60, in create_resource
    resp, body = self.post(req_uri, req_post_data)
  File "tempest/lib/common/rest_client.py", line 276, in post
    return self.request('POST', url, extra_headers, headers, body, chunked)
  File "tempest/lib/common/rest_client.py", line 664, in request
    self._error_checker(resp, resp_body)
  File "tempest/lib/common/rest_client.py", line 766, in _error_checker
    raise exceptions.BadRequest(resp_body, resp=resp)
tempest.lib.exceptions.BadRequest: Bad request
Details: {u'type': u'HTTPBadRequest', u'detail': u'', u'message': u"Unrecognized attribute(s) 'port_security_enabled'"}

Changed in tempest:
assignee: nobody → YAMAMOTO Takashi (yamamoto)
status: New → In Progress

Fix proposed to branch: master
Review: https://review.openstack.org/393618

tags: added: gate-failure
Changed in tempest:
importance: Undecided → High

Change abandoned by YAMAMOTO Takashi (<email address hidden>) on branch: master
Review: https://review.openstack.org/393618

Changed in tempest:
assignee: YAMAMOTO Takashi (yamamoto) → nobody
status: In Progress → New
Changed in tempest:
status: New → Confirmed
Changed in tempest:
status: Confirmed → In Progress
Martin Kopec (mkopec) wrote :

Moving back to Confirmed as nobody is working on it.

Changed in tempest:
status: In Progress → Confirmed
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers