Azure HPC Cache name is limited to 31 characters instead of 80

[jsmartt]

Many places in this file, there is a regex, validating the name matches the pattern ^[-0-9a-zA-Z_]{1,31}$. It will prevent any action (including get or create) with a name longer than 31 characters by raising the following error:

ArgumentError: 'cache_name' should satisfy the constraint - 'Pattern': '^[-0-9a-zA-Z_]{1,31}$'

However, the Azure console shows the following validation:

The API validation allows 80 characters, so I'm guessing this SDK needs updated.

[ghost]

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @xgithubtriage

[mmyyrroonn]

@jsmartt Hello. Our SDK is auto-generated. The root cause is that service team set this limitation in their swagger spec. We have to fix it in the swagger spec first. I add service attention to invite correct team.
https://github.com/Azure/azure-rest-api-specs/blob/62b1b41dd4637c3667943aa5b845f08e81671437/specification/storagecache/resource-manager/Microsoft.StorageCache/stable/2019-11-01/storagecache.json#L269

But it's better to open a PR in this repo https://github.com/Azure/azure-rest-api-specs/

[jsmartt]

Thanks @myronfanqiu. I did open a PR on that repo, but it looks like it needs some help from some MSFT folks to fix build/validation issues and enable SDK automation.

[mmyyrroonn]

Could you give me the PR link?

[jsmartt]

It's linked just above my comment too, but it's Azure/azure-rest-api-specs#8606

[romahamu]

Hi @jsmartt thanks for pointing it out.

@myronfanqiu There is a new version coming, which has this fix. https://github.com/Azure/azure-rest-api-specs/pull/8403/files

[mmyyrroonn]

@romahamu good to know. Once it's merged, you can request a sdk release through https://portal.azure-devex-tools.com/

[lilyjma]

Hi, this issue appears to be resolved, so I'm closing it now. Thanks!