Skip to content

Migrate training storage from SAS to managed identity - #20618

Merged
snnn (snnn) merged 14 commits into
mainfrom
zhanyi/blob
May 9, 2024
Merged

Migrate training storage from SAS to managed identity#20618
snnn (snnn) merged 14 commits into
mainfrom
zhanyi/blob

Conversation

@mszhanyi

@mszhanyi Yi Zhang (mszhanyi) commented May 9, 2024

Copy link
Copy Markdown
Contributor

Description

orttrainingtestdatascus has only save mnist whose size is only 64M in Azure File
To meet security requirements and reduce maintenance cost, move the test data to lotusscus and saved in Azure blob.

@mszhanyi
Yi Zhang (mszhanyi) requested a review from a team May 9, 2024 07:22
@mszhanyi Yi Zhang (mszhanyi) changed the title Migrate training blob storage from SAS to managed identity Migrate training storage from SAS to managed identity May 9, 2024
Yi Zhang added 2 commits May 9, 2024 21:43
@mszhanyi

Copy link
Copy Markdown
Contributor Author

/azp run orttraining-ortmodule-distributed

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@snnn snnn (snnn) 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.

We will include it in the next patch release.

@snnn

Copy link
Copy Markdown
Contributor

/azp run orttraining-linux-gpu-ci-pipeline

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@snnn
snnn (snnn) merged commit 5a18818 into main May 9, 2024
@snnn
snnn (snnn) deleted the zhanyi/blob branch May 9, 2024 22:44
@sophies927 Sophie Schoenmeyer (sophies927) added the triage:approved Approved for cherrypicks for release label Jun 12, 2024
Yifan Li (yf711) pushed a commit that referenced this pull request Jun 18, 2024
### Description
orttrainingtestdatascus has only save mnist whose size is only 64M in
Azure File
To meet security requirements and reduce maintenance cost, move the test
data to lotusscus and saved in Azure blob.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:1.18.1 triage:approved Approved for cherrypicks for release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants