Skip to content

Note added in NFS documentation - #475

Open
ffidelis wants to merge 4 commits into
MicrosoftDocs:mainfrom
ffidelis:create-volume-azure-files-1
Open

Note added in NFS documentation#475
ffidelis wants to merge 4 commits into
MicrosoftDocs:mainfrom
ffidelis:create-volume-azure-files-1

Conversation

@ffidelis

Copy link
Copy Markdown
Contributor

Changed the NFS documentation to add a note about the Storage Account "Secure transfer required" parameter should be disabled

Updated the date for the article and corrected a typo in the note about NFS protocol usage.
@prmerger-automator

Copy link
Copy Markdown
Contributor

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

@prmerger-automator

Copy link
Copy Markdown
Contributor

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

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 7a8a4e0:

✅ Validation status: passed

File Status Preview URL Details
articles/aks/create-volume-azure-files.md ✅Succeeded

For more details, please refer to the build report.

Comment thread articles/aks/create-volume-azure-files.md Outdated
@v-dirichards

Copy link
Copy Markdown
Contributor

Erin Schaffer (@schaffererin)

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

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

Updates the AKS Azure Files documentation to warn readers about a required storage account setting when using Azure Files NFS, helping prevent NFS mount failures.

Changes:

  • Updated the article date.
  • Added a NOTE under the NFS prerequisites calling out the Secure transfer required setting requirement.

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

Comment on lines +645 to +646
> [!NOTE]
> To proper use the NFS protocol natively in Azure Kubernetes Serivce the Storage Account must have the "**Secure transfer required**" parameter **disabled**. Otherwise, the volume mount operation will fail.
Co-authored-by: Diana Richards <103777760+v-dirichards@users.noreply.github.com>
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 30a8489:

❌ Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
❌Error Details

  • [Error: CannotMergeCommit] Cannot merge commit 30a84897e362ccd6cb30c7ab8749573eb23a0edf in branch create-volume-azure-files-1 of repository https://github.com/ffidelis/azure-aks-docs into branch main (commit ee3ed13fc70d94d7bcf3e99e462545fd7f4f6e1f). Please follow this documentation: https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line/ to use git.exe to resolve you content conflicts locally and then push to remote.

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.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 69e1d84:

✅ Validation status: passed

File Status Preview URL Details
articles/aks/create-volume-azure-files.md ✅Succeeded

For more details, please refer to the build report.

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