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

Update expressroute-troubleshooting-network-performance.md #110120

Merged
merged 2 commits into from
Jul 5, 2023

Conversation

dantecit0
Copy link
Contributor

Added note obtained from a recent case about throughput performance over a long distance link (from Denver DC to Virginia ExR circuit). By using the following description, we could obtain in a 2500km distance over 200mbps in a single session by tuning TCP Window. Cannot modify entire values as I don't have an ExpressRoute with 10gbps for testing and update the table accordingly:

[!NOTE] While these numbers should be taken into consideration, they were tested using AzureCT which is based in IPERF in Windows via PowerShell. In this scenario, IPERF does not honor default Windows TCP options for Scaling Factor, and uses a way lower Shift Count for the TCP Window size. The numbers represented here were performed using default IPERF values and are for general reference only. By tuning IPERF commands with "-w" switch and a big TCP Window size, better throughput can be obtained over long distances, showing significantly better throughput figures. Also, to ensure an ExpressRoute is using the full bandwidth, it's ideal to run the IPERF in multi threaded option from multiple machines simultaneously to ensure computing capacity is able to reach maximum link performance and is not limited by processing capacity of a single VM.

Added note obtained from a recent case about throughput performance over a long distance link (from Denver DC to Virginia ExR circuit). By using the following description, we could obtain in a 2500km distance over 200mbps in a single session by tuning TCP Window. Cannot modify entire values as I don't have an ExpressRoute with 10gbps for testing and update the table accordingly:

[!NOTE] While these numbers should be taken into consideration, they were tested using AzureCT which is based in IPERF in Windows via PowerShell. In this scenario, IPERF does not honor default Windows TCP options for Scaling Factor, and uses a way lower Shift Count for the TCP Window size. The numbers represented here were performed using default IPERF values and are for general reference only. By tuning IPERF commands with "-w" switch and a big TCP Window size, better throughput can be obtained over long distances, showing significantly better throughput figures. Also, to ensure an ExpressRoute is using the full bandwidth, it's ideal to run the IPERF in multi threaded option from multiple machines simultaneously to ensure computing capacity is able to reach maximum link performance and is not limited by processing capacity of a single VM.
@prmerger-automator
Copy link
Contributor

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

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 27c9632:

✅ Validation status: passed

File Status Preview URL Details
articles/expressroute/expressroute-troubleshooting-network-performance.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@American-Dipper
Copy link
Contributor

@duongau - can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, add a #sign-off comment 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 May 29, 2023
@duongau
Copy link
Contributor

duongau commented May 31, 2023

Hello @dantecit0, I have passed this along to the PG team to review and get back to me about your addtion. Thank you!

@dantecit0
Copy link
Contributor Author

Hello @dantecit0, I have passed this along to the PG team to review and get back to me about your addtion. Thank you!

Thanks Duong! I have made a pull request also for the AzureCT tool to update the script to include the newer binaries for iPerf here: Azure/NetworkMonitoring#29

If that gets accepted and corrected maybe the note I put is useless but still the table is misleading for reference as numbers would still be wrong.

Thanks!!

@duongau
Copy link
Contributor

duongau commented Jun 1, 2023

@dantecit0, please reach out to Jon Ormand he is the person that owns that NetworkMonitor branch and should be able to merge your change in the other PR.

@duongau
Copy link
Contributor

duongau commented Jun 27, 2023

Hello @dantecit0, have you been able to reach out to Jon Ormond to get that PR merged?

@duongau
Copy link
Contributor

duongau commented Jul 5, 2023

Hi @dantecit0, I'll merge this PR for now. Please let me know when your other PR gets merge and then we can remove this note. #sign-off.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 17dd809:

❌ Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
❌Error Details

  • [Error: GitBranchDeletedOrForcePushed]
Cannot sync git repo to specified commit because ref +refs/pull/110120/merge has been deleted or has been force pushed.
 If this is pull request, please ensure it is mergeable (no merge conflict) and open (not closed).

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.

For any questions, please:

@dantecit0
Copy link
Contributor Author

dantecit0 commented Jul 5, 2023 via email

@prmerger-automator
Copy link
Contributor

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @duongau

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