Bug 773261

Summary: Adding/Deleting a catalog in deployable page displays "deployable added/deleted"
Product: [Retired] CloudForms Cloud Engine Reporter: Aziza Karol <akarol>
Component: aeolus-conductorAssignee: Tzu-Mainn Chen <tzumainn>
Status: CLOSED ERRATA QA Contact: wes hayutin <whayutin>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 1.0.0CC: akarol, athomas, deltacloud-maint, slinaber, ssachdev
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-05-15 21:35:39 UTC Type: ---
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
delpoyable
none
add_deployable_message
none
deployable_removed_message none

Description Aziza Karol 2012-01-11 10:52:46 UTC
Description of problem:


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


How reproducible:


Steps to Reproduce:
1.In the deployable page,in the catalog section click on add+ button and try adding a catalog or deleting a catalog.

It displays deployable added/deployable deleted respectively.
see attach screenshot.


Expected results:
should display catalog added/catalog deleted resp.

Additional info:
 rpm -qa | grep aeolus
aeolus-conductor-doc-0.8.0-5.el6.noarch
rubygem-aeolus-image-0.3.0-2.el6.noarch
rubygem-aeolus-cli-0.3.0-3.el6.noarch
aeolus-all-0.8.0-5.el6.noarch
aeolus-conductor-0.8.0-5.el6.noarch
aeolus-configure-2.5.0-3.el6.noarch
aeolus-conductor-daemons-0.8.0-5.el6.noarch

Comment 1 Aziza Karol 2012-01-11 10:55:12 UTC
Created attachment 552077 [details]
delpoyable

Comment 2 wes hayutin 2012-01-12 16:33:04 UTC
adding to ce-sprint

Comment 3 wes hayutin 2012-01-12 16:39:42 UTC
removing ce-sprint-next tracker

Comment 4 Angus Thomas 2012-01-13 14:37:49 UTC
Unable to reproduce in aeolus-conductor-0.8.0-5.el6

Comment 5 wes hayutin 2012-01-16 13:31:12 UTC
recreated in [root@qeblade32 yum.repos.d]# rpm -qa | grep aeolus
aeolus-conductor-doc-0.8.0-5.el6.noarch
rubygem-aeolus-cli-0.3.0-3.el6.noarch
rubygem-aeolus-image-0.3.0-2.el6.noarch
aeolus-all-0.8.0-5.el6.noarch
aeolus-configure-2.5.0-4.el6.noarch
aeolus-conductor-0.8.0-5.el6.noarch
aeolus-conductor-daemons-0.8.0-5.el6.noarch

https://qeblade32.rhq.lab.eng.bos.redhat.com/conductor/catalogs/1/deployables/1

From the deployable page.. add/remove a catalog from deployable.
The message is correct.. but could be more clear..

Please Change the message from deployable added/removed

TO:

deployable added/removed from catalog $catalog_name

The page is busy enough to warrant an more clear message

Moving back to Angus

Comment 6 Tzu-Mainn Chen 2012-01-24 18:36:15 UTC
Patch submitted:

https://fedorahosted.org/pipermail/aeolus-devel/2012-January/008324.html

Comment 7 Tzu-Mainn Chen 2012-01-25 20:48:24 UTC
Updated patch set:

https://fedorahosted.org/pipermail/aeolus-devel/2012-January/008362.html

Comment 8 Tzu-Mainn Chen 2012-01-25 22:35:24 UTC
Pushed to 1.0-staging:

commit 53a538242e382a4d0ecdb895be944b0a42fcdbd6
BZ 773261 added catalog parameter to catalog_entries.flash.notice.added

commit 8e99c7368adee1461b1df85063f05fa85f7935ff
BZ 773261 updated catalog/deployable interaction messages to be clearer

Comment 9 Steve Linabery 2012-01-26 16:47:54 UTC
53a5382  8e99c73 in aeolus-conductor-0.8.0-12

Comment 12 Shveta 2012-01-31 13:29:50 UTC
Created attachment 558620 [details]
add_deployable_message

Comment 13 Shveta 2012-01-31 13:30:36 UTC
Created attachment 558621 [details]
deployable_removed_message

Comment 14 Shveta 2012-01-31 13:31:35 UTC
Appropriate messages are displayed now
verified in 
rpm -qa|grep aeolus
aeolus-conductor-doc-0.8.0-16.el6.noarch
aeolus-configure-2.5.0-11.el6.noarch
aeolus-conductor-daemons-0.8.0-16.el6.noarch
rubygem-aeolus-image-0.3.0-6.el6.noarch
aeolus-all-0.8.0-16.el6.noarch
aeolus-conductor-0.8.0-16.el6.noarch
rubygem-aeolus-cli-0.3.0-7.el6.noarch

Comment 15 errata-xmlrpc 2012-05-15 21:35:39 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-2012-0583.html