Skip to content

[add-topo] Set kernel.pty.max to 8192#19054

Merged
Blueve merged 1 commit intosonic-net:masterfrom
lizhijianrd:set-pty-8192
Jun 19, 2025
Merged

[add-topo] Set kernel.pty.max to 8192#19054
Blueve merged 1 commit intosonic-net:masterfrom
lizhijianrd:set-pty-8192

Conversation

@lizhijianrd
Copy link
Contributor

@lizhijianrd lizhijianrd commented Jun 17, 2025

Description of PR

Summary:
When deploying thousands of ceos and net containers on server, we see below failure due to PTY resource restriction.

Error response from daemon: failed to create task for container: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: open /dev/ptmx: no space left on device: unknown

The kernel.pty.max default value is 4096.

To solve this issue, set kernel.pty.max to 8192 in add-topo ansible-playbook.

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Approach

What is the motivation for this PR?

Support deploy thousands of ceos/net containers on a single server.

How did you do it?

Set kernel.pty.max to 8192 in add-topo ansible-playbook.

How did you verify/test it?

Verified by deploy physical testbed.

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

@mssonicbld
Copy link
Collaborator

/azp run

@lizhijianrd lizhijianrd requested review from Blueve and r12f June 17, 2025 06:36
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Blueve Blueve merged commit 06d855b into sonic-net:master Jun 19, 2025
21 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jun 19, 2025
What is the motivation for this PR?
Support deploy thousands of ceos/net containers on a single server.

How did you do it?
Set kernel.pty.max to 8192 in add-topo ansible-playbook.

How did you verify/test it?
Verified by deploy physical testbed.
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202505: #19087

@lizhijianrd lizhijianrd deleted the set-pty-8192 branch June 19, 2025 02:43
mssonicbld pushed a commit that referenced this pull request Jun 19, 2025
What is the motivation for this PR?
Support deploy thousands of ceos/net containers on a single server.

How did you do it?
Set kernel.pty.max to 8192 in add-topo ansible-playbook.

How did you verify/test it?
Verified by deploy physical testbed.
@lizhijianrd
Copy link
Contributor Author

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to msft-202412: Azure/sonic-mgmt.msft#446

sdszhang pushed a commit to sdszhang/sonic-mgmt that referenced this pull request Jun 30, 2025
What is the motivation for this PR?
Support deploy thousands of ceos/net containers on a single server.

How did you do it?
Set kernel.pty.max to 8192 in add-topo ansible-playbook.

How did you verify/test it?
Verified by deploy physical testbed.
nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Aug 7, 2025
What is the motivation for this PR?
Support deploy thousands of ceos/net containers on a single server.

How did you do it?
Set kernel.pty.max to 8192 in add-topo ansible-playbook.

How did you verify/test it?
Verified by deploy physical testbed.
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
What is the motivation for this PR?
Support deploy thousands of ceos/net containers on a single server.

How did you do it?
Set kernel.pty.max to 8192 in add-topo ansible-playbook.

How did you verify/test it?
Verified by deploy physical testbed.

Signed-off-by: opcoder0 <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
What is the motivation for this PR?
Support deploy thousands of ceos/net containers on a single server.

How did you do it?
Set kernel.pty.max to 8192 in add-topo ansible-playbook.

How did you verify/test it?
Verified by deploy physical testbed.

Signed-off-by: Guy Shemesh <[email protected]>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
What is the motivation for this PR?
Support deploy thousands of ceos/net containers on a single server.

How did you do it?
Set kernel.pty.max to 8192 in add-topo ansible-playbook.

How did you verify/test it?
Verified by deploy physical testbed.

Signed-off-by: Aharon Malkin <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
What is the motivation for this PR?
Support deploy thousands of ceos/net containers on a single server.

How did you do it?
Set kernel.pty.max to 8192 in add-topo ansible-playbook.

How did you verify/test it?
Verified by deploy physical testbed.

Signed-off-by: Guy Shemesh <[email protected]>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
What is the motivation for this PR?
Support deploy thousands of ceos/net containers on a single server.

How did you do it?
Set kernel.pty.max to 8192 in add-topo ansible-playbook.

How did you verify/test it?
Verified by deploy physical testbed.
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
What is the motivation for this PR?
Support deploy thousands of ceos/net containers on a single server.

How did you do it?
Set kernel.pty.max to 8192 in add-topo ansible-playbook.

How did you verify/test it?
Verified by deploy physical testbed.

Signed-off-by: Guy Shemesh <[email protected]>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
What is the motivation for this PR?
Support deploy thousands of ceos/net containers on a single server.

How did you do it?
Set kernel.pty.max to 8192 in add-topo ansible-playbook.

How did you verify/test it?
Verified by deploy physical testbed.

Signed-off-by: Yael Tzur <[email protected]>
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