Bug 1240337 - Smart state analysis fails on EC2 instances with undefined method ` + 'for nil:NilClass "
Summary: Smart state analysis fails on EC2 instances with undefined method ` + 'for ni...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.4.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: GA
: 5.5.0
Assignee: Harpreet Kataria
QA Contact: Taras Lehinevych
URL:
Whiteboard:
Depends On:
Blocks: 1245717
TreeView+ depends on / blocked
 
Reported: 2015-07-06 15:40 UTC by ldomb
Modified: 2016-08-03 01:32 UTC (History)
8 users (show)

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


Attachments (Terms of Use)


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 ldomb 2015-07-06 15:40:13 UTC
Description of problem:

The User is able to perform a smart state analysis on an EC2 instance. The functionality is not implemented yet. My VM analysis show's scanning than fails with undefined method ` + 'for nil:NilClass "


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


How reproducible:


Steps to Reproduce:
1. Add an ec2 provider
2. Click on an instance
3. launch configuration perform smart state analysis

Actual results:

Task fails with undefined method ` + 'for nil:NilClass "



Expected results:

Smart state analysis for ec2 hosts works as I am able to launch it. 


Additional info:

Comment 2 Dave Johnson 2015-07-14 13:05:33 UTC
0709 bug triage decided to make this H/H

Comment 5 CFME Bot 2015-08-18 02:26:36 UTC
New commit detected on manageiq/master:
https://github.com/ManageIQ/manageiq/commit/981632f8d77a7a0b5319d303db029fae01d5521f

commit 981632f8d77a7a0b5319d303db029fae01d5521f
Author:     Harpreet Kataria <hkataria>
AuthorDate: Tue Aug 11 12:28:44 2015 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Tue Aug 11 16:38:20 2015 -0400

    Added model support to determine if Smartstate Analysis is supported.
    
    - Added model support to determine if Smartstate Analysis button is supported for VM/Instances, added a check in toolbar code to display a hover message when button is disabled and not supported for certain types of VM/Instance.
    - Fixed hover text on the toolbar button, title should say "Perform SmartState Analysis on this Instance" instead of "Perform SmartState Analysis on this Image"
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1240337
    https://bugzilla.redhat.com/show_bug.cgi?id=1245717

 app/helpers/application_helper/toolbar_builder.rb           |  1 +
 app/models/manageiq/providers/amazon/cloud_manager/vm.rb    |  4 ++++
 app/models/manageiq/providers/cloud_manager/vm.rb           |  4 ++++
 app/models/manageiq/providers/openstack/cloud_manager/vm.rb |  4 ++++
 app/models/manageiq/providers/redhat/infra_manager/vm.rb    |  4 ++++
 app/models/manageiq/providers/vmware/infra_manager/vm.rb    |  4 ++++
 product/toolbars/x_vm_cloud_center_tb.yaml                  |  4 ++--
 spec/helpers/application_helper/toolbar_builder_spec.rb     | 11 +++++++++++
 spec/models/vm_or_template_spec.rb                          | 12 ++++++++++++
 9 files changed, 46 insertions(+), 2 deletions(-)

Comment 6 CFME Bot 2015-08-18 22:07:19 UTC
New commit detected on cfme/5.4.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=2a557dfa43c0fa2a6263c5830a4290b38c24d06d

commit 2a557dfa43c0fa2a6263c5830a4290b38c24d06d
Author:     Harpreet Kataria <hkataria>
AuthorDate: Mon Aug 17 11:38:56 2015 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Tue Aug 18 10:49:23 2015 -0400

    Added model support to determine if Smartstate Analysis is supported.
    
    - Added model support to determine if Smartstate Analysis button is supported for VM/Instances, added a check in toolbar code to display a hover message when button is disabled and not supported for certain types of VM/Instance.
    - Fixed hover text on the toolbar button, title should say "Perform SmartState Analysis on this Instance" instead of "Perform SmartState Analysis on this Image"
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1240337
    https://bugzilla.redhat.com/show_bug.cgi?id=1245717

 vmdb/app/helpers/application_helper.rb          |  1 +
 vmdb/app/models/vm.rb                           |  4 ++++
 vmdb/app/models/vm_amazon.rb                    |  4 ++++
 vmdb/app/models/vm_cloud.rb                     |  4 ++++
 vmdb/app/models/vm_openstack.rb                 |  4 ++++
 vmdb/app/models/vm_redhat.rb                    |  4 ++++
 vmdb/app/models/vm_vmware.rb                    |  4 ++++
 vmdb/product/toolbars/x_vm_cloud_center_tb.yaml |  4 ++--
 vmdb/spec/helpers/application_helper_spec.rb    | 11 +++++++++++
 vmdb/spec/models/vm_or_template_spec.rb         | 12 ++++++++++++
 10 files changed, 50 insertions(+), 2 deletions(-)

Comment 7 CFME Bot 2015-08-18 22:08:10 UTC
New commit detected on cfme/5.4.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=598799c30b3ee2a0f63409fcdb36ba7ffccc66f8

commit 598799c30b3ee2a0f63409fcdb36ba7ffccc66f8
Merge: 5ae8c86 2a557df
Author:     Dan Clarizio <dclarizi>
AuthorDate: Tue Aug 18 12:33:59 2015 -0400
Commit:     Dan Clarizio <dclarizi>
CommitDate: Tue Aug 18 12:33:59 2015 -0400

    Merge branch '54z_bz_1245717' into '5.4.z'
    
    Added model support to determine if Smartstate Analysis is supported.
    
    - Added model support to determine if Smartstate Analysis button is supported for VM/Instances, added a check in toolbar code to display a hover message when button is disabled and not supported for certain types of VM/Instance.
    - Fixed hover text on the toolbar button, title should say "Perform SmartState Analysis on this Instance" instead of "Perform SmartState Analysis on this Image"
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1240337
    https://bugzilla.redhat.com/show_bug.cgi?id=1245717
    
    @dclarizio please review, had to make some changes by hand, VM related models & toolbar builder code have been moved/renamed on upstream.
    original PR: https://github.com/ManageIQ/manageiq/pull/3808
    
    See merge request !225

 vmdb/app/helpers/application_helper.rb          |  1 +
 vmdb/app/models/vm.rb                           |  4 ++++
 vmdb/app/models/vm_amazon.rb                    |  4 ++++
 vmdb/app/models/vm_cloud.rb                     |  4 ++++
 vmdb/app/models/vm_openstack.rb                 |  4 ++++
 vmdb/app/models/vm_redhat.rb                    |  4 ++++
 vmdb/app/models/vm_vmware.rb                    |  4 ++++
 vmdb/product/toolbars/x_vm_cloud_center_tb.yaml |  4 ++--
 vmdb/spec/helpers/application_helper_spec.rb    | 11 +++++++++++
 vmdb/spec/models/vm_or_template_spec.rb         | 12 ++++++++++++
 10 files changed, 50 insertions(+), 2 deletions(-)

Comment 8 CFME Bot 2015-08-18 22:08:40 UTC
New commit detected on cfme/5.4.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=2a557dfa43c0fa2a6263c5830a4290b38c24d06d

commit 2a557dfa43c0fa2a6263c5830a4290b38c24d06d
Author:     Harpreet Kataria <hkataria>
AuthorDate: Mon Aug 17 11:38:56 2015 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Tue Aug 18 10:49:23 2015 -0400

    Added model support to determine if Smartstate Analysis is supported.
    
    - Added model support to determine if Smartstate Analysis button is supported for VM/Instances, added a check in toolbar code to display a hover message when button is disabled and not supported for certain types of VM/Instance.
    - Fixed hover text on the toolbar button, title should say "Perform SmartState Analysis on this Instance" instead of "Perform SmartState Analysis on this Image"
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1240337
    https://bugzilla.redhat.com/show_bug.cgi?id=1245717

 vmdb/app/helpers/application_helper.rb          |  1 +
 vmdb/app/models/vm.rb                           |  4 ++++
 vmdb/app/models/vm_amazon.rb                    |  4 ++++
 vmdb/app/models/vm_cloud.rb                     |  4 ++++
 vmdb/app/models/vm_openstack.rb                 |  4 ++++
 vmdb/app/models/vm_redhat.rb                    |  4 ++++
 vmdb/app/models/vm_vmware.rb                    |  4 ++++
 vmdb/product/toolbars/x_vm_cloud_center_tb.yaml |  4 ++--
 vmdb/spec/helpers/application_helper_spec.rb    | 11 +++++++++++
 vmdb/spec/models/vm_or_template_spec.rb         | 12 ++++++++++++
 10 files changed, 50 insertions(+), 2 deletions(-)

Comment 9 CFME Bot 2015-08-18 22:08:53 UTC
New commit detected on cfme/5.4.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=598799c30b3ee2a0f63409fcdb36ba7ffccc66f8

commit 598799c30b3ee2a0f63409fcdb36ba7ffccc66f8
Merge: 5ae8c86 2a557df
Author:     Dan Clarizio <dclarizi>
AuthorDate: Tue Aug 18 12:33:59 2015 -0400
Commit:     Dan Clarizio <dclarizi>
CommitDate: Tue Aug 18 12:33:59 2015 -0400

    Merge branch '54z_bz_1245717' into '5.4.z'
    
    Added model support to determine if Smartstate Analysis is supported.
    
    - Added model support to determine if Smartstate Analysis button is supported for VM/Instances, added a check in toolbar code to display a hover message when button is disabled and not supported for certain types of VM/Instance.
    - Fixed hover text on the toolbar button, title should say "Perform SmartState Analysis on this Instance" instead of "Perform SmartState Analysis on this Image"
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1240337
    https://bugzilla.redhat.com/show_bug.cgi?id=1245717
    
    @dclarizio please review, had to make some changes by hand, VM related models & toolbar builder code have been moved/renamed on upstream.
    original PR: https://github.com/ManageIQ/manageiq/pull/3808
    
    See merge request !225

 vmdb/app/helpers/application_helper.rb          |  1 +
 vmdb/app/models/vm.rb                           |  4 ++++
 vmdb/app/models/vm_amazon.rb                    |  4 ++++
 vmdb/app/models/vm_cloud.rb                     |  4 ++++
 vmdb/app/models/vm_openstack.rb                 |  4 ++++
 vmdb/app/models/vm_redhat.rb                    |  4 ++++
 vmdb/app/models/vm_vmware.rb                    |  4 ++++
 vmdb/product/toolbars/x_vm_cloud_center_tb.yaml |  4 ++--
 vmdb/spec/helpers/application_helper_spec.rb    | 11 +++++++++++
 vmdb/spec/models/vm_or_template_spec.rb         | 12 ++++++++++++
 10 files changed, 50 insertions(+), 2 deletions(-)

Comment 10 CFME Bot 2015-08-18 22:10:07 UTC
New commit detected on cfme/5.4.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=2a557dfa43c0fa2a6263c5830a4290b38c24d06d

commit 2a557dfa43c0fa2a6263c5830a4290b38c24d06d
Author:     Harpreet Kataria <hkataria>
AuthorDate: Mon Aug 17 11:38:56 2015 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Tue Aug 18 10:49:23 2015 -0400

    Added model support to determine if Smartstate Analysis is supported.
    
    - Added model support to determine if Smartstate Analysis button is supported for VM/Instances, added a check in toolbar code to display a hover message when button is disabled and not supported for certain types of VM/Instance.
    - Fixed hover text on the toolbar button, title should say "Perform SmartState Analysis on this Instance" instead of "Perform SmartState Analysis on this Image"
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1240337
    https://bugzilla.redhat.com/show_bug.cgi?id=1245717

 vmdb/app/helpers/application_helper.rb          |  1 +
 vmdb/app/models/vm.rb                           |  4 ++++
 vmdb/app/models/vm_amazon.rb                    |  4 ++++
 vmdb/app/models/vm_cloud.rb                     |  4 ++++
 vmdb/app/models/vm_openstack.rb                 |  4 ++++
 vmdb/app/models/vm_redhat.rb                    |  4 ++++
 vmdb/app/models/vm_vmware.rb                    |  4 ++++
 vmdb/product/toolbars/x_vm_cloud_center_tb.yaml |  4 ++--
 vmdb/spec/helpers/application_helper_spec.rb    | 11 +++++++++++
 vmdb/spec/models/vm_or_template_spec.rb         | 12 ++++++++++++
 10 files changed, 50 insertions(+), 2 deletions(-)

Comment 11 CFME Bot 2015-08-18 22:11:29 UTC
New commit detected on cfme/5.4.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=598799c30b3ee2a0f63409fcdb36ba7ffccc66f8

commit 598799c30b3ee2a0f63409fcdb36ba7ffccc66f8
Merge: 5ae8c86 2a557df
Author:     Dan Clarizio <dclarizi>
AuthorDate: Tue Aug 18 12:33:59 2015 -0400
Commit:     Dan Clarizio <dclarizi>
CommitDate: Tue Aug 18 12:33:59 2015 -0400

    Merge branch '54z_bz_1245717' into '5.4.z'
    
    Added model support to determine if Smartstate Analysis is supported.
    
    - Added model support to determine if Smartstate Analysis button is supported for VM/Instances, added a check in toolbar code to display a hover message when button is disabled and not supported for certain types of VM/Instance.
    - Fixed hover text on the toolbar button, title should say "Perform SmartState Analysis on this Instance" instead of "Perform SmartState Analysis on this Image"
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1240337
    https://bugzilla.redhat.com/show_bug.cgi?id=1245717
    
    @dclarizio please review, had to make some changes by hand, VM related models & toolbar builder code have been moved/renamed on upstream.
    original PR: https://github.com/ManageIQ/manageiq/pull/3808
    
    See merge request !225

 vmdb/app/helpers/application_helper.rb          |  1 +
 vmdb/app/models/vm.rb                           |  4 ++++
 vmdb/app/models/vm_amazon.rb                    |  4 ++++
 vmdb/app/models/vm_cloud.rb                     |  4 ++++
 vmdb/app/models/vm_openstack.rb                 |  4 ++++
 vmdb/app/models/vm_redhat.rb                    |  4 ++++
 vmdb/app/models/vm_vmware.rb                    |  4 ++++
 vmdb/product/toolbars/x_vm_cloud_center_tb.yaml |  4 ++--
 vmdb/spec/helpers/application_helper_spec.rb    | 11 +++++++++++
 vmdb/spec/models/vm_or_template_spec.rb         | 12 ++++++++++++
 10 files changed, 50 insertions(+), 2 deletions(-)

Comment 14 errata-xmlrpc 2015-12-08 13:21:58 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.