Bug 717538 - Pools delete button not working
Summary: Pools delete button not working
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: CloudForms Cloud Engine
Classification: Retired
Component: aeolus-conductor
Version: 0.3.1
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: beta
Assignee: Tomas Sedovic
QA Contact: wes hayutin
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-06-29 07:19 UTC by Shveta
Modified: 2012-01-26 12:27 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Description Shveta 2011-06-29 07:19:08 UTC 
Description of problem:


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1. Create a pool
2. select the new pool and click on delete/destroy button
3. Two pop -ups come :
  a: Please make a selection before clicking Delete button.
  b. Please select any pool to be deleted before clicking Delete button.
  
Actual results:


Expected results:


Additional info:

rpm -qa|grep aeolus
aeolus-configure-2.0.1-0.fc14.20110628141215gitb8aaf85.noarch
aeolus-conductor-daemons-0.3.0-0.fc14.20110628135944git2a88782.noarch
aeolus-all-0.3.0-0.fc14.20110628135944git2a88782.noarch
aeolus-conductor-doc-0.3.0-0.fc14.20110628135944git2a88782.noarch
rubygem-aeolus-cli-0.0.1-1.fc14.20110628165632git0dfe3ff.noarch
aeolus-conductor-0.3.0-0.fc14.20110628135944git2a88782.noarch
Comment 1 Tomas Sedovic 2011-06-30 14:48:28 UTC 
This has been fixed in next already (works on my current HEAD: e491e0f6e96c85b5c8f46018d4c5aeb0290c979f), but I've found another issue related to deleting pools in the Filter View so I've fixed that one instead :-)

When the user switched to the filter view and wanted to delete pools from the table, they didn't get the delete confirmation. So here's the fix:

https://fedorahosted.org/pipermail/aeolus-devel/2011-June/002786.html
Comment 2 Tomas Sedovic 2011-07-01 13:06:47 UTC 
Pushed in next: f7958022af74429bb253aa3b16be995c08b7fe1f
Comment 3 Shveta 2011-07-04 06:42:00 UTC 
Verified
Comment 4 wes hayutin 2011-07-05 15:04:21 UTC 
removing ce-p2-beta
Comment 5 wes hayutin 2011-07-05 15:08:10 UTC 
second attempt to remove ce-p2-beta
Comment 6 wes hayutin 2011-08-01 19:56:31 UTC 
release pending...
Comment 7 wes hayutin 2011-08-01 19:57:59 UTC 
release pending...
Comment 9 wes hayutin 2011-12-08 13:52:07 UTC 
perm close
Comment 10 wes hayutin 2011-12-08 13:55:00 UTC 
closing out old bugs
Note You need to log in before you can comment on or make changes to this bug.