Skip to content

Commit

Permalink
[renumber-topo] Add extra mgmt ip to ptf redeploy step (#9230)
Browse files Browse the repository at this point in the history
What is the motivation for this PR?
If a testbed supports extra mgmt IP then the PTF gets the extra IP when add-topo is run. This change was done as part of #8432
However, if PTF redeploy is done, the extra IP added by add-topo is lost as PTF is recreated.
This issue is fixed in this PR by adding extra mgmt IP support in renumber_topo ansible playbook.

How did you do it?

How did you verify/test it?
Tested on physical testbed.
  • Loading branch information
vaibhavhd authored and mssonicbld committed Nov 7, 2024
1 parent 442a8a3 commit 3da9e10
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions ansible/roles/vm_set/tasks/renumber_topo.yml
Original file line number Diff line number Diff line change
Expand Up @@ -150,6 +150,7 @@
vm_type: "{{ vm_type }}"
vm_properties: "{{ vm_properties if vm_properties is defined else omit }}"
ptf_mgmt_ip_addr: "{{ ptf_ip }}"
ptf_extra_mgmt_ip_addr: "{{ ptf_extra_mgmt_ip.split(',') | default([]) }}"
ptf_mgmt_ipv6_addr: "{{ ptf_ipv6 }}"
ptf_mgmt_ip_gw: "{{ mgmt_gw }}"
ptf_mgmt_ipv6_gw: "{{ mgmt_gw_v6 | default(None) }}"
Expand Down

0 comments on commit 3da9e10

Please sign in to comment.