Bug 1369228

Summary:	Error on Azure Cloud Discovery: wrong number of arguments
Product:	Red Hat CloudForms Management Engine	Reporter:	Jerome Marc <jmarc>
Component:	Providers	Assignee:	Daniel Berger <dberger>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Jeff Teehan <jteehan>
Severity:	medium	Docs Contact:
Priority:	high
Version:	5.6.0	CC:	akarve, cpelland, dberger, gblomqui, jfrey, jhardy, obarenbo, simaishi
Target Milestone:	GA	Keywords:	TestOnly, ZStream
Target Release:	5.7.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	provider:azure:discover
Fixed In Version:	5.7.0.1	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1374815 (view as bug list)		Environment:
Last Closed:	2017-01-11 19:49:29 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Bug Depends On:
Bug Blocks:	1374815

Description Jerome Marc 2016-08-22 17:53:33 UTC

Description of problem:
The following error occurs when performing a Cloud Discovery for Azure:
'Cloud Providers Discovery returned: wrong number of arguments (3 for 4)'

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

How reproducible:
Always 

Steps to Reproduce:
1. Go to Compute > Clouds > Cloud Providers
2. Select Configuration > Discover Cloud Providers
3. Select Azure and type your credentials (Client ID, Client Key & Azure Tenant ID)

Actual results:
An error is displayed:
'Cloud Providers Discovery returned: wrong number of arguments (3 for 4)'

Expected results:
No error

Additional info:

Comment 2 Jerome Marc 2016-08-23 20:44:10 UTC

Seems to be fixed in latest errata RHSA-2016:1634 - Bug 1362654

Comment 3 Greg Blomquist 2016-08-25 14:00:58 UTC

Dan, I wonder if this is fixed upstream already

Comment 4 Jeff Teehan 2016-08-30 16:57:51 UTC

This is not fixed upstream.  It is working on 5612

Checked just now against ManageIQ
Version master.20160830110816_7d6e70f
Server Name EVM
User Name Administrator

Still missing the sub field upstream.
https://10.16.5.120/ems_infra/show_list

Comment 5 Aparna Karve 2016-09-06 18:29:57 UTC

Upstream PR -- https://github.com/ManageIQ/manageiq/pull/10531

Comment 6 Daniel Berger 2016-09-07 19:08:06 UTC

Aparna's PR fixes the immediate problem, and will work so long as you provide a subscription ID. We are currently debating what the behavior should be if no subscription ID is provided.

Comment 8 Daniel Berger 2016-09-09 16:35:34 UTC

Additional upstream PR: https://github.com/ManageIQ/manageiq/pull/11116.

It was decided that the subscription ID always be mandatory for now.

Comment 10 Jeff Teehan 2016-09-22 16:38:55 UTC

Verified this is working on 10.16.6.191 running 5.7.0.2

Added Polarion test case id.