Workaround - Machine Learning Batch Endpoint identity#5204
Conversation
|
|
jackofallops
left a comment
jackofallops
left a comment
There was a problem hiding this comment.
Thanks @promisinganuj - one question below to potentially avoid bumping into this again if the service team do not fix this in an upcoming version.
| func (workaroundMachineLearning40149) IsApplicable(serviceName string, apiVersion sdkModels.APIVersion) bool { | ||
| serviceMatches := serviceName == "MachineLearningServices" | ||
| // 2025-06-01 is currently used for Machine Learning RPs, 2025-09-01 is the latest stable. So patching both. | ||
| apiVersionMatches := apiVersion.APIVersion == "2025-06-01" || apiVersion.APIVersion == "2025-09-01" |
There was a problem hiding this comment.
If this is present in all versions, is it worth omitting the version filter at this time? (Or are you aware of an incoming fix for future versions?).
There was a problem hiding this comment.
@jackofallops , I have reached out to ST but haven't got any committment from them yet around the fix. Given that there is already another stable version 2025-12-01 released last week which still have this issue, it makes sense to remove the version filter. I have pushed this change.
jackofallops
left a comment
jackofallops
left a comment
There was a problem hiding this comment.
Hi @promisinganuj - One change required below, since there's no point keeping the additional logic at this time.
Thanks
Co-authored-by: jackofallops <11830746+jackofallops@users.noreply.github.com>
|
@jackofallops , please have another look. |
sreallymatt
left a comment
sreallymatt
left a comment
There was a problem hiding this comment.
Thanks @promisinganuj - LGTM ✅
4 participants
Workaround for Azure/azure-rest-api-specs#40149