-
Notifications
You must be signed in to change notification settings - Fork 21.8k
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
Conversation
Added new scenerio and example
Learn Build status updates of commit 4110279:
|
File | Status | Preview URL | Details |
---|---|---|---|
articles/storage/common/storage-use-azcopy-blobs-synchronize.md | 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:
- Try searching the learn.microsoft.com contributor guides
- Post your question in the Learn support channel
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" |
@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" |
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 @microsoft-github-policy-service agree company="Microsoft" |
@prkana : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
Can you review the proposed changes? Important: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
There was a problem hiding this 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!
articles/storage/common/storage-use-azcopy-blobs-synchronize.md
Outdated
Show resolved
Hide resolved
articles/storage/common/storage-use-azcopy-blobs-synchronize.md
Outdated
Show resolved
Hide resolved
articles/storage/common/storage-use-azcopy-blobs-synchronize.md
Outdated
Show resolved
Hide resolved
articles/storage/common/storage-use-azcopy-blobs-synchronize.md
Outdated
Show resolved
Hide resolved
articles/storage/common/storage-use-azcopy-blobs-synchronize.md
Outdated
Show resolved
Hide resolved
suggested changes modified
Learn Build status updates of commit a967214:
|
File | Status | Preview URL | Details |
---|---|---|---|
articles/storage/common/storage-use-azcopy-blobs-synchronize.md | 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:
- Try searching the learn.microsoft.com contributor guides
- Post your question in the Learn support channel
@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. |
The necessary response has been provided. Please share any further suggestions. |
@normesta - Would you add a comment to indicate what action to take on this PR? Thanks! |
#sign-off |
Sounds good. Thank you! |
articles/storage/common/storage-use-azcopy-blobs-synchronize.md
Outdated
Show resolved
Hide resolved
articles/storage/common/storage-use-azcopy-blobs-synchronize.md
Outdated
Show resolved
Hide resolved
Learn Build status updates of commit 5abde73:
|
File | Status | Preview URL | Details |
---|---|---|---|
articles/storage/common/storage-use-azcopy-blobs-synchronize.md | 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.
articles/storage/common/storage-use-azcopy-blobs-synchronize.md
Outdated
Show resolved
Hide resolved
Learn Build status updates of commit 1685caa: ✅ Validation status: passed
For more details, please refer to the build report. |
Added new scenerio and example