azure-powershell icon indicating copy to clipboard operation
azure-powershell copied to clipboard

[Az.Migrate] Stabilize command internals & update policy defaults

Open minhsuanlee opened this issue 1 month ago • 6 comments

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.)

    • [x] General release
    • [ ] Public preview
    • [ ] Private preview
    • [ ] Engineering build
    • [ ] No need for a release
  • [x] Check this box to confirm: I have read the Submitting Changes section of CONTRIBUTING.md and reviewed the following information:

  • 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

minhsuanlee avatar Nov 14 '25 00:11 minhsuanlee

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

/azp run

isra-fel avatar Nov 14 '25 01:11 isra-fel

Azure Pipelines successfully started running 3 pipeline(s).

azure-pipelines[bot] avatar Nov 14 '25 01:11 azure-pipelines[bot]

Replication policy now aligns with Portal UX for default config. image

Passing dev test with private module 3.0.17: https://dev.azure.com/msazure/One/_build/results?buildId=143763316&view=results

image

minhsuanlee avatar Nov 14 '25 23:11 minhsuanlee

Hey @minhsuanlee, @mayada83,

  1. What are these changes related to?
  2. Why did you remove all the test cases?

notyashhh avatar Nov 21 '25 05:11 notyashhh

@minhsuanlee, @mayada83, Is this PR still targeting 15.1?

notyashhh avatar Nov 25 '25 02:11 notyashhh

Hey @minhsuanlee, @mayada83,

  1. What are these changes related to?
  2. Why did you remove all the test cases?
  1. What are these changes related to? a. Putting back usage of $PSBoundParameters in AzLocal commands b. Update Policy default values to align with the product

  2. Why did you remove all the test cases? Following the suggestion by Joyer Jin from PS team, I've created a more comprehensive unit test of Test-AzMigrateLocalEndToEnd.Recording.json to test the entire scenario of AzLocal migrate instead of testing individual commands separately

minhsuanlee avatar Dec 10 '25 18:12 minhsuanlee