Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ReleasePR migrate] Adding new Tracked Resource Type for Microsoft.Migrate RP with new Ap… #2785

Merged
1 commit merged into from
Feb 8, 2023

Conversation

azure-sdk
Copy link

Adding new Tracked Resource Type for Microsoft.Migrate RP with new Ap… (#19045)

* Adding new Tracked Resource Type for Microsoft.Migrate RP with new Api Version

* Adding tag in readme and IISAKS to custom words

* Restructuring example files

* Renaming files

* Fixing Avocado error

* Rectifying swagger model validation

* Addressing code style and validation issues

* Implementing review comment

* fixing prettier check

* Deleting patch operations for proxy resources

* Modifying readme.md to accomodate new resourcetype

* Resolving errors

* Resolving errors

* Chankging tracked resource name from migrateDataCenter to modernizeProject

* Reverting addition of id fiels in migrate and hubmigrate.json

* .

* Delete non-existing x-ms-examples from swagger

* Auto-generate examples

* run prettier changes

* .

* prettier

* fix: Deleting and modifying examples

* fix: Generated new examples to remove some errors

* fix: Added x-ms-long-running-operation-options

* refactor: Ran prettier fix

* fix: Spell check error

* fix: Removed 2 unnecessary filenames from inputfiles section under 'package-2022-05' tag.

* feat: Latest Swagger updated. New examples generated.

* fix: Resolving LintDiff Errors

* fix: Ran prettier fix

* fix: fixing model validation errors

* fix: model validation error fix

* fix: Removed patch operation and updated camelCase

* fix: updated 'name' and restore readme.md

* fix: removed  tag: package-2022-05 and reverted the default tag to 2019-10

* fix:  OperationsAPIImplementation

* fix: OperationsAPI Implementaiton shifted to migrateEngine file

* fix: ran prettier fix on 2 new example files

* fix: updated package-2019 with package-migrate-2019

* refactor: Changed random guid to reopen the PR

* fix: Updated the swagger till date 19Jan

* fix: added 'fareast' in custom-words

* fix: Recreated examples

* fix: ran prettier command

* fix: camelCase at 3 places

* fix: Re generated examples

* fix: Removed the Loglink from ContainerImageProperties

* fix: Regenerated examples

* Added GMSA Configuration State

* fix: Added two DeployedResourcesProperties: isCleanUpDone, isTestMigration

* fix: Removed spellcheck

* fix: Added Produces in operations

* fix: schema error

* fix: Removed unnecessary configurations

---------

Co-authored-by: Shishir Gupta <shig@microsoft.com>
Co-authored-by: Shreyas Mali <shreyasmali@microsoft.com>
Co-authored-by: shishir-msft <55269898+shishir-msft@users.noreply.github.com>
Co-authored-by: shreyasmalims <117085373+shreyasmalims@users.noreply.github.com>
@ghost ghost added the automerge label Feb 8, 2023
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

This PR has been auto-approved by the @msftbot for the azure-resource-manager-schemas repo.

@ghost
Copy link

ghost commented Feb 8, 2023

Hello @msftbot[bot]!

Because this pull request has the automerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

Do note that I've been instructed to only help merge pull requests of this repository that have been opened for at least 1 minute. No worries though, I will be back when the time is right! 😉

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit 9db352b into Azure:main Feb 8, 2023
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant