Bug 1707871 - Unable to assign alert profile to "Selected" anything
Summary: Unable to assign alert profile to "Selected" anything
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.11.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.11.0
Assignee: Harpreet Kataria
QA Contact: John Dupuy
Red Hat CloudForms Documentation
URL:
Whiteboard:
Depends On:
Blocks: 1658670
TreeView+ depends on / blocked
 
Reported: 2019-05-08 15:16 UTC by John Dupuy
Modified: 2019-12-13 15:16 UTC (History)
8 users (show)

Fixed In Version: 5.11.0.4
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-12-13 15:16:44 UTC
Category: ---
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
no "Selections" have loaded (24.75 KB, image/png)
2019-05-08 15:16 UTC, John Dupuy
no flags Details
how this looks in cfme 5.11.0.2 (22.25 KB, image/png)
2019-05-08 15:17 UTC, John Dupuy
no flags Details

Description John Dupuy 2019-05-08 15:16:17 UTC
Created attachment 1565703 [details]
no "Selections" have loaded

Description of problem:
Unable to assign an alert profile to something that requires a selection, e.g. "Selected Servers", because the "Selection" does not load. This is a regression BZ, it worked in CFME 5.11.0.2.

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

How reproducible:
100%

Steps to Reproduce:
1. Make a Server alert
2. Make a Server alert profile and assign the Server alert to it
3. Edit assignments for the alert-profile, select "Selected Servers"

Actual results:
Nothing loads from which you can select a server (see screenshot)

Expected results:
The "Selections" widget should load

Additional info:
This appears to be affecting anything that requires a selection, e.g. "Selected Clusters" for a VM and Instance alert.

Comment 2 John Dupuy 2019-05-08 15:17:35 UTC
Created attachment 1565704 [details]
how this looks in cfme 5.11.0.2

Comment 5 CFME Bot 2019-05-09 09:01:03 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/295ac4c9c1eff7fe14f2721981d52664d2d90d0f
commit 295ac4c9c1eff7fe14f2721981d52664d2d90d0f
Author:     Harpreet Kataria <hkataria>
AuthorDate: Wed May  8 16:21:26 2019 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Wed May  8 16:21:26 2019 -0400

    Removed 'type' argument passed to the tree builder

    Fixed 2 issues
    - Fixed `wrong number of arguments (given 4, expected 3)` error, removed 'type' argument that was being passed in to tree builder code, was introduced by https://github.com/ManageIQ/manageiq-ui-classic/pull/5495
    - Fixed `comparison of Integer with String failed` error for `@assign[:new][:objects].sort!`, changed to push integer into objects array to fix comparison error on the sort when selecting nodes in the tree in 'Selections' box.

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1707871
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1707886

 app/controllers/miq_policy_controller/alert_profiles.rb | 13 +-
 spec/controllers/miq_policy_controller/alert_profiles_spec.rb | 25 +
 2 files changed, 31 insertions(+), 7 deletions(-)

Comment 6 John Dupuy 2019-05-15 18:32:33 UTC
Verified in CFME 5.11.0.4.20190514210444_0c91ee1

Steps of verification:
1. Make a Server alert
2. Make a Server alert profile and assign the Server alert to it
3. Edit assignments for the alert-profile, select "Selected Servers"
4. Verified that the selections displayed properly and that you can select and save the "EVM" server


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