)]}'
{"ansible/network.yml":[{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"9c905a644ccbf0f457250c0f80a271cf9f49b241","unresolved":true,"context_lines":[{"line_number":19,"context_line":"          with_items: \"{{ ether_interfaces }}\""},{"line_number":20,"context_line":"          when: not item | net_interface"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"        - name: Validate network interface name length"},{"line_number":23,"context_line":"          fail:"},{"line_number":24,"context_line":"            msg: \u003e"},{"line_number":25,"context_line":"              Network interface validation failed - interface name {{ item |"},{"line_number":26,"context_line":"              net_interface }} is longer than 15 characters."},{"line_number":27,"context_line":"          with_items: \"{{ ether_interfaces + bridge_interfaces + bond_interfaces }}\""},{"line_number":28,"context_line":"          when: item | net_interface | length \u003e 15"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"        - name: Validate bridge interface configuration"},{"line_number":31,"context_line":"          fail:"},{"line_number":32,"context_line":"            msg: \u003e"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"61721951_0c1c9994","line":29,"range":{"start_line":22,"start_character":0,"end_line":29,"end_character":0},"updated":"2026-10-16 11:49:30.000000000","message":"move it first in this block?","commit_id":"800a269c0f1887bf33d8499a6a44c291f6f45c3f"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"11f9c267d92f17b6bf71218236d5e08853da6402","unresolved":true,"context_lines":[{"line_number":19,"context_line":"          with_items: \"{{ ether_interfaces }}\""},{"line_number":20,"context_line":"          when: not item | net_interface"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"        - name: Validate network interface name length"},{"line_number":23,"context_line":"          fail:"},{"line_number":24,"context_line":"            msg: \u003e"},{"line_number":25,"context_line":"              Network interface validation failed - interface name {{ item |"},{"line_number":26,"context_line":"              net_interface }} is longer than 15 characters."},{"line_number":27,"context_line":"          with_items: \"{{ ether_interfaces + bridge_interfaces + bond_interfaces }}\""},{"line_number":28,"context_line":"          when: item | net_interface | length \u003e 15"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"        - name: Validate bridge interface configuration"},{"line_number":31,"context_line":"          fail:"},{"line_number":32,"context_line":"            msg: \u003e"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"951781ad_e97237a6","line":29,"range":{"start_line":22,"start_character":0,"end_line":29,"end_character":0},"in_reply_to":"3dc663dd_69bf5e04","updated":"2026-10-16 12:52:19.000000000","message":"The prior task checks that there is an interface configured at all.","commit_id":"800a269c0f1887bf33d8499a6a44c291f6f45c3f"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"3956825d7c5efae460a3b3a5ab4b5423712857dc","unresolved":true,"context_lines":[{"line_number":19,"context_line":"          with_items: \"{{ ether_interfaces }}\""},{"line_number":20,"context_line":"          when: not item | net_interface"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"        - name: Validate network interface name length"},{"line_number":23,"context_line":"          fail:"},{"line_number":24,"context_line":"            msg: \u003e"},{"line_number":25,"context_line":"              Network interface validation failed - interface name {{ item |"},{"line_number":26,"context_line":"              net_interface }} is longer than 15 characters."},{"line_number":27,"context_line":"          with_items: \"{{ ether_interfaces + bridge_interfaces + bond_interfaces }}\""},{"line_number":28,"context_line":"          when: item | net_interface | length \u003e 15"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"        - name: Validate bridge interface configuration"},{"line_number":31,"context_line":"          fail:"},{"line_number":32,"context_line":"            msg: \u003e"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"604e76ab_eeffb989","line":29,"range":{"start_line":22,"start_character":0,"end_line":29,"end_character":0},"in_reply_to":"47efa67d_17c36493","updated":"2026-10-21 15:49:27.000000000","message":"I see Maksim\u0027s point: Doesn\u0027t `item | net_interface | length \u003e 15` fail if net_interface returns none and net_interface is only guaranteed to be none after the other validation tasks in this block (for bridges and bonds at least)","commit_id":"800a269c0f1887bf33d8499a6a44c291f6f45c3f"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"abd1fef2177f96b36b96d382598e7651fbb6a041","unresolved":true,"context_lines":[{"line_number":19,"context_line":"          with_items: \"{{ ether_interfaces }}\""},{"line_number":20,"context_line":"          when: not item | net_interface"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"        - name: Validate network interface name length"},{"line_number":23,"context_line":"          fail:"},{"line_number":24,"context_line":"            msg: \u003e"},{"line_number":25,"context_line":"              Network interface validation failed - interface name {{ item |"},{"line_number":26,"context_line":"              net_interface }} is longer than 15 characters."},{"line_number":27,"context_line":"          with_items: \"{{ ether_interfaces + bridge_interfaces + bond_interfaces }}\""},{"line_number":28,"context_line":"          when: item | net_interface | length \u003e 15"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"        - name: Validate bridge interface configuration"},{"line_number":31,"context_line":"          fail:"},{"line_number":32,"context_line":"            msg: \u003e"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"6f718cd7_3f0bc69c","line":29,"range":{"start_line":22,"start_character":0,"end_line":29,"end_character":0},"in_reply_to":"604e76ab_eeffb989","updated":"2026-10-21 19:34:10.000000000","message":"Sure, we need to check the length separately with a valid check at least. The priority is debatable.","commit_id":"800a269c0f1887bf33d8499a6a44c291f6f45c3f"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"332a985f80cde8da35e3fb2a2f18a1376a81cfcb","unresolved":true,"context_lines":[{"line_number":19,"context_line":"          with_items: \"{{ ether_interfaces }}\""},{"line_number":20,"context_line":"          when: not item | net_interface"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"        - name: Validate network interface name length"},{"line_number":23,"context_line":"          fail:"},{"line_number":24,"context_line":"            msg: \u003e"},{"line_number":25,"context_line":"              Network interface validation failed - interface name {{ item |"},{"line_number":26,"context_line":"              net_interface }} is longer than 15 characters."},{"line_number":27,"context_line":"          with_items: \"{{ ether_interfaces + bridge_interfaces + bond_interfaces }}\""},{"line_number":28,"context_line":"          when: item | net_interface | length \u003e 15"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"        - name: Validate bridge interface configuration"},{"line_number":31,"context_line":"          fail:"},{"line_number":32,"context_line":"            msg: \u003e"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3dc663dd_69bf5e04","line":29,"range":{"start_line":22,"start_character":0,"end_line":29,"end_character":0},"in_reply_to":"61721951_0c1c9994","updated":"2026-10-16 12:03:56.000000000","message":"Why?","commit_id":"800a269c0f1887bf33d8499a6a44c291f6f45c3f"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"dca6ba0705df23682896d7ef541a78c395b29db8","unresolved":true,"context_lines":[{"line_number":19,"context_line":"          with_items: \"{{ ether_interfaces }}\""},{"line_number":20,"context_line":"          when: not item | net_interface"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"        - name: Validate network interface name length"},{"line_number":23,"context_line":"          fail:"},{"line_number":24,"context_line":"            msg: \u003e"},{"line_number":25,"context_line":"              Network interface validation failed - interface name {{ item |"},{"line_number":26,"context_line":"              net_interface }} is longer than 15 characters."},{"line_number":27,"context_line":"          with_items: \"{{ ether_interfaces + bridge_interfaces + bond_interfaces }}\""},{"line_number":28,"context_line":"          when: item | net_interface | length \u003e 15"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"        - name: Validate bridge interface configuration"},{"line_number":31,"context_line":"          fail:"},{"line_number":32,"context_line":"            msg: \u003e"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"47efa67d_17c36493","line":29,"range":{"start_line":22,"start_character":0,"end_line":29,"end_character":0},"in_reply_to":"951781ad_e97237a6","updated":"2026-10-16 13:10:56.000000000","message":"It checks only the ether. Later tasks check the bridge, bond, and so on. So, this way, we need to validate the name after all the checks are done at the bottom of the block?","commit_id":"800a269c0f1887bf33d8499a6a44c291f6f45c3f"}]}
