Bug 1246994 - VM provision dialog shows incorrect cpu count for RHEV CFME templates
Summary: VM provision dialog shows incorrect cpu count for RHEV CFME templates
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Provisioning
Version: 5.4.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.5.0
Assignee: Lucy Fu
QA Contact: Nandini Chandra
URL:
Whiteboard:
Depends On:
Blocks: 1268320
TreeView+ depends on / blocked
 
Reported: 2015-07-27 06:38 UTC by Nandini Chandra
Modified: 2015-12-08 13:24 UTC (History)
7 users (show)

Fixed In Version: 5.5.0.5
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1268320 (view as bug list)
Environment:
Last Closed: 2015-12-08 13:24:05 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
screenshot of RHEV provision dialog (32.44 KB, image/png)
2015-07-27 06:38 UTC, Nandini Chandra
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2015:2551 0 normal SHIPPED_LIVE Moderate: CFME 5.5.0 bug fixes and enhancement update 2015-12-08 17:58:09 UTC

Description Nandini Chandra 2015-07-27 06:38:34 UTC
Created attachment 1056451 [details]
screenshot of RHEV provision dialog

Description of problem:
-----------------------
The following bug addressed the issue where CFME UI listed the cpu count for CFME appliances and templates on RHEV as 1 and not 4.

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

But,the VM provision dialog still shows incorrect cpu count for RHEV CFME templates.


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

How reproducible:


Steps to Reproduce:
-------------------
1.
2.
3.

Actual results:
---------------
VM provisioning dialog shows cpu count for RHEV CFME templates as 1.


Expected results:
-----------------
VM provisioning dialog should show cpu count for RHEV CFME templates as 4.


Additional info:
---------------

Comment 5 CFME Bot 2015-10-02 13:20:30 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/aa7cfb6cb910a8dc03990cf67f7ee02e49ac12f4

commit aa7cfb6cb910a8dc03990cf67f7ee02e49ac12f4
Author:     Lucy Fu <lufu>
AuthorDate: Tue Aug 18 13:29:35 2015 -0400
Commit:     Lucy Fu <lufu>
CommitDate: Thu Oct 1 15:17:05 2015 -0400

    logical_cpus is the variable for total CPUs.
    
    num_cpu is the number of socket.
    https://bugzilla.redhat.com/show_bug.cgi?id=1246994

 app/models/manageiq/providers/infra_manager/provision_workflow.rb     | 4 ++--
 .../manageiq/providers/vmware/infra_manager/provision_workflow.rb     | 4 ++--
 app/views/miq_request/_pre_prov.html.haml                             | 2 +-
 app/views/miq_request/_prov_vm_grid.html.haml                         | 4 ++--
 4 files changed, 7 insertions(+), 7 deletions(-)

Comment 6 CFME Bot 2015-10-02 13:20:35 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/3bef457d972a4753f428fa269658166ce44f6ac5

commit 3bef457d972a4753f428fa269658166ce44f6ac5
Author:     Lucy Fu <lufu>
AuthorDate: Wed Aug 26 19:40:13 2015 -0400
Commit:     Lucy Fu <lufu>
CommitDate: Thu Oct 1 15:17:05 2015 -0400

    More cases where total CPUs is expected.
    
    logical_cpus is the variable for total CPUs.
    num_cpu is the number of socket.
    https://bugzilla.redhat.com/show_bug.cgi?id=1246994

 app/controllers/application_controller/ci_processing.rb       | 2 +-
 app/controllers/application_controller/miq_request_methods.rb | 4 ++--
 app/models/miq_provision_virt_workflow.rb                     | 2 +-
 app/models/mixins/active_vm_aggregation_mixin.rb              | 2 +-
 app/models/mixins/miq_provision_quota_mixin.rb                | 2 +-
 app/models/service/aggregation.rb                             | 4 ++--
 6 files changed, 8 insertions(+), 8 deletions(-)

Comment 7 CFME Bot 2015-10-02 13:20:40 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/bdcbcd79fe1a557a2a46196137cacfe552d61a73

commit bdcbcd79fe1a557a2a46196137cacfe552d61a73
Author:     Lucy Fu <lufu>
AuthorDate: Mon Aug 24 19:20:53 2015 -0400
Commit:     Lucy Fu <lufu>
CommitDate: Thu Oct 1 15:17:05 2015 -0400

    Add test case for ManageIQ::Providers::Vmware::InfraManager::ProvisionWorkflow#get_cpu_values_hash.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1246994

 .../providers/vmware/infra_manager/provision_workflow_spec.rb | 11 +++++++++++
 1 file changed, 11 insertions(+)

Comment 8 CFME Bot 2015-10-02 13:20:45 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/9b8a11841fa7ff1872b6f2b1b5384018bdde3587

commit 9b8a11841fa7ff1872b6f2b1b5384018bdde3587
Author:     Lucy Fu <lufu>
AuthorDate: Wed Aug 26 20:30:49 2015 -0400
Commit:     Lucy Fu <lufu>
CommitDate: Thu Oct 1 15:17:05 2015 -0400

    More cases where total CPUs is expected in db/fixtures, product/reports and spec.
    
    logical_cpus is the variable for total CPUs.
    num_cpu is the number of socket.
    https://bugzilla.redhat.com/show_bug.cgi?id=1246994

 .../ManageIQ/Control/Email.class/__methods__/vm_alert.rb          | 2 +-
 .../Methods.class/__methods__/cluster_workload_management.rb      | 2 +-
 .../150_All Departments with Performance.yaml                     | 8 ++++----
 .../110_Provisioning Activity - by Approver.yaml                  | 4 ++--
 .../120_Provisioning Activity - by Datastore.yaml                 | 4 ++--
 .../130_Provisioning Activity - by Requester.yaml                 | 4 ++--
 .../140_Provisioning Activity - by VM.yaml                        | 4 ++--
 spec/models/miq_group_spec.rb                                     | 8 ++++----
 spec/models/user_spec.rb                                          | 8 ++++----
 9 files changed, 22 insertions(+), 22 deletions(-)

Comment 9 Nandini Chandra 2015-10-28 16:52:41 UTC
Verified in 5.5.0.7

Comment 11 errata-xmlrpc 2015-12-08 13:24:05 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-2015:2551


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