Bug 1295941 - Orchestration template : Able to delete template in use
Summary: Orchestration template : Able to delete template in use
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.5.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.6.0
Assignee: Milan Zázrivec
QA Contact: Shveta
URL:
Whiteboard: service:orchestration
: 1295942 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-01-05 20:25 UTC by Shveta
Modified: 2016-06-29 15:25 UTC (History)
6 users (show)

Fixed In Version: 5.6.0.4
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-06-29 15:25:10 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
remove template (166.56 KB, image/png)
2016-01-05 20:25 UTC, Shveta
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2016:1348 0 normal SHIPPED_LIVE CFME 5.6.0 bug fixes and enhancement update 2016-06-29 18:50:04 UTC

Description Shveta 2016-01-05 20:25:06 UTC
Created attachment 1111933 [details]
remove template

Description of problem:


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


How reproducible:


Steps to Reproduce:
1. Create an orchestration template and save 
2. Read -Only flag is set to False 
3. Select template "Remove this template" under Configuration is disabled.

Actual results:


Expected results:


Additional info:
5.5.2.0.20151219144653_653c0d4

Comment 2 Shveta 2016-01-05 22:14:46 UTC
*** Bug 1295942 has been marked as a duplicate of this bug. ***

Comment 3 Shveta 2016-01-05 22:34:15 UTC
Hover text on Remove template option says "Can't delete template in use"
I created a service dialog from that template so may be its working as designed.

But when i select "check All" for templates and delete , the template gets deleted.

Comment 5 Shveta 2016-01-06 22:54:34 UTC
Another similar bug

1. Create an orchestration template
2. "remove this template" option under configuration is greyed out
3. Hover text on the option says "Orchestration template in use cannot be deleted"
4. But the template is not used anywhere.

Comment 7 CFME Bot 2016-01-19 17:31:21 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/d03ddd93d6dc3ed6bbdef94dc02022cd3002ae18

commit d03ddd93d6dc3ed6bbdef94dc02022cd3002ae18
Author:     Bill Wei <bilwei>
AuthorDate: Mon Jan 18 17:54:06 2016 -0500
Commit:     Bill Wei <bilwei>
CommitDate: Tue Jan 19 09:57:51 2016 -0500

    Add before_destroy callback to OrchestrationTemplate
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1295941

 app/models/orchestration_template.rb       | 8 ++++++++
 spec/models/orchestration_template_spec.rb | 7 +++++++
 2 files changed, 15 insertions(+)

Comment 8 Bill Wei 2016-01-19 18:30:30 UTC
I could not reproduce any reported problems in latest ManageIQ master. Everything works as designed. 

The fix in backend is responded to comment #4. This is the last defense to keep a template in use from being deleted. Assign back to UI team to investigate whether there  are reported problems with 5.5.

Shevta, I believe the hover text is static. It does not change according to the readonly flag of a template.

Comment 11 Milan Zázrivec 2016-04-19 08:55:13 UTC
I'm not entirely sure what needs to be done here -- Bill are you perhaps aware of any (even UI)
issues that need to be addressed wrt. orchestration templates?

Thanks.

Comment 12 Bill Wei 2016-04-19 13:16:05 UTC
Milan,
I reported in #8 that I could not reproduce the problem in latest master. Can you reproduce in 5.5? If not, we can close this one.
Please test to delete a single template, as well as multiple selections of templates.

Comment 13 Milan Zázrivec 2016-04-19 14:36:42 UTC
One of the problem still present in current master (and CFME 5.5 as well)
is the misleading tooltip on edit / remove button telling the template
cannot be edited / removed, regardless of whether or not the template
in question is in use.

PR: https://github.com/ManageIQ/manageiq/pull/8090

Comment 14 CFME Bot 2016-04-19 16:21:02 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/40302b168463700ee1c087126de9aad2eba200be

commit 40302b168463700ee1c087126de9aad2eba200be
Author:     Milan Zazrivec <mzazrivec>
AuthorDate: Tue Apr 19 16:32:53 2016 +0200
Commit:     Milan Zazrivec <mzazrivec>
CommitDate: Tue Apr 19 16:32:53 2016 +0200

    Fix misleading tooltip in orchestration template toolbar
    
    Tooltip telling the orchestration template cannot be removed / edited
    needs to be put in place only if the template is in use (not in
    call cases).
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1295941

 .../button/orchestration_template_edit_remove.rb           | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

Comment 15 Shveta 2016-04-22 19:12:52 UTC
5.6.0.4-beta2.3.20160421172650_719e256

Comment 17 errata-xmlrpc 2016-06-29 15:25:10 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.

https://access.redhat.com/errata/RHBA-2016:1348


Note You need to log in before you can comment on or make changes to this bug.