BadStoreUri doesn't get the expected arguments in several places

Bug #1287506 reported by Arnaud Legendre
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Fix Released
Low
Unassigned

Bug Description

BadStoreUri is not used correctly in several places.
For example, in the Cinder store:
https://github.com/openstack/glance/blob/master/glance/store/cinder.py#L120
raise exception.BadStoreUri(uri, reason)

where BadStoreUri expects:
https://github.com/openstack/glance/blob/master/glance/common/exception.py#L39

Changed in glance:
assignee: nobody → Arnaud Legendre (arnaudleg)
Zhi Yan Liu (lzy-dev)
Changed in glance:
status: New → Triaged
importance: Undecided → Low
Revision history for this message
Arnaud Legendre (arnaudleg) wrote :
Changed in glance:
assignee: Arnaud Legendre (arnaudleg) → nobody
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to glance (master)

Reviewed: https://review.openstack.org/95816
Committed: https://git.openstack.org/cgit/openstack/glance/commit/?id=97882f796c0e8969c606ae723d14b6b443e2e2f9
Submitter: Jenkins
Branch: master

commit 97882f796c0e8969c606ae723d14b6b443e2e2f9
Author: Erno Kuvaja <email address hidden>
Date: Fri Jun 6 14:43:57 2014 +0000

    Unifies how BadStoreUri gets raised and logged

    Refactoring store code to raise BadStoreUri in unified way.

    Changes BadStoreUri messages being logged as info from debug.

    Fixes translations accordingly.

    Ensures no URI's nor credentials logged with these messages.

    Closes bug #1287506

    Change-Id: I246885d68b0af9e91ede48aaa2b1e6849e5b3b3d

Changed in glance:
status: Triaged → Fix Committed
Changed in glance:
milestone: none → juno-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in glance:
milestone: juno-2 → 2014.2
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.