Skip to content

[DOC-1361][2025.2.4][2026.1] Add flags for CM ctrl conn creds and socket backlog size#32144

Open
vdamle-yb wants to merge 2 commits into
yugabyte:masterfrom
vdamle-yb:cm-flags-add
Open

[DOC-1361][2025.2.4][2026.1] Add flags for CM ctrl conn creds and socket backlog size#32144
vdamle-yb wants to merge 2 commits into
yugabyte:masterfrom
vdamle-yb:cm-flags-add

Conversation

@vdamle-yb

@vdamle-yb vdamle-yb commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

This doc change adds descriptions in ycm-setup.md for the following 3 flags:

  • ysql_conn_mgr_internal_conn_db
  • ysql_conn_mgr_internal_conn_user
  • ysql_conn_mgr_internal_socket_listen_backlog

These flags are present in 2025.2 (release 2025.2.4 onwards) and 2026.1 (2026.1.0 onward) as well.

@vdamle-yb vdamle-yb requested review from aishwarya24 and ddhodge June 11, 2026 18:47

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request adds documentation for three new YSQL Connection Manager flags: ysql_conn_mgr_internal_conn_db, ysql_conn_mgr_internal_conn_user, and ysql_conn_mgr_socket_listen_backlog. A review comment notes a discrepancy between the documented flag name ysql_conn_mgr_socket_listen_backlog and the name ysql_conn_mgr_internal_tcp_listen_backlog used in the PR description, which needs to be verified and corrected.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

| ysql_conn_mgr_reserve_internal_conns | The number of physical connections to reserve for internal operations, out of the total number of connections (per node) as set using [ysql_max_connections](../../../reference/configuration/yb-tserver/#ysql-max-connections). The reserved connections bypass YSQL Connection Manager; the remaining connections are available for the Connection Manager pool. For example, if `ysql_max_connections` is 300 and this flag is set to 15, YSQL Connection Manager will have a physical connection limit of 285 (300 - 15) per node.<br>Default: 15. v2025.2.1.0 and later only. |
| ysql_conn_mgr_internal_conn_db | Database to which YSQL Connection Manager connects when creating internal database connections.<br>Default: yugabyte |
| ysql_conn_mgr_internal_conn_user | Username used by YSQL Connection Manager when creating internal database connections.<br>Default: yugabyte |
| ysql_conn_mgr_socket_listen_backlog | Maximum number of pending TCP connections queued by the kernel on Connection Manager's listening socket (the backlog argument to `listen(2)`). Incoming connections beyond this limit may be refused or dropped during connection bursts.<br>Default: 128 |

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.

medium

There is a discrepancy between the flag name documented here (ysql_conn_mgr_socket_listen_backlog) and the one listed in the PR description (ysql_conn_mgr_internal_tcp_listen_backlog). Please verify which flag name is correct in the codebase and update the documentation or PR description accordingly.

References
  1. Anything that contradicts a comment, docstring, or commit message in the same diff should be flagged. (link)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated the PR description

@netlify

netlify Bot commented Jun 11, 2026

Copy link
Copy Markdown

Deploy Preview for infallible-bardeen-164bc9 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 79ad44e
🔍 Latest deploy log https://app.netlify.com/projects/infallible-bardeen-164bc9/deploys/6a2b02a75d29440008688c6f
😎 Deploy Preview https://deploy-preview-32144--infallible-bardeen-164bc9.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Jun 11, 2026

Copy link
Copy Markdown

Deploy Preview for infallible-bardeen-164bc9 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit b8fbe16
🔍 Latest deploy log https://app.netlify.com/projects/infallible-bardeen-164bc9/deploys/6a2b02f627de46000838fe27
😎 Deploy Preview https://deploy-preview-32144--infallible-bardeen-164bc9.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant