Bug 999393

Summary: Horizon displays two contradictory Messages when a user deletes a non empty Swift container
Product: Red Hat OpenStack Reporter: Nir Magnezi <nmagnezi>
Component: python-django-horizonAssignee: Julie Pichon <jpichon>
Status: CLOSED ERRATA QA Contact: Amit Ugol <augol>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 3.0CC: aortega, athomas, augol, dnavale, jpichon, mrunge, yeylon
Target Milestone: Upstream M1Keywords: Triaged
Target Release: 5.0 (RHEL 7)   
Hardware: Unspecified   
OS: Linux   
Whiteboard:
Fixed In Version: python-django-horizon-2014.1-4.el7ost Doc Type: Bug Fix
Doc Text:
A container that is not empty cannot be deleted. Previously, two contradictory messages, one for success and one for warning that a non-empty container cannot be deleted were displayed when a user tried to delete a non-empty container. With this fix, only the warning message is displayed.
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-07-08 15:42:34 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
screen shot none

Description Nir Magnezi 2013-08-21 09:15:55 UTC
Description of problem:
=======================
Horizon displays two contradictory Messages when a user deletes a non empty swift container.
In addition to that, The Swift container is not really deleted.

Version-Release number of selected component (if applicable):
=============================================================
Grizzly sync Puddle: 2013-08-15.1
python-django-horizon-2013.1.3-1.el6ost.noarch

How reproducible:
=================
Always

Steps to Reproduce:
===================
1. Login and Browse to: http://<HOST_FQDN>/dashboard/project/containers/
2. Create a Swift container via Horizon.
3. Upload an object to that container.
4. Click Delete Container.

Actual results:
===============
Both messages:
Warning: The container cannot be deleted since it's not empty.
Success: Deleted Container: test_container

Expected results:
=================
I'm not sure what is the expected result here, Meaning whether the container should be deleted or not.
According to: https://bugs.launchpad.net/horizon/+bug/1096084/comments/6 , the container content should be deleted recursively.

Comment 1 Nir Magnezi 2013-08-21 09:16:54 UTC
Created attachment 788774 [details]
screen shot

Comment 2 Nir Magnezi 2013-08-21 09:18:59 UTC
jpichon mentioned that this reproduces in Havana

Comment 4 Julie Pichon 2013-08-21 10:49:41 UTC
The bug comment referenced mentions 2 options, and the path followed in the patch was to display a warning message (option #1). We definitely shouldn't have both a warning message and a success message though.

Comment 5 Julie Pichon 2014-01-21 16:07:24 UTC
Committed upstream in Icehouse 1 (commit 52957cc7d9).

Comment 6 Julie Pichon 2014-05-27 09:57:56 UTC
This was merged during upstream Icehouse and should already be available in our packages.

Comment 7 Amit Ugol 2014-06-18 09:07:00 UTC
Verified on python-django-horizon-2014.1-7.el7ost.noarch,
trying to delete a non empty container will only show the not empty error.

Comment 9 errata-xmlrpc 2014-07-08 15:42:34 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHEA-2014-0855.html