Log image_id with all BadStoreURI error messages

Bug #1243704 reported by Venkatesh Sampath
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Fix Released
Wishlist
Venkatesh Sampath

Bug Description

In the following migration scripts, not all BadStoreUri error messages are logged with image_id. Hence it might become difficult to track the record which has got BadStoreUri. Including the image_id with the error log messages would help in tracking the faulty records easily when BadStoreUri exception is raised.

015_quote_swift_credentials.py
017_quote_encrypted_swift_credentials.py

Changed in glance:
assignee: nobody → Venkatesh Sampath (venkateshsampath)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to glance (master)

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

Changed in glance:
importance: Undecided → Wishlist
milestone: none → icehouse-1
Thierry Carrez (ttx)
Changed in glance:
milestone: icehouse-1 → icehouse-2
Changed in glance:
milestone: icehouse-2 → icehouse-3
Thierry Carrez (ttx)
Changed in glance:
milestone: icehouse-3 → icehouse-rc1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to glance (master)

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

commit c0e1d5bd2944a011c0cbaba61d284aedec086b60
Author: Venkatesh Sampath <email address hidden>
Date: Fri Feb 21 19:33:40 2014 +0530

    Log 'image_id' with all BadStoreURI error messages

    - handle all BadStoreUri exceptions raised by 'legacy_parse_uri'
      function in
    one location so that it becomes easier to log the error message with
    'image_id'
    - add tests for 'legacy_parse_uri'

    Closes-Bug: #1243704

    Change-Id: Ifc5de11832860ed51c1eb359d6f5cf78de8c0ba4

Changed in glance:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in glance:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in glance:
milestone: icehouse-rc1 → 2014.1
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.