Bug 790861 - Delete operation of deployable should be changed to remove as it also deletes deployable from other catalogs to which the same deployable was added.
Summary: Delete operation of deployable should be changed to remove as it also deletes...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: CloudForms Cloud Engine
Classification: Retired
Component: aeolus-conductor
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: rc
Assignee: Jiri Tomasek
QA Contact: wes hayutin
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-02-15 15:23 UTC by Rehana
Modified: 2012-08-30 17:18 UTC (History)
4 users (show)

Fixed In Version: v0.8.0-40
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:

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

Description Rehana 2012-02-15 15:23:10 UTC 
Description of problem:


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


How reproducible:


Steps to Reproduce:
1. create a "Application Blueprint from outline" to two different catalog at the same time
2. The deployable are created in two selected catalog
3. From one of the catalog, select the deployable and delete
4. Verify the second catalog
  
Actual results:

The deployable got deleted from both the catalogs

Expected results:

The code should remove only from one catalog,and delete is expected to happen only if it's the last catalog the deployable belongs to

Additional info:

[root@dell-pe860-01 aeolus-conductor]# rpm -qa |grep aeolus
aeolus-conductor-0.8.0-26.el6.noarch
aeolus-conductor-doc-0.8.0-26.el6.noarch
aeolus-configure-2.5.0-13.el6.noarch
aeolus-all-0.8.0-26.el6.noarch
rubygem-aeolus-image-0.3.0-7.el6.noarch
rubygem-aeolus-cli-0.3.0-8.el6.noarch
aeolus-conductor-daemons-0.8.0-26.el6.noarch
Comment 1 wes hayutin 2012-02-16 17:28:18 UTC 
Angus, what are your thoughts?
Comment 2 Jiri Tomasek 2012-02-22 14:38:58 UTC 
I think deleting deployable from deployable show page should delete deployable (with all its catalog entries - references to catalogs). But when deleting from catalog/show page (multi destroy), just catalog entry should be deleted if there are any other catalogs related to deployable. Othervise if deployable is in only this catalog, it gets deleted.
Comment 3 Jiri Tomasek 2012-02-23 14:18:42 UTC 
fix posted in https://fedorahosted.org/pipermail/aeolus-devel/2012-February/009124.html
Comment 4 Jiri Tomasek 2012-03-01 14:32:30 UTC 
rebased and resent https://fedorahosted.org/pipermail/aeolus-devel/2012-March/009309.html
Comment 5 Jiri Tomasek 2012-03-01 15:38:00 UTC 
fixed in commit 27d8490c626a6107f559af4bd0e92ce37b17388f
Comment 6 Shveta 2012-03-05 06:36:16 UTC 
Created a BP in two catalogs together .
Deleted BP from catalog page , it gets deleted from that particular catalog only.


============================

Created another BP in two catalogs and deleted it from deployable details page.
It got deleted from both catalogs

=====================================
Verified as mentioned in comment 2

rpm -qa|grep aeolus
rubygem-aeolus-cli-0.3.0-12.el6.noarch
aeolus-conductor-doc-0.8.0-40.el6.noarch
aeolus-conductor-daemons-0.8.0-40.el6.noarch
rubygem-aeolus-image-0.3.0-12.el6.noarch
aeolus-configure-2.5.0-17.el6.noarch
aeolus-conductor-0.8.0-40.el6.noarch
aeolus-all-0.8.0-40.el6.noarch
Note You need to log in before you can comment on or make changes to this bug.