Missing doNotRunExtensionsOnOverprovisionedVMs in @azure/arm-compute model and docs

[michax]

• Package Name: @azure/arm-compute
• Package Version: 11.0.0
• Operating system: Windows 10
• nodejs
  • version: 10.15.3
• browser
  • name/version:
• typescript
  • version: node package: 3.3.3, tsc: 1.5.3
• Is the bug related to documentation in
  • README.md
  • source code documentation
  • SDK API docs on https://docs.microsoft.com

Describe the bug
Missing property doNotRunExtensionsOnOverprovisionedVMs in ComputeManagementModels.VirtualMachineScaleSetUpdate typescript interface from @azure/arm-compute. This property is supported by 2019-03-01 api version:
https://docs.microsoft.com/en-us/rest/api/compute/virtualmachinescalesets/update

This property is also missing in sdk references:
https://azure.github.io/azure-sdk-for-js/arm-compute/interfaces/virtualmachinescalesetupdate.html

To Reproduce
Steps to reproduce the behavior:

1. Go to @azure/arm-comptue docs: https://azure.github.io/azure-sdk-for-js/arm-compute/interfaces/virtualmachinescalesetupdate.html
2. Property "doNotRunExtensionsOnOverprovisionedVMs" is missing there. And it's missing in typescript interface as well.

Expected behavior
New vmss property called "doNotRunExtensionsOnOverprovisionedVMs" should be added both to docs and typescript interface "ComputeManagementModels.VirtualMachineScaleSetUpdate"

Screenshots

[ghost]

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

[ghost]

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

[Drewm3]

Hi @michax, I am checking with the dev team to make sure the definition for this API is documented correctly. If everything is correct on the VMSS API side, then I will move this over to the java script SDK team to determine why this property is not showing up in the current SDK.

[ruowan]

Hi, @michax . The property doNotRunExtensionsOnOverprovisionedVMs is defined here https://azure.github.io/azure-sdk-for-js/arm-compute/interfaces/virtualmachinescaleset.html.

[Drewm3]

@michax, please let me know if ruowan's comment clarifies things. If it does, then I'll close this issue.

[michax]

@ruowan @Drewm3 But shouldn't it also be part of actual method used for updating VMSS?
Currently I'm unable to pass doNotRunExtensionsOnOverprovisionedVMs in VMSS beginUpdate method payload.
https://azure.github.io/azure-sdk-for-js/arm-compute/interfaces/virtualmachinescalesetupdate.html

However i can easily update this value using just 'raw' rest api. I believe sdk should allow for the same with statically typed approach.

[ruowan]

Hi @michax . You can try VirtualMachineScaleSets.createOrUpdate method which allow this property. I also agree with you. We will release a new version of this package after discussing it with team.

[yungezz]

Azure/azure-rest-api-specs#7513

[michax]

@ruowan Thanks!

[ghost]

Thanks for working with Microsoft on GitHub! Tell us how you feel about your experience using the reactions on this comment.