Skip to content

Fix management live-test authentication and retire Storage Mover tests - #50074

Merged
Weidong Xu (weidongxu-microsoft) merged 4 commits into
Azure:mainfrom
weidongxu-microsoft:weidongxu/remove-storagemover-mgmt-tests-pipeline
Aug 10, 2026
Merged

Fix management live-test authentication and retire Storage Mover tests#50074
Weidong Xu (weidongxu-microsoft) merged 4 commits into
Azure:mainfrom
weidongxu-microsoft:weidongxu/remove-storagemover-mgmt-tests-pipeline

Conversation

@weidongxu-microsoft

@weidongxu-microsoft Weidong Xu (weidongxu-microsoft) commented Aug 10, 2026

Copy link
Copy Markdown
Member

Summary

  • warm federated authentication through the Resources inner client before Chaos and Machine
    Learning live tests enter older fluent asynchronous create paths
  • remove the Storage Mover management test pipeline configuration
  • retire the disabled daily and weekly Storage Mover management test definitions

Validation

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: b6797498-e941-465a-b215-87ed36922894
Copilot AI lite review requested due to automatic review settings August 10, 2026 05:47
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
34 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 removes the Azure Pipelines configuration used to run Storage Mover management live tests for the storagemover service directory, aligning the repo with the intent to retire that management test pipeline.

Changes:

  • Deleted sdk/storagemover/tests.mgmt.yml, removing the management test pipeline configuration (trigger/pr disabled, archetype test stage template, and Windows-only matrix filter).

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: b6797498-e941-465a-b215-87ed36922894
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: b6797498-e941-465a-b215-87ed36922894
@weidongxu-microsoft
Weidong Xu (weidongxu-microsoft) marked this pull request as draft August 10, 2026 06:14
auto-merge was automatically disabled August 10, 2026 06:14

Pull request was converted to draft

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: b6797498-e941-465a-b215-87ed36922894
@weidongxu-microsoft Weidong Xu (weidongxu-microsoft) changed the title Remove Storage Mover management test pipeline Fix management live-test authentication and retire Storage Mover tests Aug 10, 2026

@XiaofeiCao Xiaofei Cao (XiaofeiCao) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is it for ProviderRegistrationPolicy?

@weidongxu-microsoft

Weidong Xu (weidongxu-microsoft) commented Aug 10, 2026

Copy link
Copy Markdown
Member Author

Is it for ProviderRegistrationPolicy?

I am checking to re-use these live tests to run in AutoPR.

Letting agent to fix some existing ones that now fails.

@weidongxu-microsoft
Weidong Xu (weidongxu-microsoft) marked this pull request as ready for review August 10, 2026 11:40
@weidongxu-microsoft
Weidong Xu (weidongxu-microsoft) merged commit 3dd334a into Azure:main Aug 10, 2026
37 of 38 checks passed
@weidongxu-microsoft
Weidong Xu (weidongxu-microsoft) deleted the weidongxu/remove-storagemover-mgmt-tests-pipeline branch August 10, 2026 11:41
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
34 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

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.

3 participants