Skip to content

Update howto-network-fabric-quality-of-service.md#4340

Open
rkaraddi wants to merge 3 commits into
MicrosoftDocs:mainfrom
rkaraddi:rkaraddi/qosDocChanges
Open

Update howto-network-fabric-quality-of-service.md#4340
rkaraddi wants to merge 3 commits into
MicrosoftDocs:mainfrom
rkaraddi:rkaraddi/qosDocChanges

Conversation

@rkaraddi

Copy link
Copy Markdown

Fixing errors in the QoS doc

@prmerger-automator

Copy link
Copy Markdown
Contributor

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

@prmerger-automator prmerger-automator Bot requested a review from dougbristow June 18, 2026 22:39
@prmerger-automator

Copy link
Copy Markdown
Contributor

@rkaraddi : 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 084e9fa:

✅ Validation status: passed

File Status Preview URL Details
operator-nexus/howto-network-fabric-quality-of-service.md ✅Succeeded

For more details, please refer to the build report.

@bartpinto bartpinto 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.

A few recommendations below.

Comment thread operator-nexus/howto-network-fabric-quality-of-service.md Outdated
Comment thread operator-nexus/howto-network-fabric-quality-of-service.md
Comment thread operator-nexus/howto-network-fabric-quality-of-service.md Outdated
Comment thread operator-nexus/howto-network-fabric-quality-of-service.md

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

This PR updates the Azure Operator Nexus QoS documentation to correct the Azure CLI commands used to lock a fabric configuration and commit configuration changes.

Changes:

  • Replaced the configuration lock example with the newer az networkfabric fabric lock-fabric command and parameters.
  • Replaced the commit example with az networkfabric fabric commit-configuration.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread operator-nexus/howto-network-fabric-quality-of-service.md Outdated
Comment thread operator-nexus/howto-network-fabric-quality-of-service.md Outdated
@Court72

Court72 commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

@dougbristow

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 Jun 19, 2026
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit f1d8c94:

✅ Validation status: passed

File Status Preview URL Details
operator-nexus/howto-network-fabric-quality-of-service.md ✅Succeeded

For more details, please refer to the build report.

@prmerger-automator

Copy link
Copy Markdown
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 8483519:

✅ Validation status: passed

File Status Preview URL Details
operator-nexus/howto-network-fabric-quality-of-service.md ✅Succeeded

For more details, please refer to the build report.

@prmerger-automator

Copy link
Copy Markdown
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

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.

5 participants