errors in clear_isolated_creds are not covered with suppressing

Bug #1408584 reported by Valeriy Ponomaryov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Shared File Systems Service (Manila)
Fix Released
High
Valeriy Ponomaryov

Bug Description

If some error appearing releasing isolated creds we get following errors:

ft4.1: tearDownClass (tempest.api.share.admin.test_share_servers.ShareServersAdminTest)_StringException: Traceback (most recent call last):
  File "tempest/api/share/base.py", line 179, in tearDownClass
    super(BaseSharesTest, cls).tearDownClass()
  File "tempest/test.py", line 302, in tearDownClass
    teardown()
  File "tempest/api/share/base.py", line 440, in clear_isolated_creds
    ic["method"]()
  File "tempest/common/isolated_creds.py", line 341, in clear_isolated_creds
    self._clear_isolated_net_resources()
  File "tempest/common/isolated_creds.py", line 330, in _clear_isolated_net_resources
    self._clear_isolated_router(router['id'], router['name'])
  File "tempest/common/isolated_creds.py", line 282, in _clear_isolated_router
    net_client.delete_router(router_id)
  File "tempest/services/network/network_client_base.py", line 125, in _delete
    resp, body = self.delete(uri)
  File "tempest/services/network/network_client_base.py", line 84, in delete
    return self.rest_client.delete(uri, headers)
  File "tempest/common/rest_client.py", line 242, in delete
    return self.request('DELETE', url, extra_headers, headers, body)
  File "tempest/common/rest_client.py", line 450, in request
    resp, resp_body)
  File "tempest/common/rest_client.py", line 503, in _error_checker
    raise exceptions.Conflict(resp_body)
Conflict: An object with that identifier already exists
Details: {u'detail': u'', u'type': u'RouterInUse', u'message': u'Router fa200a2f-65c9-4133-b53d-296f8e61c2e8 still has ports'}

It is unrelated to Manila tests directly. So, need add possibility to suppress such errors.

Tags: tempest
Changed in manila:
assignee: nobody → Valeriy Ponomaryov (vponomaryov)
milestone: none → kilo-2
importance: Undecided → High
tags: added: tempest
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to manila (master)

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

Changed in manila:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to manila (master)

Reviewed: https://review.openstack.org/145782
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=5705f12de2fafd06083beca3f5df974070d7b76a
Submitter: Jenkins
Branch: master

commit 5705f12de2fafd06083beca3f5df974070d7b76a
Author: vponomaryov <email address hidden>
Date: Thu Jan 8 15:52:28 2015 +0200

    Add error suppressing to isolated creds cleanup in Tempest plugin

    For the moment we have error suppressing only for cleanup of Manila resources.
    Add error suppressing to cleanup of isolated creds resources too.
    It is useful for case of CI Tempest jobs, where we do not verify proper release
    of not Manila's resources and do not want to fail on cleanup.

    Change-Id: I44db8e22e9794c94c4dc04faf6a954ab9396f96f
    Closes-Bug: #1408584

Changed in manila:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in manila:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in manila:
milestone: kilo-2 → 2015.1.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.