From 69c933c4fe4852d7c12763b93da4deaf1305c350 Mon Sep 17 00:00:00 2001
From: Dashuai Zhang <164845223+sdszhang@users.noreply.github.com>
Date: Tue, 8 Oct 2024 12:38:07 +1100
Subject: [PATCH] change ptf container memory in restart_ptf to 16G which is
 same as add-topo. (#14841)

Description of PR
Summary:
PTF container's memory limit is 16G in add-topo after the change in #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: jianquanye@microsoft.com
---
 ansible/roles/vm_set/tasks/renumber_topo.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/ansible/roles/vm_set/tasks/renumber_topo.yml b/ansible/roles/vm_set/tasks/renumber_topo.yml
index 8d471407264..2c8d22e8f99 100644
--- a/ansible/roles/vm_set/tasks/renumber_topo.yml
+++ b/ansible/roles/vm_set/tasks/renumber_topo.yml
@@ -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 }}