overlogging failed container delete

Bug #1284884 reported by Matthew D. Wood
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Low
Matthew D. Wood

Bug Description

Currently, when a user tries to delete a non-empty container, an exception is raised and logged. There really is no need to log this exception. Instead, the user should be given the error message (which they are today) and the browser should redirect them back to the referring page.

NOTE: Don't reintroduce this bug: https://bugs.launchpad.net/horizon/+bug/1214842

Changed in horizon:
assignee: nobody → Matthew D. Wood (woodm1979)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

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

Changed in horizon:
status: New → In Progress
David Lyle (david-lyle)
Changed in horizon:
importance: Undecided → Low
David Lyle (david-lyle)
Changed in horizon:
milestone: none → juno-2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/76387
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=a650928663f3e11a1dc8b2826a3881b7f4bf5611
Submitter: Jenkins
Branch: master

commit a650928663f3e11a1dc8b2826a3881b7f4bf5611
Author: woodm1979 <email address hidden>
Date: Tue Feb 25 16:01:12 2014 -0700

    1284884 Overlogging of failed container delete

    Currently, when a user tries to delete a non-empty container, an
    exception is raised and logged. There really is no need to log this
    exception. Instead, the user should be given the error message (which
    they are today) and the browser should redirect them back to the
    referring page.

    Change-Id: If18cee000ec5ac1c741b62e72002139dd087e272
    Closes-Bug: 1284884

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