Skip to content

Commit

Permalink
change ptf container memory in restart_ptf to 16G which is same as ad…
Browse files Browse the repository at this point in the history
…d-topo. (sonic-net#14841)

Description of PR
Summary:
PTF container's memory limit is 16G in add-topo after the change in sonic-net#7413.
But it's still 8G when doing restart-ptf.

In current code base:

:~$ docker stats --no-stream | grep ptf
b303f551ff99   ptf_ptf-01            121.43%   1.26GiB / 16GiB     7.88%     0B / 0B   0B / 618kB        69

# after restart-ptf

:~$ docker stats --no-stream | grep ptf
1c4e10810aff   ptf_ptf-01            0.09%     21.98MiB / 8GiB     0.27%     0B / 0B   0B / 0B           4
:~$ 

Approach
What is the motivation for this PR?
Keep the same memory limit for add-topo and restart-ptf

How did you do it?
How did you verify/test it?
Tested in physical testbed. PTF memory stays at 16G after restart-ptf

co-authorized by: [email protected]
  • Loading branch information
sdszhang authored Oct 8, 2024
1 parent b781bad commit 69c933c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions ansible/roles/vm_set/tasks/renumber_topo.yml
Original file line number Diff line number Diff line change
Expand Up @@ -118,8 +118,8 @@
capabilities:
- net_admin
privileged: yes
memory: 8G
memory_swap: 8G
memory: 16G
memory_swap: 32G
become: yes

- name: Enable ipv6 for docker container ptf_{{ vm_set_name }}
Expand Down

0 comments on commit 69c933c

Please sign in to comment.