MicrosoftDocs / azure-docs

Open source documentation of Microsoft Azure
https://docs.microsoft.com/azure
Creative Commons Attribution 4.0 International
9.98k stars 20.95k forks source link

Update configure-azure-elastic-san.md #121966

Open DP-Miller opened 1 week ago

DP-Miller commented 1 week ago

Change made to the text concerning, how to remove an ESAN VMFS datastore. Instructions require a graphic to ensure this process is more intuitive.

prmerger-automator[bot] commented 1 week ago

@DP-Miller : Thanks for your contribution! The author(s) have been notified to review your proposed change.

learn-build-service-prod[bot] commented 1 week ago

Learn Build status updates of commit a0d2381:

:warning: Validation status: warnings

File Status Preview URL Details
articles/azure-vmware/configure-azure-elastic-san.md :warning:Warning Details

articles/azure-vmware/configure-azure-elastic-san.md

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:

Court72 commented 1 week ago

@jjaygbay1

Can you review the proposed changes? Note that the validation warning must be resolved before merging.

When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

label:"aq-pr-triaged"

@MicrosoftDocs/public-repo-pr-review-team

learn-build-service-prod[bot] commented 1 week ago

Learn Build status updates of commit 6973580:

:white_check_mark: Validation status: passed

File Status Preview URL Details
articles/azure-vmware/configure-azure-elastic-san.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

prmerger-automator[bot] commented 1 week ago

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
Image File This PR added or updated an image file(s).
learn-build-service-prod[bot] commented 1 week ago

Learn Build status updates of commit 2273a55:

:x: Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
articles/azure-vmware/configure-azure-elastic-san.md :x:Error Details

articles/azure-vmware/configure-azure-elastic-san.md

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:

learn-build-service-prod[bot] commented 1 week ago

Learn Build status updates of commit d426931:

:white_check_mark: Validation status: passed

File Status Preview URL Details
articles/azure-vmware/configure-azure-elastic-san.md :white_check_mark:Succeeded
articles/azure-vmware/media/configure-azure-elastic-san/esan-datastorelist-ellipsis-removal.png :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

prmerger-automator[bot] commented 1 week ago

PRMerger Results

Issue Description
Added File(s) This PR contains added files. New files require human review.
Image File This PR added or updated an image file(s).
jjaygbay1 commented 3 days ago

sign-off

Court72 commented 3 days ago

Thanks, @jjaygbay1!

@DP-Miller, before we can merge this PR, the following changes need to be made to the new image filename:

articles/azure-vmware/media/configure-azure-elastic-san/esan-datastorelist-ellipsis-removal.png

Thanks!

hold-off