Bug 1374815

Summary: Error on Azure Cloud Discovery: wrong number of arguments
Product: Red Hat CloudForms Management Engine Reporter: Satoe Imaishi <simaishi>
Component: ProvidersAssignee: Daniel Berger <dberger>
Status: CLOSED ERRATA QA Contact: Jeff Teehan <jteehan>
Severity: medium Docs Contact:
Priority: high    
Version: 5.6.0CC: akarve, cpelland, dberger, gblomqui, jfrey, jhardy, obarenbo, simaishi
Target Milestone: GAKeywords: ZStream
Target Release: 5.6.2   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: provider:azure:discover
Fixed In Version: 5.6.2.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1369228 Environment:
Last Closed: 2016-10-04 14:36:40 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1369228    
Bug Blocks:    

Comment 2 CFME Bot 2016-09-09 22:25:59 UTC
New commit detected on ManageIQ/manageiq/darga:
https://github.com/ManageIQ/manageiq/commit/e256208d8608382edf5f9926bb89a52233be4196

commit e256208d8608382edf5f9926bb89a52233be4196
Author:     Aparna Karve <akarve>
AuthorDate: Tue Aug 16 15:13:04 2016 -0700
Commit:     Aparna Karve <akarve>
CommitDate: Fri Sep 9 13:55:07 2016 -0700

    Add subscription to Azure Discovery screen
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1374815

 app/controllers/application_controller/ci_processing.rb | 4 +++-
 app/views/layouts/_discover_azure_credentials.html.haml | 9 +++++++++
 2 files changed, 12 insertions(+), 1 deletion(-)

Comment 3 CFME Bot 2016-09-09 22:30:59 UTC
New commit detected on ManageIQ/manageiq/darga:
https://github.com/ManageIQ/manageiq/commit/f09d1b49dad0b4a7875c70e73605f3791b64cf39

commit f09d1b49dad0b4a7875c70e73605f3791b64cf39
Author:     Dan Clarizio <dclarizi>
AuthorDate: Thu Sep 8 14:36:42 2016 -0700
Commit:     Oleg Barenboim <chessbyte>
CommitDate: Fri Sep 9 18:26:43 2016 -0400

    Merge pull request #11116 from AparnaKarve/subscription_id_mandatory
    
    Make Subscription ID field mandatory in Azure discovery
    (cherry picked from commit 91243b96b7f85aa584377b85f2ca9f8aafd11934)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1374815

 app/controllers/application_controller/ci_processing.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comment 4 Jeff Teehan 2016-09-16 14:43:16 UTC
This was already working in 5.6.1, but I tested it for 5.6.2 and it still works.  All four fields are required.  I tried mixing and matching blank values which produced appropriate warning.

Moving to verified using 10.16.6.182

Comment 5 Jeff Teehan 2016-09-16 14:44:26 UTC
Closed loop on Test Case ID

Comment 7 errata-xmlrpc 2016-10-04 14:36:40 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://rhn.redhat.com/errata/RHSA-2016-1996.html