Bug 1519473

Summary:	VMs on SCVMM report cores per socket and number of sockets incorrectly
Product:	Red Hat CloudForms Management Engine	Reporter:	Satoe Imaishi <simaishi>
Component:	Providers	Assignee:	Adam Grare <agrare>
Status:	CLOSED ERRATA	QA Contact:	Niyaz Akhtar Ansari <nansari>
Severity:	medium	Docs Contact:
Priority:	high
Version:	5.8.0	CC:	cpelland, dajohnso, fdewaley, gblomqui, jfrey, jhardy, obarenbo, rwolters
Target Milestone:	GA
Target Release:	5.9.0
Hardware:	All
OS:	All
Whiteboard:
Fixed In Version:	5.9.0.12	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:	1514463	Environment:
Last Closed:	2018-03-01 13:21:45 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:	1514463
Bug Blocks:

Comment 2 CFME Bot 2017-11-30 18:33:42 UTC

New commit detected on ManageIQ/manageiq-ui-classic/gaprindashvili:
https://github.com/ManageIQ/manageiq-ui-classic/commit/c1d3725f7e4d25f417c53d4971a3a452282803b2

commit c1d3725f7e4d25f417c53d4971a3a452282803b2
Author:     Harpreet Kataria <hkataria>
AuthorDate: Wed Nov 29 16:54:21 2017 -0500
Commit:     Satoe Imaishi <simaishi>
CommitDate: Thu Nov 30 13:20:37 2017 -0500

    Merge pull request #2873 from agrare/fix_textual_devices_cpu_description
    
    Only show sockets and cores_per_socket if present
    (cherry picked from commit 92418a76eb5d3e5a95e10e799d73138ffed643a4)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1519473

 app/helpers/textual_mixins/textual_devices.rb | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

Comment 3 CFME Bot 2017-12-01 13:55:29 UTC

New commit detected on ManageIQ/manageiq-providers-scvmm/gaprindashvili:
https://github.com/ManageIQ/manageiq-providers-scvmm/commit/06e7d5671d252815f2d688ae2df113e1624a1ba4

commit 06e7d5671d252815f2d688ae2df113e1624a1ba4
Author:     Daniel Berger <djberg96>
AuthorDate: Wed Nov 29 10:31:42 2017 -0700
Commit:     Satoe Imaishi <simaishi>
CommitDate: Fri Dec 1 08:49:21 2017 -0500

    Merge pull request #51 from agrare/bz_1514463_dont_report_cpu_sockets_or_cores_per_socket
    
    Don't report misleading sockets/cores
    (cherry picked from commit bfc5ea32a7f9e986b2ca222f5e90c0fa60870cb0)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1519473

 app/models/manageiq/providers/microsoft/infra_manager/refresh_parser.rb | 2 --
 .../models/manageiq/providers/microsoft/infra_manager/refresher_spec.rb | 2 --
 2 files changed, 4 deletions(-)

Comment 4 CFME Bot 2017-12-01 14:28:43 UTC

New commit detected on ManageIQ/manageiq-ui-classic/gaprindashvili:
https://github.com/ManageIQ/manageiq-ui-classic/commit/8b954742cdbc633a6c52e24a42f3bf89cac22ece

commit 8b954742cdbc633a6c52e24a42f3bf89cac22ece
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Thu Nov 30 14:34:37 2017 +0100
Commit:     Satoe Imaishi <simaishi>
CommitDate: Fri Dec 1 09:25:28 2017 -0500

    Merge pull request #2872 from agrare/fix_vm_cpu_count_textual_summary
    
    Fix CPU summary for VMs without cores_per_socket
    (cherry picked from commit 07e40e5afebc3972763e556567e8cd1efb147813)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1519473

 app/helpers/vm_helper/textual_summary.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 5 Niyaz Akhtar Ansari 2017-12-13 06:02:49 UTC

Verified in 5.9.0.12

Comment 8 errata-xmlrpc 2018-03-01 13:21:45 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/RHSA-2018:0380