Skip to content

Conversation

amarjeetkr
Copy link
Member

Description

  • Published the stable API version 2025-06-30 for DMS APIs.
  • Added Retry API for Database Migration.
  • Introduced Delete API for SQL Database Migration.
  • Introduced Delete API for SQL VM Migration.

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

@Copilot Copilot AI review requested due to automatic review settings August 25, 2025 12:13
Copy link

Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR publishes the stable API version 2025-06-30 for Azure Database Migration Service (DMS) APIs, replacing the preview version 2025-03-15-preview. It introduces new retry and delete operations for database migrations.

  • Updates the API version from preview (2025-03-15-preview) to stable (2025-06-30)
  • Adds new Retry API for Database Migration to SQL Database
  • Introduces Delete APIs for SQL Database Migration and SQL VM Migration

Reviewed Changes

Copilot reviewed 26 out of 26 changed files in this pull request and generated 8 comments.

Show a summary per file
File Description
src/DataMigration/DataMigration.Autorest/readme.azure.noprofile.md New AutoRest configuration file for generating PowerShell cmdlets
src/DataMigration/DataMigration.Autorest/test/*.Tests.ps1 New test files for the retry and delete cmdlets with placeholder implementations
src/DataMigration/DataMigration.Autorest/examples/*.md Example documentation files with placeholder content for new cmdlets
src/DataMigration/DataMigration.Autorest/docs/*.md Updated documentation files with new API version references and new cmdlet documentation
src/DataMigration/DataMigration.Autorest/UX/*.json Updated API version references and new UX metadata for retry operation
Comments suppressed due to low confidence (2)

Comment on lines +1 to +22
### Example 1: {{ Add title here }}
```powershell
{{ Add code here }}
```

```output
{{ Add output here }}
```

{{ Add description here }}

### Example 2: {{ Add title here }}
```powershell
{{ Add code here }}
```

```output
{{ Add output here }}
```

{{ Add description here }}

Copy link
Preview

Copilot AI Aug 25, 2025

Choose a reason for hiding this comment

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

The example documentation contains only placeholder content. Real examples should be provided showing how to use the Remove-AzDataMigrationToSqlVM cmdlet with actual parameter values and expected outputs.

Suggested change
### Example 1: {{ Add title here }}
```powershell
{{ Add code here }}
```
```output
{{ Add output here }}
```
{{ Add description here }}
### Example 2: {{ Add title here }}
```powershell
{{ Add code here }}
```
```output
{{ Add output here }}
```
{{ Add description here }}
### Example 1: Remove a Data Migration to SQL VM by Name
```powershell
Remove-AzDataMigrationToSqlVM -ResourceGroupName "MyResourceGroup" -SqlVirtualMachineName "MySqlVM" -Name "MyMigration"
True

This example removes the data migration named "MyMigration" from the SQL virtual machine "MySqlVM" in the resource group "MyResourceGroup". The command returns True if the operation succeeds.

Example 2: Remove a Data Migration to SQL VM using the Input Object

$migration = Get-AzDataMigrationToSqlVM -ResourceGroupName "MyResourceGroup" -SqlVirtualMachineName "MySqlVM" -Name "MyMigration"
Remove-AzDataMigrationToSqlVM -InputObject $migration
True

This example retrieves the data migration object and then removes it using the -InputObject parameter. This is useful when you want to pipe or work with objects directly.

Copilot uses AI. Check for mistakes.

@amarjeetkr amarjeetkr changed the title Powershell: Publishing DMS APIs stable vetsion Powershell: Publishing DMS APIs stable version Aug 25, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@isra-fel
Copy link
Member

/azp run

Copy link
Contributor

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

@NoriZC
Copy link
Contributor

NoriZC commented Aug 26, 2025

/azp run

Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@VeryEarly VeryEarly self-assigned this Aug 26, 2025
@@ -0,0 +1,47 @@
# Azure PowerShell AutoRest Configuration
Copy link
Collaborator

Choose a reason for hiding this comment

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

please do not add this file

Copy link
Collaborator

@VeryEarly VeryEarly left a comment

Choose a reason for hiding this comment

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

please check-in all changed files under src/DataMigration

Copy link
Collaborator

@VeryEarly VeryEarly left a comment

Choose a reason for hiding this comment

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

please complete new examples and test cases

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants