Bug 1383222

Summary: Error: "undefined method `to_sym' for nil:NilClass on validate scvmm provider.
Product: Red Hat CloudForms Management Engine Reporter: Aziza Karol <akarol>
Component: UI - OPSAssignee: Aparna Karve <akarve>
Status: CLOSED CURRENTRELEASE QA Contact: Niyaz Akhtar Ansari <nansari>
Severity: medium Docs Contact:
Priority: high    
Version: 5.7.0CC: akarol, akarve, cpelland, hkataria, jhardy, mpovolny, nansari, obarenbo, simaishi
Target Milestone: GAKeywords: TestOnly
Target Release: 5.8.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: ui:scvmm:validation
Fixed In Version: 5.8.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1387409 (view as bug list) Environment:
Last Closed: 2017-06-12 17:20:38 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Bug Depends On:    
Bug Blocks: 1387409    

Description Aziza Karol 2016-10-10 08:49:14 UTC
Created attachment 1208770 [details]
error

Description of problem:


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

How reproducible:
5.7.0.4

Steps to Reproduce:
1.Validate scvmm provider without selecting security protocol(enter rest of the fields)
2.
3.

Actual results:
undefined method `to_sym' for nil:NilClass is displayed in UI. see attached screenshot.

Expected results:


Additional info:

Comment 4 Aziza Karol 2016-10-18 05:28:19 UTC
Dan,

It was a flash validation error in UI if we did't select the security protocol.

evm.log:
[----] I, [2016-10-18T01:30:45.757996 #11842:160399c]  INFO -- : MIQ(ManageIQ::Providers::Microsoft::InfraManager#with_provider_connection) Connecting through ManageIQ::Providers::Microsoft::InfraManager: [SCVMM2]
[----] W, [2016-10-18T01:30:45.758583 #11842:160399c]  WARN -- : MIQ(ManageIQ::Providers::Microsoft::InfraManager#authentication_check_no_validation) type: ["default"] for [] [SCVMM2] Validation failed: error, Unable to connect: undefined method `to_sym' for nil:NilClass
[----] E, [2016-10-18T01:30:45.758721 #11842:160399c] ERROR -- : MIQ(ems_infra_controller-create): Credential validation was not successful: Unable to connect: undefined method `to_sym' for nil:NilClass
[----] I, [2016-10-18T01:30:47.263463 #11760:f79158]  INFO -- : MIQ(MiqScheduleWorker::Runner#do_work) Number of scheduled items to be processed: 1.
[----] I, [2016-10-18T01:30:47.289188 #11760:f79158]  INFO -- : MIQ(MiqQueue.put) Message id: [869],  id: [], Zone: [default], Role: [smartstate], Server: [], Ident: [generic], Target id: [], Instance id: [], Task id: [job_dispatcher], Command: [JobProxyDispatcher.dispatch], Timeout: [600], Priority: [20], State: [ready], Deliver On: [], Data: [], Args: []


Seems to be fixed by Aparna in above PR.

Thanks,
Aziza

Comment 5 CFME Bot 2016-10-18 15:31:12 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/ac8099023090728565285637e2d83ceb62329283

commit ac8099023090728565285637e2d83ceb62329283
Author:     Aparna Karve <akarve>
AuthorDate: Fri Oct 14 16:45:55 2016 -0700
Commit:     Aparna Karve <akarve>
CommitDate: Fri Oct 14 16:45:55 2016 -0700

    For scvmm, set the default security protocol to ssl
    
    This will ensure that Validation takes into account the security
    protocol for scvmm
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1383222

 .../javascripts/controllers/ems_common/ems_common_form_controller.js    | 2 ++
 1 file changed, 2 insertions(+)

Comment 6 CFME Bot 2016-10-19 03:57:13 UTC
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/e42e425cb732c768ecbd9f3057e38566b127006d

commit e42e425cb732c768ecbd9f3057e38566b127006d
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Tue Oct 18 17:28:12 2016 +0200
Commit:     Oleg Barenboim <chessbyte>
CommitDate: Tue Oct 18 22:51:46 2016 -0500

    Merge pull request #11965 from AparnaKarve/bz1383222_security_protocol_mandatotory
    
    For scvmm, set the default security protocol to ssl
    (cherry picked from commit 2f44353da52391d1edf2e562ac7d52ea2aed00ac)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1383222

 .../javascripts/controllers/ems_common/ems_common_form_controller.js    | 2 ++
 1 file changed, 2 insertions(+)

Comment 8 Aparna Karve 2016-10-19 13:44:58 UTC
Yes, moving it to POST.

Comment 10 Niyaz Akhtar Ansari 2017-03-20 11:35:12 UTC
It looks good to me. 
Verified in version 5.8.0.4