Skip to content

CMK support for VMware migration and addition of few properties for VMware DR monitoring#10477

Merged
isra-fel merged 12 commits intoAzure:masterfrom
AsrOneSdk:anbhat/cmk_resync
Mar 12, 2020
Merged

CMK support for VMware migration and addition of few properties for VMware DR monitoring#10477
isra-fel merged 12 commits intoAzure:masterfrom
AsrOneSdk:anbhat/cmk_resync

Conversation

@anmolbhatia289
Copy link
Contributor

SDK for net changes corresponding to the changes in PR : Azure/azure-rest-api-specs#8390

@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

@isra-fel isra-fel self-assigned this Mar 10, 2020
@isra-fel isra-fel added Mgmt This issue is related to a management package. needs-review labels Mar 10, 2020
Copy link
Member

@isra-fel isra-fel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @anmolbhatia289 , could you

  • Check in the .txt file generated when you run generate.ps1(details)
    • In case you have trouble running generate.ps1, please update the autorest version in it from latest to 2.0.4413
  • Update release notes in csproj
  • Write tests to cover new operations

Thanks

@anmolbhatia289
Copy link
Contributor Author

anmolbhatia289 commented Mar 11, 2020

We are planning to add tests related to all migration item operations in a separate PR. Currently, none of the migration item operations have tests.

@anmolbhatia289 anmolbhatia289 requested a review from isra-fel March 11, 2020 17:25
@anmolbhatia289 anmolbhatia289 removed their assignment Mar 11, 2020
@isra-fel isra-fel self-assigned this Mar 12, 2020
Copy link
Member

@isra-fel isra-fel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK. Looks good.

@isra-fel isra-fel merged commit c96c98c into Azure:master Mar 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Mgmt This issue is related to a management package.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants