Skip to content

Commit 1e5d7b0

Browse files
hjensasbshewale
authored andcommitted
[reproducer] Fix MAC address corruption in bm_info
The `_with_nics` variable already evaluates to a Python dict from the `combine()` call. Piping it through `| from_yaml` caused Ansible to serialize it as a YAML string and re-parse it. PyYAML's YAML 1.1 sexagesimal rule then corrupted **all-numeric** MAC addresses (e.g. `41:13:61:22:00:02`) into integers (`41136122002`). Remove the redundant `| from_yaml` to prevent the lossy round-trip. Closes: OSPRH-31852 Assisted-By: Claude (claude-4.6-sonnet-medium) Signed-off-by: Harald Jensås <hjensas@redhat.com>
1 parent fc87553 commit 1e5d7b0

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

roles/reproducer/tasks/generate_bm_info.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@
108108
libvirt_manager_bm_info_data: >-
109109
{{
110110
libvirt_manager_bm_info_data | default({}) |
111-
combine((_with_nics | from_yaml), recursive=true) |
111+
combine(_with_nics, recursive=true) |
112112
combine((_oringal_cifmw_baremetal_hosts | default({}) | from_yaml), recursive=true)
113113
}}
114114
loop: "{{ cifmw_libvirt_manager_uuids | dict2items }}"

0 commit comments

Comments
 (0)