Bug 1512964

Summary: Smartstate Analysis Snapshot of Azure Managed Disks fails with "The value of parameter snapshot.name is invalid. (cause: 400 Bad Request) creating SSA Snapshot" if the disk name exceeds 60 characters.
Product: Red Hat CloudForms Management Engine Reporter: Satoe Imaishi <simaishi>
Component: SmartState AnalysisAssignee: Jerry Keselman <jkeselma>
Status: CLOSED CURRENTRELEASE QA Contact: Satyajit Bulage <sbulage>
Severity: high Docs Contact:
Priority: high    
Version: 5.9.0CC: agrare, gblomqui, jhardy, jkeselma, obarenbo, sbulage, simaishi
Target Milestone: GA   
Target Release: 5.9.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.9.0.7 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1508577 Environment:
Last Closed: 2018-03-06 14:59:17 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: 1508577    
Bug Blocks:    

Comment 2 Greg Blomquist 2017-11-16 23:12:16 UTC
Fine backport details:

$ git log -1
commit dd70691735386c387183f899e15d2ac47037ee3e
Author: Daniel Berger <djberg96>
Date:   Fri Nov 3 11:37:15 2017 -0600

    Merge pull request #157 from jerryk55/snapshot_disknames_longer_than_60chars
    
    Smart state Snapshot Managed Disk Name 80 Char Limit
    (cherry picked from commit e6d33b9bec96a69f48e1a356dc2060765069a2a8)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1512967

Comment 3 Satoe Imaishi 2017-11-16 23:37:32 UTC
This BZ is for Gaprindashvili :)

$ git log -1
commit 0709b6adff5ea2826b52522691c8fbb9ed3f82dc
Author: Daniel Berger <djberg96>
Date:   Fri Nov 3 11:37:15 2017 -0600

    Merge pull request #157 from jerryk55/snapshot_disknames_longer_than_60chars
    
    Smart state Snapshot Managed Disk Name 80 Char Limit
    (cherry picked from commit e6d33b9bec96a69f48e1a356dc2060765069a2a8)

Comment 4 CFME Bot 2017-11-29 20:32:05 UTC
New commit detected on ManageIQ/manageiq-providers-azure/gaprindashvili:
https://github.com/ManageIQ/manageiq-providers-azure/commit/28acba5cd4ce03937e5c08dacb4adf4a4a23228f

commit 28acba5cd4ce03937e5c08dacb4adf4a4a23228f
Author:     Richard Oliveri <oliveri.richard.github>
AuthorDate: Wed Nov 15 11:36:34 2017 -0500
Commit:     Satoe Imaishi <simaishi>
CommitDate: Mon Nov 20 14:46:56 2017 -0500

    Merge pull request #166 from jerryk55/manageiq-smartstate_0.2.3
    
    Upgrade manageiq-smartstate gem to 0.2.3
    (cherry picked from commit 1754f638f38874a90f5344608e28b31ab0f69e03)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1512964

 manageiq-providers-azure.gemspec | 1 +
 1 file changed, 1 insertion(+)

Comment 5 Satyajit Bulage 2018-01-10 10:16:32 UTC
As discussed with Jerry, counted disk/blob chars and perform SSA which retrieves required data.

Verified Version: 5.9.0.15.20180103231404_ee61a82