)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"f77110d07bdf992c253cedf79fd4a737a5ddab31","unresolved":true,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Fix this by normalizing the nested inventory to use ipv4 values for all"},{"line_number":22,"context_line":"ansible_host values in the inventory if any node does not have an ipv6"},{"line_number":23,"context_line":"address."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Change-Id: I545e77920df17c26668bd3bfd1150e290cf8f9d8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":16,"id":"9216baa3_2d3812b6","line":23,"updated":"2026-05-20 16:22:27.000000000","message":"Doesn\u0027t this throw out IPv6 pretty aggressively? It seems like you want \"use ipv4 everywhere if bridge doesn\u0027t have ipv6\" and \"use ipv4 for each node if that node doesn\u0027t have ipv6\" Or are we thinking more of a \"meh\"?","commit_id":"83d175d8f05aa82a2d10789694d57158b51a4fb7"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"86cc1c8772aa7ed37b0cd92a049c1712ca804340","unresolved":true,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Fix this by normalizing the nested inventory to use ipv4 values for all"},{"line_number":22,"context_line":"ansible_host values in the inventory if any node does not have an ipv6"},{"line_number":23,"context_line":"address."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Change-Id: I545e77920df17c26668bd3bfd1150e290cf8f9d8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":16,"id":"9555e19c_0013df3c","line":23,"in_reply_to":"9216baa3_2d3812b6","updated":"2026-05-20 16:23:52.000000000","message":"If any node doesn\u0027t have ipv6 then we want all to use ipv4 to ensure they can talk to each other. Though this particular case is special because it is specifically for the nested ansible inventory on bridge. So I think we probably could optimize it to \"if bridge doesn\u0027t have ipv6 then use ipv4 everywhere\" in this particular case. But I think consistency is nice.","commit_id":"83d175d8f05aa82a2d10789694d57158b51a4fb7"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"bff3ebf31364a1002eb67bd1eea638fbbdae2077","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Fix this by normalizing the nested inventory to use ipv4 values for all"},{"line_number":22,"context_line":"ansible_host values in the inventory if any node does not have an ipv6"},{"line_number":23,"context_line":"address."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Change-Id: I545e77920df17c26668bd3bfd1150e290cf8f9d8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":16,"id":"042b96cc_badc16f4","line":23,"in_reply_to":"9555e19c_0013df3c","updated":"2026-05-20 17:25:57.000000000","message":"For me, the up-side to this approach is that it gives us the freedom to do other mixed-provider integration tests in the future without having to go back and undo a bridge-only optimization.","commit_id":"83d175d8f05aa82a2d10789694d57158b51a4fb7"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"f77110d07bdf992c253cedf79fd4a737a5ddab31","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"6bb1359c_85d26e7d","updated":"2026-05-20 16:22:27.000000000","message":"playbook looks fine, question about the","commit_id":"83d175d8f05aa82a2d10789694d57158b51a4fb7"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"c936770987034ae2d712219db6ac96fac40fe987","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"7fc656b4_6c776f72","updated":"2026-05-20 19:22:02.000000000","message":"recheck","commit_id":"83d175d8f05aa82a2d10789694d57158b51a4fb7"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"9c356cecc273b08af47888ac5aeb10c88163525c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"cd038580_c1069050","updated":"2026-05-28 20:30:53.000000000","message":"recheck eventually things will be happy I hope","commit_id":"83d175d8f05aa82a2d10789694d57158b51a4fb7"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"18af2582b6807661913f50b548a07696eb59427f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"1c0ad6ad_1f75db20","updated":"2026-05-28 18:21:56.000000000","message":"recheck the expected fix for the dns failures has landed.","commit_id":"83d175d8f05aa82a2d10789694d57158b51a4fb7"}]}