Bug 1642478 - Azure refresh fails with [NoMethodError]: undefined method `sku'
Summary: Azure refresh fails with [NoMethodError]: undefined method `sku'
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Providers
Version: 5.9.4
Hardware: Unspecified
OS: Unspecified
medium
high
Target Milestone: GA
: 5.10.0
Assignee: Daniel Berger
QA Contact: Angelina Vasileva
URL:
Whiteboard:
: 1652101 (view as bug list)
Depends On:
Blocks: 1646646
TreeView+ depends on / blocked
 
Reported: 2018-10-24 13:38 UTC by Niladri Roy
Modified: 2021-12-10 18:02 UTC (History)
9 users (show)

Fixed In Version: 5.10.0.23
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1646646 (view as bug list)
Environment:
Last Closed: 2019-02-12 16:50:31 UTC
Category: ---
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

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. ***


Note You need to log in before you can comment on or make changes to this bug.