HDI add autoscale feature support#6158
Conversation
Automation for azure-sdk-for-rubyNothing to generate for azure-sdk-for-ruby |
Automation for azure-sdk-for-pythonThe initial PR has been merged into your service PR: |
Automation for azure-sdk-for-jsA PR has been created for you: |
Automation for azure-sdk-for-javaThe initial PR has been merged into your service PR: |
Automation for azure-sdk-for-goThe initial PR has been merged into your service PR: |
|
Can one of the admins verify this patch? |
e35e81d to
5692d42
Compare
NelsonDaniel
left a comment
There was a problem hiding this comment.
I left some review comments. Also, solve merge conflicts
...ation/hdinsight/resource-manager/Microsoft.HDInsight/preview/2015-03-01-preview/cluster.json
Outdated
Show resolved
Hide resolved
...ation/hdinsight/resource-manager/Microsoft.HDInsight/preview/2015-03-01-preview/cluster.json
Outdated
Show resolved
Hide resolved
...ation/hdinsight/resource-manager/Microsoft.HDInsight/preview/2015-03-01-preview/cluster.json
Outdated
Show resolved
Hide resolved
...ation/hdinsight/resource-manager/Microsoft.HDInsight/preview/2015-03-01-preview/cluster.json
Outdated
Show resolved
Hide resolved
...ation/hdinsight/resource-manager/Microsoft.HDInsight/preview/2015-03-01-preview/cluster.json
Outdated
Show resolved
Hide resolved
...cation/hdinsight/resource-manager/Microsoft.HDInsight/stable/2018-06-01-preview/cluster.json
Outdated
Show resolved
Hide resolved
...cation/hdinsight/resource-manager/Microsoft.HDInsight/stable/2018-06-01-preview/cluster.json
Outdated
Show resolved
Hide resolved
...cation/hdinsight/resource-manager/Microsoft.HDInsight/stable/2018-06-01-preview/cluster.json
Outdated
Show resolved
Hide resolved
...ation/hdinsight/resource-manager/Microsoft.HDInsight/preview/2015-03-01-preview/cluster.json
Outdated
Show resolved
Hide resolved
...cation/hdinsight/resource-manager/Microsoft.HDInsight/stable/2018-06-01-preview/cluster.json
Outdated
Show resolved
Hide resolved
|
Merge conflicts need to be resolved. |
cc046e5 to
64381fd
Compare
SDK Automation [Logs] (Generated from edd2f70)
|
64381fd to
a1be505
Compare
|
Looks good to me. Just waiting for ARM feedback. Need to reassign this PR since I will be OOF, and this needs to be merged before I come back |
|
@gaoxiaoning - New model types have been added to the swagger specification. I don't see any updates to the corresponding x-ms-examples in the operation where these types are being used. Please update the existing examples or add new examples that show the usage of newly added model types. |
|
@amarzavery , could you please help merge the PR or let us know if you have other concerns:) |
* removed unused properties and renamed example folder * removed provider version property * undone readme changes
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub - one location for using our validation tools and finding your workflow.
Contribution checklist:
ARM API Review Checklist
Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.
Please follow the link to find more details on API review process.