Skip to content

Update storage-use-azcopy-blobs-synchronize.md #126968

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jun 10, 2025

Conversation

prkana
Copy link
Contributor

@prkana prkana commented May 29, 2025

Added new scenerio and example

Added new scenerio and example
Copy link
Contributor

Learn Build status updates of commit 4110279:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/storage/common/storage-use-azcopy-blobs-synchronize.md ⚠️Warning Details

articles/storage/common/storage-use-azcopy-blobs-synchronize.md

  • Line 128, Column 56: [Warning: hard-coded-locale - See documentation] Link 'https://azure.microsoft.com/en-us/products/storage/storage-explorer' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@prkana
Copy link
Contributor Author

prkana commented May 29, 2025

Line 128, Column 56: [Warning: hard-coded-locale - See documentation] Link 'https://azure.microsoft.com/en-us/products/storage/storage-explorer' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

This is Microsoft official site to download the "Azure Storage Explorer"

@v-regandowner
Copy link
Contributor

v-regandowner commented May 29, 2025

@prkana - Thank you for your contribution. Would you take a moment to sign the Contributor License Agreement (CLA)? After the CLA is signed, someone can review your pull request. Thanks!

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label May 29, 2025
@prkana
Copy link
Contributor Author

prkana commented May 29, 2025

I agree with the agreement.

@microsoft-github-policy-service agree [company="{Microsoft}"]

Options:

• (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

• (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"

@prkana
Copy link
Contributor Author

prkana commented May 29, 2025

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree company="Microsoft"

@prmerger-automator prmerger-automator bot requested a review from normesta May 29, 2025 15:57
Copy link
Contributor

@prkana : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@v-dirichards
Copy link
Contributor

@normesta

Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Copy link
Contributor

@normesta normesta left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution. I added some suggestions for you to review and accept. Question - what motivates the introduction of a SAS section here. To keep things simple and consistent, all examples exclude the token. We have a section in each article which states that. We also have a section in the AzCopy overview which shows an example of using SAS should folks need to. In general, SAS is not the preferred method of auth. That is why we avoid promoting it in examples. If you have some specific business need for including a section here for SAS, feel free. I plan to sweep through this content a bit later in the year and do a better job driving eyeballs to our guidance about SAS so that we don't have to include article-specific SAS examples. At any rate, thank you for showing interest. Please review my suggestions. Cheers!

Copy link
Contributor

Learn Build status updates of commit a967214:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/storage/common/storage-use-azcopy-blobs-synchronize.md ⚠️Warning Details

articles/storage/common/storage-use-azcopy-blobs-synchronize.md

  • Line 128, Column 56: [Warning: hard-coded-locale - See documentation] Link 'https://azure.microsoft.com/en-us/products/storage/storage-explorer' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@prkana
Copy link
Contributor Author

prkana commented May 31, 2025

@normesta I appreciate your support and guidance. Kindly review my justifications.

Question - what motivates the introduction of a SAS section here. To keep things simple and consistent, all examples exclude the token. We have a section in each article which states that. We also have a section in the AzCopy overview which shows an example of using SAS should folks need to. In general, SAS is not the preferred method of auth. That is why we avoid promoting it in examples. If you have some specific business need for including a section here for SAS, feel free. I plan to sweep through this content a bit later in the year and do a better job driving eyeballs to our guidance about SAS so that we don't have to include article-specific SAS examples.

Answer I have encountered multiple cases where I had the opportunity to work with customers using SAS, particularly when they experienced issues with AzCopy or AzCopy sync. In most scenarios, we were able to successfully mitigate these issues when customers used SAS. Customers also reported feeling more comfortable using SAS, especially during operations like AzCopy copy/sync from on-premises, intranet-based AzCopy, or copy/sync from their local machines.

Providing at least one example on this page would make the process more convenient for customers and would also be beneficial for support engineers.

@prkana
Copy link
Contributor Author

prkana commented May 31, 2025

The necessary response has been provided. Please share any further suggestions.

@v-dirichards
Copy link
Contributor

@normesta - Would you add a comment to indicate what action to take on this PR?

Thanks!

@normesta
Copy link
Contributor

#sign-off

@normesta
Copy link
Contributor

@normesta I appreciate your support and guidance. Kindly review my justifications.

Question - what motivates the introduction of a SAS section here. To keep things simple and consistent, all examples exclude the token. We have a section in each article which states that. We also have a section in the AzCopy overview which shows an example of using SAS should folks need to. In general, SAS is not the preferred method of auth. That is why we avoid promoting it in examples. If you have some specific business need for including a section here for SAS, feel free. I plan to sweep through this content a bit later in the year and do a better job driving eyeballs to our guidance about SAS so that we don't have to include article-specific SAS examples.

Answer I have encountered multiple cases where I had the opportunity to work with customers using SAS, particularly when they experienced issues with AzCopy or AzCopy sync. In most scenarios, we were able to successfully mitigate these issues when customers used SAS. Customers also reported feeling more comfortable using SAS, especially during operations like AzCopy copy/sync from on-premises, intranet-based AzCopy, or copy/sync from their local machines.

Providing at least one example on this page would make the process more convenient for customers and would also be beneficial for support engineers.

Sounds good. Thank you!

Copy link
Contributor

Learn Build status updates of commit 5abde73:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/storage/common/storage-use-azcopy-blobs-synchronize.md ⚠️Warning Details

articles/storage/common/storage-use-azcopy-blobs-synchronize.md

  • Line 128, Column 56: [Warning: hard-coded-locale - See documentation] Link 'https://azure.microsoft.com/en-us/products/storage/storage-explorer' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

Copy link
Contributor

Learn Build status updates of commit 1685caa:

✅ Validation status: passed

File Status Preview URL Details
articles/storage/common/storage-use-azcopy-blobs-synchronize.md ✅Succeeded

For more details, please refer to the build report.

@v-dirichards v-dirichards merged commit 4f788b8 into MicrosoftDocs:main Jun 10, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants