Bug 1369228 - Error on Azure Cloud Discovery: wrong number of arguments
Summary: Error on Azure Cloud Discovery: wrong number of arguments
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Providers
Version: 5.6.0
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: GA
: 5.7.0
Assignee: Daniel Berger
QA Contact: Jeff Teehan
URL:
Whiteboard: provider:azure:discover
Depends On:
Blocks: 1374815
TreeView+ depends on / blocked
 
Reported: 2016-08-22 17:53 UTC by Jerome Marc
Modified: 2017-01-12 05:08 UTC (History)
8 users (show)

Fixed In Version: 5.7.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1374815 (view as bug list)
Environment:
Last Closed: 2017-01-11 19:49:29 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

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.
Note You need to log in before you can comment on or make changes to this bug.