Skip to content

Update USP docs under keepalive connections for better readability#2168

Open
Morgan-P-04 wants to merge 1 commit into
ngf-release-2.7from
ngf/usp-keepalive-default-amended
Open

Update USP docs under keepalive connections for better readability#2168
Morgan-P-04 wants to merge 1 commit into
ngf-release-2.7from
ngf/usp-keepalive-default-amended

Conversation

@Morgan-P-04

Copy link
Copy Markdown

Proposed changes

This PR updates the Enable keepalive connections section of the Upstream Settings Policy API to improve the readability for users. The change re-orders where the default value for the keepalive directive is mentioned so that users do not have to switch context as much, to align with a suggestion from the team: #2160 (comment)

Checklist

Before sharing this pull request, I completed the following checklist:

Footnotes

  1. Potentially sensitive information includes personally identify information (PII), authentication credentials, and live URLs. Refer to the style guide for guidance about placeholder content.

@Morgan-P-04
Morgan-P-04 requested a review from a team July 24, 2026 10:13
@Morgan-P-04
Morgan-P-04 requested a review from a team as a code owner July 24, 2026 10:13
@github-actions github-actions Bot added documentation Improvements or additions to documentation product/ngf Issues related to NGINX Gateway Fabric labels Jul 24, 2026
@github-actions

Copy link
Copy Markdown

Deploy Preview will be available once build job completes!

Name Link
😎 Deploy Preview https://frontdoor-test-docs.nginx.com/previews/docs/2168/

@bjee19

bjee19 commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Just for the sake of the guide, since the default nginx keepalive is 32, can we adjust the guide example so the USP isn't setting the keepalive to 32, but instead perhaps 24?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation product/ngf Issues related to NGINX Gateway Fabric

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants