Bug 1469111 - Total matches of Ems Cluster roles showing wrong count
Summary: Total matches of Ems Cluster roles showing wrong count
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Providers
Version: 5.8.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.10.0
Assignee: Martin Slemr
QA Contact: Yadnyawalk Tale
URL:
Whiteboard: cluster
Depends On:
Blocks: 1560100 1565414
TreeView+ depends on / blocked
 
Reported: 2017-07-10 12:23 UTC by Yadnyawalk Tale
Modified: 2018-08-01 02:49 UTC (History)
4 users (show)

Fixed In Version: 5.10.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1560100 1565414 (view as bug list)
Environment:
Last Closed: 2018-08-01 02:49:15 UTC
Category: ---
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Comment 3 Greg Blomquist 2017-07-13 14:30:31 UTC
Julian, we moved this from User Experience to Providers BZ Component.  Can you check on this?  I'm not sure if the UI is using backend queries to construct this data, or doing something on the front end to select the right data.

If it's all built in the front end, we can have Harpreet look at it.

Comment 5 Martin Slemr 2018-03-19 12:44:18 UTC
Same problem which is solved by this PR:
compare button in:

* Compute -> Infrastructure -> Clusters [reported in this bug]
* Compute -> Infrastructure -> Hosts 
* Compute -> Infrastructure -> Virtual Machines
* Compute -> Clouds -> Instances

Comment 6 Martin Slemr 2018-03-19 17:33:15 UTC
also applicable to:

* Compute -> Clouds -> Availability Zones -> [detail] -> Instances

and Drifts (similar problem, but different code):
* Compute -> Infrastructure -> Clusters -> [detail]
* Compute -> Infrastructure -> Hosts -> [detail]

then button Perform Smart state analysis (at least twice) -> Refresh page and goto Drift history
-> Check 2+ histories and push button Select Up to 10 timestamps for Drift Analysis

Comment 7 CFME Bot 2018-03-21 10:07:41 UTC
New commits detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/4a04f4f93ce4c7000e1c8008976d5b59757cb5ed
commit 4a04f4f93ce4c7000e1c8008976d5b59757cb5ed
Author:     Martin Slemr <mslemr>
AuthorDate: Mon Mar 12 12:01:55 2018 -0400
Commit:     Martin Slemr <mslemr>
CommitDate: Mon Mar 12 12:01:55 2018 -0400

    Repairs compare cluster's counter

    Computes counter in brackets when "same" or "diff" filter used

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

 app/controllers/application_controller/compare.rb | 27 +-
 1 file changed, 25 insertions(+), 2 deletions(-)


https://github.com/ManageIQ/manageiq-ui-classic/commit/aacc5e3908b60bb232f7010b9b9afb66868fbefd
commit aacc5e3908b60bb232f7010b9b9afb66868fbefd
Author:     Martin Slemr <mslemr>
AuthorDate: Wed Mar 14 12:43:34 2018 -0400
Commit:     Martin Slemr <mslemr>
CommitDate: Wed Mar 14 12:43:34 2018 -0400

    Tests compare cluster's counter

    Computes counter in brackets when "same" or "diff" filter used

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

 spec/controllers/application_controller/compare_spec.rb | 139 +
 1 file changed, 139 insertions(+)


https://github.com/ManageIQ/manageiq-ui-classic/commit/aad923b7fa327adcf7e186eed5d87be689b112ce
commit aad923b7fa327adcf7e186eed5d87be689b112ce
Author:     Martin Slemr <mslemr>
AuthorDate: Wed Mar 14 12:53:12 2018 -0400
Commit:     Martin Slemr <mslemr>
CommitDate: Wed Mar 14 12:53:12 2018 -0400

    Rubocop autofix

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

 spec/controllers/application_controller/compare_spec.rb | 15 +-
 1 file changed, 5 insertions(+), 10 deletions(-)


https://github.com/ManageIQ/manageiq-ui-classic/commit/2efe1bf86b1485fcd1537e2428237af24cdbbfa6
commit 2efe1bf86b1485fcd1537e2428237af24cdbbfa6
Author:     Martin Slemr <mslemr>
AuthorDate: Wed Mar 14 13:19:00 2018 -0400
Commit:     Martin Slemr <mslemr>
CommitDate: Wed Mar 14 13:19:00 2018 -0400

    Review refactoring

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

 spec/controllers/application_controller/compare_spec.rb | 154 +-
 1 file changed, 60 insertions(+), 94 deletions(-)


https://github.com/ManageIQ/manageiq-ui-classic/commit/41646df50631fe98ab7d95ad96a67c40eb9a6c62
commit 41646df50631fe98ab7d95ad96a67c40eb9a6c62
Author:     Martin Slemr <mslemr>
AuthorDate: Mon Mar 19 13:46:21 2018 -0400
Commit:     Martin Slemr <mslemr>
CommitDate: Mon Mar 19 13:46:21 2018 -0400

    Fixes drift's total counter

    Like compare's total counters, similar code

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

 app/controllers/application_controller/compare.rb | 124 +-
 spec/controllers/application_controller/compare_spec.rb | 136 +-
 2 files changed, 171 insertions(+), 89 deletions(-)


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