Bug 1508577 - 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.
Summary: Smartstate Analysis Snapshot of Azure Managed Disks fails with "The value of ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: SmartState Analysis
Version: 5.9.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: GA
: 5.10.0
Assignee: Jerry Keselman
QA Contact: Dave Johnson
URL:
Whiteboard:
Depends On:
Blocks: 1512964 1512967
TreeView+ depends on / blocked
 
Reported: 2017-11-01 17:32 UTC by Jerry Keselman
Modified: 2019-08-06 20:04 UTC (History)
5 users (show)

Fixed In Version: 5.10.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1512964 1512967 (view as bug list)
Environment:
Last Closed: 2018-06-21 20:40:22 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Jerry Keselman 2017-11-01 17:32:44 UTC
Description of problem: When running Smartstate Analysis on Azure Managed disks, we compose the name of the snapshot by appending the 20 characters string "__EVM__SSA__SNAPSHOT" to the os disk name string.  There is a snapshot name limitation of 80 characters, so if the disk name exceeds 60 characters, the call fails with "The value of parameter snapshot.name is invalid. (cause: 400 Bad Request) creating SSA Snapshot" and the name of the snapshot.


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


How reproducible: When the name of the Instance's managed disk exceeds 60 characters (including the GUID added internally by Azure).


Steps to Reproduce:
1. Find a managed disk Instance with an OS disk name longer than 60 characters
2. Run Smartstate Analysis on it.
3.

Actual results: The error above is returned.


Expected results: Smartstate Analysis succeeds


Additional info:

Comment 2 Jerry Keselman 2017-11-01 17:37:09 UTC
We will probably change the code to use the name of the instance instead of the name of the os disk.  This should shorten it considerably.  We can also shorten the snapshot suffix to _SSA_SNAPSHOT - this will shorten it to 13 characters from 20.

Comment 3 Jerry Keselman 2017-11-03 19:10:27 UTC
PRs https://github.com/ManageIQ/manageiq-providers-azure/pull/157 and https://github.com/ManageIQ/manageiq-smartstate/pull/44 have been added for this problem.  The Azure provider PR has been merged upstream.  Once the smart state gem has been updated the Azure provider Gemfile will need to be updated as well under the auspices of another PR.

Comment 6 Jerry Keselman 2017-11-13 15:03:35 UTC
Now moving to post as the manageiq-smartstate gem version 0.2.3 has been pushed out.


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