Azure-docs: Couple of issue / comments

[Digiroka]

In section 2 The "-name" parameter does not exist for join-AzStorageAccountForAuth, it should be "-StorageAccountName"

In section 2 - please elaborate that the AD distinguished name attribute is not supported for the parameter "-OrganizationalUnitName", and just the name of the OU should be used. This may seem obvious, but in most scenarios the distinguished name is used, so I thought I had an AD replication issue!

---

Document Details

⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

• ID: fba964be-3f77-d80c-f1a5-35bb1430bb8f
• Version Independent ID: fe70b11e-10fe-5a8c-1fad-b22a83693503
• Content: Enable Active Directory authentication over SMB for Azure Files
• Content Source: articles/storage/files/storage-files-identity-auth-active-directory-enable.md
• Service: storage
• GitHub Login: @roygara
• Microsoft Alias: rogarana

[SumanthMarigowda-MSFT]

@Digiroka Thanks for your question. We are checking on this and will respond to you soon.

[SumanthMarigowda-MSFT]

@Digiroka Firstly, apologies for the delay in responding here and any inconvenience this issue may have caused. Your feedback has been shared with the content owner for further review.

[OffColour]

It should use Distinguished Name otherwise it'll just go to the first OU it finds that matches than name when there could be multiple.

[JohnLBevan]

Note: There is an OrganizationalUnitDistringuishedName parameter that can be used if you wish to target an explicit OU by DN.

[Digiroka]

Note: There is an OrganizationalUnitDistringuishedName parameter that can be used if you wish to target an explicit OU by DN.

Surprised I missed that - very useful, thank you.

[pafnow]

This part of the script is really unclear and should be fixed

[roygara]

@Digiroka Sorry for the delayed response but thanks for bringing these to our attention.

We've gone ahead and corrected both issues, I've changed the script to just use the OrganizationalUnitDistinguishedName parameter by default. Since that seemed optimal.

The changes will be live later today, accordingly, I'll close this out.

Thanks for helping us improve our docs, we greatly appreciate it!

please-close