Bug 1686225 - Remove the deprecated "Discover Cloud Providers" option from CloudForms UI
Summary: Remove the deprecated "Discover Cloud Providers" option from CloudForms UI
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.10.1
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: GA
: 5.11.0
Assignee: Greg Piatigorski
QA Contact: Parthvi Vala
Red Hat CloudForms Documentation
URL:
Whiteboard:
Depends On:
Blocks: 1692488 1695626
TreeView+ depends on / blocked
 
Reported: 2019-03-07 01:06 UTC by Nikhil Gupta
Modified: 2019-12-13 15:08 UTC (History)
9 users (show)

Fixed In Version: 5.11.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1695626 (view as bug list)
Environment:
Last Closed: 2019-12-13 15:08:43 UTC
Category: Feature
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Knowledge Base (Solution) 3967641 0 Supportability None The Discover Cloud Providers option is disabled in CloudForms UI. 2019-03-07 01:32:14 UTC

Description Nikhil Gupta 2019-03-07 01:06:31 UTC
Description of problem:
Remove the "Discover Cloud Providers" option from CloudForms UI.
The feature "Discover Cloud Providers" was deprecated in CloudForms 4.6 and the only supported way of adding a cloud provider is by using the "Add a New Cloud Provider" option.


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

How reproducible:
Always

Steps to Reproduce:
1. Go to Compute -> Clouds -> Providers
2. Click on Configuration

Actual results:
"Discover Cloud Providers" option is disabled.

Expected results:
"Discover Cloud Providers" option should not be there as it is deprecated.

Comment 7 Greg Piatigorski 2019-03-27 22:46:03 UTC
Yes, testing code removal and should be creating the PR later tonight or tomorrow. Just want to make sure I do not break anything. It will be multi-PR since both base ad UI sides have changes.

Comment 9 Greg Piatigorski 2019-03-28 20:29:31 UTC
Code refactoring requires code refactoring for both base and classic UI branches. In the base branch one needs to run MiqShortcut.seed as part of testing/implementation. 

Classic UI: https://github.com/ManageIQ/manageiq-ui-classic/pull/5395

Comment 10 Greg Piatigorski 2019-03-28 20:36:32 UTC
Base branch PR: https://github.com/ManageIQ/manageiq/pull/18607

Comment 11 CFME Bot 2019-04-02 16:50:44 UTC
New commit detected on ManageIQ/manageiq/master:

https://github.com/ManageIQ/manageiq/commit/b1ecb04ca1d2bf1be63ab75c55a731f44e146b81
commit b1ecb04ca1d2bf1be63ab75c55a731f44e146b81
Author:     Greg Piatigorski <gpiatigo>
AuthorDate: Thu Mar 28 16:31:03 2019 -0400
Commit:     Greg Piatigorski <gpiatigo>
CommitDate: Thu Mar 28 16:31:03 2019 -0400

    Remove Discover Cloud Providers option in toolbar drop-down

    https://bugzilla.redhat.com/show_bug.cgi?id=1686225

 db/fixtures/miq_product_features.yml | 4 -
 locale/manageiq.pot | 5 -
 2 files changed, 9 deletions(-)

Comment 12 CFME Bot 2019-04-02 16:51:35 UTC
New commits detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/954ba4738e49411141574f7eab29a5f6df7ff4c9
commit 954ba4738e49411141574f7eab29a5f6df7ff4c9
Author:     Greg Piatigorski <gpiatigo>
AuthorDate: Thu Mar 28 16:07:37 2019 -0400
Commit:     Greg Piatigorski <gpiatigo>
CommitDate: Thu Mar 28 16:07:37 2019 -0400

    Remove deprecated Discover Cloud Providers drop-down selection

    https://bugzilla.redhat.com/show_bug.cgi?id=1686225

 app/controllers/mixins/actions/host_actions/discover.rb | 15 +-
 app/helpers/application_helper/toolbar/ems_clouds_center.rb | 8 -
 locale/ManageIQ_UI_Classic.pot | 4 -
 spec/controllers/ems_cloud_controller_spec.rb | 6 -
 4 files changed, 2 insertions(+), 31 deletions(-)


https://github.com/ManageIQ/manageiq-ui-classic/commit/57b0d056b24afa343d215405c03a9b0829826e70
commit 57b0d056b24afa343d215405c03a9b0829826e70
Author:     Greg Piatigorski <gpiatigo>
AuthorDate: Mon Apr  1 15:34:52 2019 -0400
Commit:     Greg Piatigorski <gpiatigo>
CommitDate: Mon Apr  1 15:34:52 2019 -0400

    Remove deprecated Discover Cloud Providers

    Additional cleanup to remove all remnants of ems_cloud reference/checks

    https://bugzilla.redhat.com/show_bug.cgi?id=1686225

 app/controllers/mixins/actions/host_actions/discover.rb | 61 +-
 spec/controllers/application_controller/ci_processing_spec.rb | 15 -
 2 files changed, 16 insertions(+), 60 deletions(-)

Comment 14 Parthvi Vala 2019-05-02 15:44:14 UTC
FIXED. Verified on 5.11.0.2.20190430174828_0e34dea.


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