Bug 1475540

Summary: Need to verify that SSA works with Azure Managed Storage
Product: Red Hat CloudForms Management Engine Reporter: bascar
Component: SmartState AnalysisAssignee: Jerry Keselman <jkeselma>
Status: CLOSED CURRENTRELEASE QA Contact: Satyajit Bulage <sbulage>
Severity: urgent Docs Contact:
Priority: unspecified    
Version: 5.8.0CC: cpelland, djoo, jhardy, jmarc, mike, ncatling, obarenbo, roliveri, sbulage, simaishi
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.9.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.9.0.1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1488967 (view as bug list) Environment:
Last Closed: 2018-03-06 14:37:14 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: 1488967    

Comment 2 Jerry Keselman 2017-09-05 12:59:17 UTC
Four PRs have been implemented and merged upstream to handle this functionality - 

https://github.com/ManageIQ/manageiq-providers-azure/pull/117

https://github.com/ManageIQ/manageiq-smartstate/pull/23

https://github.com/ManageIQ/manageiq/pull/15865

and
https://github.com/ManageIQ/azure-armrest/pull/299.

The Azure-Armrest gem has been pushed out as Gem version 8.2

We are working through what needs to be done to push these changes downstream.

Comment 3 Jerry Keselman 2017-09-05 13:02:32 UTC
*** Bug 1459612 has been marked as a duplicate of this bug. ***

Comment 4 Jerry Keselman 2017-09-05 21:06:24 UTC
Another PR has been implemented and merged https://github.com/ManageIQ/manageiq-smartstate/pull/25 to increment the gem version number.  

Finally https://github.com/ManageIQ/manageiq/pull/15938 has been added to pull in the incremented version of the manageiq-smartstate gem.

Comment 5 Satoe Imaishi 2017-09-06 13:54:52 UTC
Also need https://github.com/ManageIQ/manageiq-providers-azure/pull/120

Comment 7 Jerry Keselman 2017-09-11 20:44:36 UTC
*** Bug 1490486 has been marked as a duplicate of this bug. ***

Comment 8 Jerry Keselman 2017-09-13 13:17:38 UTC
Additional changes are needed to wait for the snapshot status to be marked SUCCEEDED.

Comment 9 Jerry Keselman 2017-09-13 17:34:33 UTC
https://github.com/ManageIQ/manageiq-providers-azure/pull/125 has been added to handle the wait functionality.  Waiting for review and merge and back port to Fine.

Comment 11 CFME Bot 2017-09-15 21:08:12 UTC
New commit detected on ManageIQ/manageiq-providers-azure/master:
https://github.com/ManageIQ/manageiq-providers-azure/commit/dda46b223ee4f3123662d3297e9c859b49e893f2

commit dda46b223ee4f3123662d3297e9c859b49e893f2
Author:     Richard Oliveri <roliveri>
AuthorDate: Fri Sep 15 16:36:34 2017 -0400
Commit:     Richard Oliveri <roliveri>
CommitDate: Fri Sep 15 16:36:34 2017 -0400

    Changes to wait for snapshot completion.
    
    Loosened check for success. Added debugging to help determine the problem.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1463780
    https://bugzilla.redhat.com/show_bug.cgi?id=1475540

 app/models/manageiq/providers/azure/cloud_manager.rb | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

Comment 13 CFME Bot 2017-09-23 16:34:16 UTC
New commit detected on ManageIQ/manageiq-smartstate/master:
https://github.com/ManageIQ/manageiq-smartstate/commit/389ee0fc3854e44f97794c885cecce9533a902e3

commit 389ee0fc3854e44f97794c885cecce9533a902e3
Author:     Jerry Keselman <jerryk>
AuthorDate: Fri Sep 22 18:07:06 2017 -0400
Commit:     Jerry Keselman <jerryk>
CommitDate: Fri Sep 22 18:07:06 2017 -0400

    Rerecorded Azure Managed Disk tests
    
    As a result of several of the Azure Managed Disk changes for BZ
    https://bugzilla.redhat.com/show_bug.cgi?id=1475540
    these tests needed to be rerecorded.

 .../azure_managed_disk_spec_blockSize-1.yml        | 583 ++++++++++------
 .../azure_managed_disk_spec_diskType-1.yml         | 583 ++++++++++------
 .../azure_managed_disk_spec_endByteAddr-1.yml      | 581 ++++++++++------
 .../azure_managed_disk_spec_endByteAddr-2.yml      | 583 ++++++++++------
 .../azure_managed_disk_spec_getPartitions-1.yml    | 735 ++++++++++++++-------
 .../azure_managed_disk_spec_lbaEnd-1.yml           | 583 ++++++++++------
 .../azure_managed_disk_spec_lbaStart-1.yml         | 583 ++++++++++------
 .../azure_managed_disk_spec_new-1.yml              | 313 +++++----
 .../azure_managed_disk_spec_new-2.yml              | 583 ++++++++++------
 .../azure_managed_disk_spec_partNum-1.yml          | 580 +++++++++-------
 .../azure_managed_disk_spec_partType-1.yml         | 581 ++++++++++------
 .../azure_managed_disk_spec_size-1.yml             | 581 ++++++++++------
 .../azure_managed_disk_spec_startByteAddr-1.yml    | 581 ++++++++++------
 13 files changed, 4835 insertions(+), 2615 deletions(-)

Comment 14 Satyajit Bulage 2017-11-29 17:00:00 UTC
Performed SSA on Azure provider instance having managed disk.

Verified Version: 5.9.0.11.20171127204214_e316988