Dev/ccelik/dedicated host swagger

[celikcigdem]

Added new APIs for DedicatedHostGroups and DedicatedHosts

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:

• I have reviewed the documentation for the workflow.
• Validation tools were run on swagger spec(s) and have all been fixed in this PR.
• The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

• Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
• adding/removing APIs.
• adding/removing properties.
• adding/removing API-version.
• adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

• If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
  Please follow the link to find more details on API review process.

[openapi-sdkautomation]

SDK Automation [Logs] (Generated from 2b0c742)

Python: Azure/azure-sdk-for-python

• Package generation pending.

Java: Azure/azure-sdk-for-java

• Package generation pending.

JavaScript: Azure/azure-sdk-for-js

• Package generation pending.

Go: Azure/azure-sdk-for-go

• Package generation pending.

Ruby: Azure/azure-sdk-for-ruby

• Package generation pending.

[AutorestCI]

Automation for azure-sdk-for-go

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-go#5195

[AutorestCI]

Automation for azure-sdk-for-python

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-python#5853

[AutorestCI]

Automation for azure-sdk-for-ruby

Nothing to generate for azure-sdk-for-ruby

[AutorestCI]

Automation for azure-sdk-for-java

Nothing to generate for azure-sdk-for-java

[azuresdkci]

Can one of the admins verify this patch?

[anuchandy]

@sarangan12 @sergey-shandar linter emitted below error:

R2020 - RequiredPropertiesMissingInResourceModel
Model definition 'DedicatedHostGroupUpdate' must have the properties 'name', 'id' and 'type' in its hierarchy and these properties must be marked as readonly. 
at specification/compute/resource-manager/Microsoft.Compute/stable/2019-03-01/compute.json:5491

This is the PATCH request payload and is not used as response model.

However this type derive UpdateResource type which is marked as "x-ms-azure-resource", I don't see a need to mark it so though. Do you think linter emit this error due to this annotation? or anything else causing this error to trigger?

[sarangan12]

The new file is not referenced in the readme file. Please update it

[celikcigdem]

The new file is not referenced in the readme file. Please update it

@sarangan12, @anuchandy, I resolved your comments, the new iteration is available. Could you please review it?

[celikcigdem]

@hyonholee , @huangpf could you please review this PR?

[adewaleo]

@celikcigdem @sarangan12, should this be #/definitions/sku or something else?

This definition does not exist

[adewaleo]

Correction: this Sku exists. Its comment says it describes a virtual machine scale set sku. I will use it for now, but I would have imagined that a new type DedicatedHostSkuTypes would be defined for this property.