Bug 790861

Summary:	Delete operation of deployable should be changed to remove as it also deletes deployable from other catalogs to which the same deployable was added.
Product:	[Retired] CloudForms Cloud Engine	Reporter:	Rehana <redakkan>
Component:	aeolus-conductor	Assignee:	Jiri Tomasek <jtomasek>
Status:	CLOSED CURRENTRELEASE	QA Contact:	wes hayutin <whayutin>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	1.0.0	CC:	akarol, deltacloud-maint, jtomasek, ssachdev
Target Milestone:	rc
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	v0.8.0-40	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:		Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

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