Skip to content

Update deployment-virtual.md - adjust system requirements #3830

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Update deployment-virtual.md - adjust system requirements #3830

wants to merge 10 commits into from

Conversation

Karl-WE
Copy link
Contributor

@Karl-WE Karl-WE commented Sep 2, 2025 •
edited
Loading

  1. consolidating virtual disks which was partly 6 and partly 4.

  2. the new requirements allow deployments with only 2 disks.

  3. Changes to network adapters so that the EnvironmentChecker will not fail.

  4. Raising RAM requirements to comply with new system requirements for Azure Local configurator app.

@Karl-WE
Update deployment-virtual.md
782f4e8 
1. consolidating virtual disks which was partly 6 and partly 4.

2. the new requirements allow deployments with only 2 disks.
@prmerger-automator prmerger-automator bot requested a review from alkohli September 2, 2025 12:12
@prmerger-automator
Copy link
Contributor

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

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 782f4e8:

✅ Validation status: passed

File Status Preview URL Details
azure-local/deploy/deployment-virtual.md ✅Succeeded

For more details, please refer to the build report.

@Karl-WE Karl-WE changed the title Update deployment-virtual.md Update deployment-virtual.md - adjust system requirements Sep 2, 2025
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 3040502:

✅ Validation status: passed

File Status Preview URL Details
azure-local/deploy/deployment-virtual.md ✅Succeeded

For more details, please refer to the build report.

@Karl-WE
Update deployment-virtual.md
be9f8cd 
rename Azure Stack HCI to Azure Local (even though the OS is not renamed but it is so confusing)
corrected information about number of disks as I forgot to mention and count OS disk in previous PR.
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit be9f8cd:

✅ Validation status: passed

File Status Preview URL Details
azure-local/deploy/deployment-virtual.md ✅Succeeded

For more details, please refer to the build report.

@v-dirichards
Copy link
Contributor

@alkohli

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

@prmerger-automator prmerger-automator bot added the aq-pr-triaged Tracking label for the PR review team label Sep 2, 2025
@Karl-WE
Copy link
Contributor Author

Karl-WE commented Sep 22, 2025

Hi @alkohli any news on this?

@Karl-WE
Aktualisieren von deployment-virtual.md
5ee0e4a 
Bold text,because validation will fail when not set. The idea was that it's only necessary on adapters when teamed but that's not the case.
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 5ee0e4a:

✅ Validation status: passed

File Status Preview URL Details
azure-local/deploy/deployment-virtual.md ✅Succeeded

For more details, please refer to the build report.

Karl-WE
Karl-WE commented Sep 22, 2025
View reviewed changes
azure-local/deploy/deployment-virtual.md
| Virtual machine (VM) type | Secure Boot and Trusted Platform Module (TPM) enabled. |
| vCPUs | Four cores. |
| Memory | A minimum of 24 GB. |
| Memory | A minimum of 32 GB. |
Copy link
Contributor Author

Choose a reason for hiding this comment

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

To comply with latest HW requirements as per docs
AzL configurator app will hardcheck for 32 GB.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 995c1a6:

✅ Validation status: passed

File Status Preview URL Details
azure-local/deploy/deployment-virtual.md ✅Succeeded

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 1c3b57f:

✅ Validation status: passed

File Status Preview URL Details
azure-local/deploy/deployment-virtual.md ✅Succeeded

For more details, please refer to the build report.

Karl-WE
Karl-WE commented Sep 22, 2025
View reviewed changes
Copy link
Contributor Author

@Karl-WE Karl-WE left a comment

Choose a reason for hiding this comment

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

Renamed Azure Local OS back to Azure Stack HCI. (search and replace error)

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 765bd02:

✅ Validation status: passed

File Status Preview URL Details
azure-local/deploy/deployment-virtual.md ✅Succeeded

For more details, please refer to the build report.

Karl-WE
Karl-WE commented Sep 22, 2025
View reviewed changes
Copy link
Contributor Author

@Karl-WE Karl-WE left a comment

Choose a reason for hiding this comment

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

Changed note for support statement

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 7ba1c1a:

✅ Validation status: passed

File Status Preview URL Details
azure-local/deploy/deployment-virtual.md ✅Succeeded

For more details, please refer to the build report.

@Karl-WE
Aktualisieren von deployment-virtual.md
bbbf593 
Changes to system requirements table
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit bbbf593:

✅ Validation status: passed

File Status Preview URL Details
azure-local/deploy/deployment-virtual.md ✅Succeeded

For more details, please refer to the build report.

ronmiab
ronmiab reviewed Oct 13, 2025
View reviewed changes
ronmiab
ronmiab reviewed Oct 13, 2025
View reviewed changes
azure-local/deploy/deployment-virtual.md
Comment on lines 24 to 25
>[!NOTE]
>Microsoft Support doesn't support virtual deployments.
Copy link
Contributor

Choose a reason for hiding this comment

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

@Karl-WE - add space between > on both lines

Copy link
Contributor Author

Choose a reason for hiding this comment

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

please re-review my edit. TY.

ronmiab
ronmiab reviewed Oct 13, 2025
View reviewed changes
ronmiab
ronmiab reviewed Oct 13, 2025
View reviewed changes
azure-local/deploy/deployment-virtual.md
```
1. Create extra drives to be used as the boot disk and hard disks for Storage Spaces Direct. After these commands are executed, six new VHDXs will be created in the `C:\vms\Node1` directory as shown in this example:
1. Create extra drives to be used as one for boot disk and two hard disks for Storage Spaces Direct. After these commands are executed, three new VHDXs will be created in the `C:\vms\Node1` directory as shown in this example:
Copy link
Contributor

Choose a reason for hiding this comment

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

@Karl-WE - suggested rewrite:

Create extra drives to be used as one for the boot disk and two hard disks for Storage Spaces Direct.

  • After you run these commands, as shown in the example, three new VHDXs appear in the C:\vms\Node1 directory.

ronmiab
ronmiab reviewed Oct 13, 2025
View reviewed changes
Copy link
Contributor

@ronmiab ronmiab left a comment

Choose a reason for hiding this comment

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

Hi @Karl-WE - Great work! Here's my feedback.

@Karl-WE
Copy link
Contributor Author

Karl-WE commented Oct 15, 2025

@ronmiab thanks for your comments. Do I understand your comments correctly that I am supposed to incorporate your great suggestions or are these done already?

Thanks!

@ronmiab
Copy link
Contributor

ronmiab commented Oct 15, 2025

@Karl-WE - I didn't make any updates directly because I wanted to let you review my feedback and decide if you're good with it. Kindly incorporate all suggestions that you're good with. Thanks!

@Karl-WE
Revise deployment instructions for Azure Local
10533cd 
Updated the title and improved clarity in the instructions regarding MAC address spoofing and teaming for network adapters.

added space after pipes |
@Karl-WE
Copy link
Contributor Author

Karl-WE commented Oct 16, 2025 •
edited
Loading

@ronmiab @alkohli I have done edits and agree with the suggestions made in the review of Ronmia. Please have another final read and hope we are good to signoff. As this took quite some time to edit, hope we are not running out of sync. Thanks for considering the priority based on that.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 10533cd:

✅ Validation status: passed

File Status Preview URL Details
azure-local/deploy/deployment-virtual.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

Reviewers

@alkohli alkohli Awaiting requested review from alkohli

1 more reviewer

@ronmiab ronmiab ronmiab left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@alkohli alkohli

Labels

aq-pr-triaged Tracking label for the PR review team azure-local/svc Change sent to author do-not-merge

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Karl-WE @v-dirichards @ronmiab @alkohli