Skip to content
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.

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

Add more details for ThrottleRateActionBitsPerSecond parameter in New-NetQosPolicy #3839

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

YuxuanZhuAtMS
Copy link

PR Summary

Add more details for ThrottleRateActionBitsPerSecond parameter in New-NetQosPolicy for some extreme conditions.

We're using New-NetQosPolicy in production to choke some application by setting ThrottleRateActionBitsPerSecond as low as possible, but it turns out that:

  • 0 means no rate limiting, will be ignored
  • the value will be rounded down to nearest multiple of 8 (e.g. 500 -> 496)

Adding these details into documentation to avoid confusion for others.

PR Checklist

  • Descriptive Title: This PR's title is a synopsis of the changes it proposes.
  • Summary: This PR's summary describes the scope and intent of the change.
  • Contributor's Guide: I have read the contributors guide.
  • Style: This PR adheres to the style guide.

@officedocspr2
Copy link
Collaborator

Thanks for submitting your pull request. To improve the quality of your content and make sure all errors are resolved, we recommend that you use the PubOps publishing process.

  1. Add the Sign off label when you think your PR is ready to merge. A PR reviewer either merges it, or adds the Back to Submitter label with a comment about what needs to be fixed and sends it back to you.

  2. If you need to update your content per the PR reviewer feedback, add the Sign off label again.

  3. Your content is published live during one of the next publishing cycles, which are weekdays at 10:30 AM Pacific Time and 3:30 PM Pacific Time.

If you have a question, post it on the Ask an Admin support channel.

Copy link
Contributor

Learn Build status updates of commit f2edcf8:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2022-ps/netqos/New-NetQosPolicy.md ✅Succeeded View (WindowsServer2022-ps)

For more details, please refer to the build report.

For any questions, please:

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.

None yet

2 participants