Skip to content

Commit 8d9ec3c

Browse files
authored
Fix the issue introduced by the PR #16716 (#17516)
Description of PR Summary: After the #16716 merged, the remove topo failed. fix it in this PR Fixes # (issue) Approach What is the motivation for this PR? How did you do it? How did you verify/test it? Deploy the setup could pass with the fix co-authorized by: jianquanye@microsoft.com
1 parent 2edf5b1 commit 8d9ec3c

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

ansible/roles/vm_set/tasks/get_dut_port.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,6 @@
5454

5555
- set_fact:
5656
duts_fp_ports: "{{ duts_fp_ports|default({}) | combine( { dut_name: dut_fp_ports } ) }}"
57-
duts_midplane_ports: "{{ duts_midplane_ports|default({}) | combine( { dut_name: dut_midplane_ports } ) }}"
58-
duts_inband_ports: "{{ duts_inband_ports|default({}) | combine( { dut_name: dut_inband_ports } ) }}"
57+
duts_midplane_ports: "{{ duts_midplane_ports|default({}) | combine( { dut_name: dut_midplane_ports|default([]) } ) }}"
58+
duts_inband_ports: "{{ duts_inband_ports|default({}) | combine( { dut_name: dut_inband_ports|default([]) } ) }}"
5959
duts_mgmt_port: "{{ duts_mgmt_port|default([]) + [ dut_mgmt_port ] }}"

0 commit comments

Comments
 (0)