Bug 1642478

Summary: Azure refresh fails with [NoMethodError]: undefined method `sku'
Product: Red Hat CloudForms Management Engine Reporter: Niladri Roy <niroy>
Component: ProvidersAssignee: Daniel Berger <dberger>
Status: CLOSED CURRENTRELEASE QA Contact: Angelina Vasileva <anikifor>
Severity: high Docs Contact:
Priority: medium    
Version: 5.9.4CC: anikifor, dmetzger, gblomqui, jfrey, jhardy, ncatling, niroy, obarenbo, simaishi
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.10.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.10.0.23 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1646646 (view as bug list) Environment:
Last Closed: 2019-02-12 16:50:31 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1646646    

Description Niladri Roy 2018-10-24 13:38:37 UTC
Description of problem:
Azure provider refresh fails with below error

[----] E, [2018-10-24T05:30:11.029046 #34199:8e3118] ERROR -- : [NoMethodError]: undefined method `sku' for #<Azure::Armrest::Storage::Disk:0x000000000f616c78>  Method:[block in method_missing]


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

How reproducible:


Steps to Reproduce:
1. Add Azure provider
2. Trigger a refresh
3.

Actual results:
Refresh fails with error

Expected results:
Refresh should succeed

Additional info:

Comment 4 Daniel Berger 2018-10-24 20:18:37 UTC
Can you tell me what the output of the following command shows?

az disk list --query '[].{Name:name,Sku:sku}'

I'm interested to see if any of them don't have a sku.

Comment 11 CFME Bot 2018-11-02 14:13:35 UTC
New commit detected on ManageIQ/manageiq-providers-azure/hammer:

https://github.com/ManageIQ/manageiq-providers-azure/commit/07691319edf14af6f6edf64c382f37fd616bc072
commit 07691319edf14af6f6edf64c382f37fd616bc072
Author:     Ladislav Smola <lsmola>
AuthorDate: Thu Nov  1 12:02:53 2018 -0400
Commit:     Ladislav Smola <lsmola>
CommitDate: Thu Nov  1 12:02:53 2018 -0400

    Merge pull request #300 from djberg96/disk_sku

    Handle possibility that disk might not have sku

    (cherry picked from commit 60d75641b09d7c79e8c27890ca39c772a9836b94)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1642478

 app/models/manageiq/providers/azure/cloud_manager/refresh_parser.rb | 4 +-
 app/models/manageiq/providers/azure/inventory/parser/cloud_manager.rb | 2 +-
 spec/models/manageiq/providers/azure/cloud_manager/azure_refresher_spec_common.rb | 2 +
 3 files changed, 5 insertions(+), 3 deletions(-)

Comment 12 CFME Bot 2018-11-02 14:13:42 UTC
New commit detected on ManageIQ/manageiq-providers-azure/hammer:

https://github.com/ManageIQ/manageiq-providers-azure/commit/07691319edf14af6f6edf64c382f37fd616bc072
commit 07691319edf14af6f6edf64c382f37fd616bc072
Author:     Ladislav Smola <lsmola>
AuthorDate: Thu Nov  1 12:02:53 2018 -0400
Commit:     Ladislav Smola <lsmola>
CommitDate: Thu Nov  1 12:02:53 2018 -0400

    Merge pull request #300 from djberg96/disk_sku

    Handle possibility that disk might not have sku

    (cherry picked from commit 60d75641b09d7c79e8c27890ca39c772a9836b94)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1642478

 app/models/manageiq/providers/azure/cloud_manager/refresh_parser.rb | 4 +-
 app/models/manageiq/providers/azure/inventory/parser/cloud_manager.rb | 2 +-
 spec/models/manageiq/providers/azure/cloud_manager/azure_refresher_spec_common.rb | 2 +
 3 files changed, 5 insertions(+), 3 deletions(-)

Comment 14 Angelina Vasileva 2018-11-15 14:47:53 UTC
Fixed and verified in 5.10.0.24.20181113213923_03b81fd

Comment 15 Bronagh Sorota 2018-11-26 20:51:12 UTC
*** Bug 1652101 has been marked as a duplicate of this bug. ***