Update StorageMover project configurations and test recordings #28603
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Modified Update-AzStorageMoverProject recording JSON to reflect new resource group and project names. - Updated environment variables in env.json to match new naming conventions and resource identifiers. - Adjusted utility script to set the correct resource group name. - Changed project GUID in solution file to maintain consistency. - Updated Az.StorageMover module generation date in the PSD1 file. - Enhanced help documentation for New-AzStorageMoverAzStorageContainerEndpoint and New-AzStorageMoverMultiCloudConnectorEndpoint with identity details. - Corrected online version link for Update-AzStorageMoverAzNfsFileShareEndpoint help documentation. - Added identity information to the output properties in help documentation for storage container and multi-cloud connector endpoints.
| Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status. |
There was a problem hiding this comment.
Pull Request Overview
This PR updates StorageMover project configurations and test recordings to reflect new resource group and naming conventions, along with enhanced help documentation for specific endpoints.
- Updates test recordings and environment variables with new resource group name (
choudharysutest) and random string identifier (o7je935h) - Enhances help documentation with identity information for storage container and multi-cloud connector endpoints
- Corrects online help link for Azure NFS file share endpoint documentation
Reviewed Changes
Copilot reviewed 45 out of 47 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
Update-AzStorageMoverAzStorageContainerEndpoint.md |
Adds identity properties to help documentation output |
Update-AzStorageMoverAzNfsFileShareEndpoint.md |
Fixes online version link URL |
New-AzStorageMoverMultiCloudConnectorEndpoint.md |
Removes identity line from help documentation |
New-AzStorageMoverAzStorageContainerEndpoint.md |
Adds identity properties to help documentation output |
Az.StorageMover.psd1 |
Updates module generation date |
StorageMover.sln |
Changes project GUID |
utils.ps1 |
Updates resource group name in test utility |
env.json |
Updates environment variables with new naming conventions |
| Multiple test recording files | Updates recorded API responses with new resource group and identifier names |
src/StorageMover/StorageMover/help/Update-AzStorageMoverAzNfsFileShareEndpoint.md
Outdated
Show resolved
Hide resolved
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
…ileShareEndpoint.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
VeryEarly
reviewed
Sep 25, 2025
|
|
||
| $Properties.EndpointType = "AzureMultiCloudConnector" | ||
| $PSBoundParameters.Add("Property", $Properties) | ||
|
|
There was a problem hiding this comment.
Is the default value returned from server "SystemAssigned"? If not this could be a possible breaking chang
There was a problem hiding this comment.
Actually the original PR for this new api version is not released yet, this is just a fix PR in succession to that.:
#28488
So should not cause any issue.
VeryEarly
requested changes
Sep 25, 2025
There was a problem hiding this comment.
Please update https://github.com/sssash18/azure-powershell/blob/choudharysu/fix_new_api_version/src/StorageMover/StorageMover/ChangeLog.md
Simply copy the PR description you have.
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
VeryEarly
approved these changes
Sep 26, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Mandatory Checklist
Please choose the target release of Azure PowerShell. (⚠️ Target release is a different concept from API readiness. Please click below links for details.)
Check this box to confirm: I have read the Submitting Changes section of
CONTRIBUTING.mdand reviewed the following information:ChangeLog.mdfile(s) appropriatelysrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.## Upcoming Releaseheader in the past tense.ChangeLog.mdif no new release is required, such as fixing test case only.