)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"0f9c4e019227378f3512103fbb4d35f8a9be976d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2c54e1f4_1e4ffd09","updated":"2026-01-15 18:37:33.000000000","message":"Verified locally on a single test case (testing on CI few jobs before review) for `configured_hosts` override, since additional fixes required for inventory parsing code to be corrected, and maybe will be done on follow up patch.","commit_id":"7aea0f438e438e5d228ffe3226e0d7691f0786d6"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"d4dd8ef18e1a7867728060ee7531b6842ad4807e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"c79f2bac_f8708c29","updated":"2026-03-04 11:50:47.000000000","message":"a couple of comments that I think are important but won\u0027t block this patch (i.e. they can be addressed in a follow up patch)","commit_id":"f98acd231c1ddf63a8dc979949d87147c5603e90"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"7712c4c5aed1396059881758892528c2e93a8611","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"b7bddd04_b394eed6","updated":"2026-02-25 18:23:38.000000000","message":"recheck NTP fix merged https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/977958","commit_id":"f98acd231c1ddf63a8dc979949d87147c5603e90"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"6168bc2d038eff19f8d063720d46a1d5ed711e48","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"5c9ea81b_5d716288","updated":"2026-02-24 16:41:37.000000000","message":"recheck unrelated failures (relate to oslo resolved configuration issue)","commit_id":"f98acd231c1ddf63a8dc979949d87147c5603e90"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"2e1f75b7fe6e114e79b132670fe98c7be9ba2d63","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"a990562b_a88293df","updated":"2026-03-04 18:08:41.000000000","message":"rebased on recent changes","commit_id":"f16fe5cd564d575796e997aed78421e8a75b150d"}],"whitebox_neutron_tempest_plugin/tests/scenario/base.py":[{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"d4dd8ef18e1a7867728060ee7531b6842ad4807e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"4cd97b3c_1a7bfac3","updated":"2026-03-04 11:50:47.000000000","message":"as @ralonsoh@redhat.com mentioned, it would help a lot some documentation on is_controller, is_compute, is_networker, is_ocp, etc.","commit_id":"f98acd231c1ddf63a8dc979949d87147c5603e90"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"356930c45674d93fe8df56951ab6db226341711a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"f3e03821_f03cc736","in_reply_to":"4cd97b3c_1a7bfac3","updated":"2026-03-04 15:14:33.000000000","message":"Done, added clearer comments where these booleans are defined.","commit_id":"f98acd231c1ddf63a8dc979949d87147c5603e90"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"d4dd8ef18e1a7867728060ee7531b6842ad4807e","unresolved":true,"context_lines":[{"line_number":1474,"context_line":"        for node in self.nodes:"},{"line_number":1475,"context_line":"            if not (node.get(\u0027ovs_pod\u0027) or"},{"line_number":1476,"context_line":"                    node[\u0027is_compute\u0027] or"},{"line_number":1477,"context_line":"                    node.get(\u0027is_networker\u0027)):"},{"line_number":1478,"context_line":"                LOG.debug(\u0027Traffic is not captured on node %s because it is \u0027"},{"line_number":1479,"context_line":"                          \u0027none of these:\\n\u0027"},{"line_number":1480,"context_line":"                          \u0027- a controller running an ovs pod\\n\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"b23099e5_5f66b47f","line":1477,"range":{"start_line":1477,"start_character":25,"end_line":1477,"end_character":28},"updated":"2026-03-04 11:50:47.000000000","message":"We could initialize all these booleans to False when a node is created.","commit_id":"f98acd231c1ddf63a8dc979949d87147c5603e90"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"356930c45674d93fe8df56951ab6db226341711a","unresolved":false,"context_lines":[{"line_number":1474,"context_line":"        for node in self.nodes:"},{"line_number":1475,"context_line":"            if not (node.get(\u0027ovs_pod\u0027) or"},{"line_number":1476,"context_line":"                    node[\u0027is_compute\u0027] or"},{"line_number":1477,"context_line":"                    node.get(\u0027is_networker\u0027)):"},{"line_number":1478,"context_line":"                LOG.debug(\u0027Traffic is not captured on node %s because it is \u0027"},{"line_number":1479,"context_line":"                          \u0027none of these:\\n\u0027"},{"line_number":1480,"context_line":"                          \u0027- a controller running an ovs pod\\n\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"28bc173b_e3aa4755","line":1477,"range":{"start_line":1477,"start_character":25,"end_line":1477,"end_character":28},"in_reply_to":"b23099e5_5f66b47f","updated":"2026-03-04 15:14:33.000000000","message":"Good idea, thanks, done.","commit_id":"f98acd231c1ddf63a8dc979949d87147c5603e90"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"a5756b238ad5c5e6e45fad11dd3d72227b78acce","unresolved":true,"context_lines":[{"line_number":1480,"context_line":"        for node in self.nodes:"},{"line_number":1481,"context_line":"            if not (node.get(\u0027ovs_pod\u0027) or"},{"line_number":1482,"context_line":"                    node[\u0027is_compute\u0027] or"},{"line_number":1483,"context_line":"                    node.get(\u0027is_networker\u0027)):"},{"line_number":1484,"context_line":"                LOG.debug(\u0027Traffic is not captured on node %s because it is \u0027"},{"line_number":1485,"context_line":"                          \u0027none of these:\\n\u0027"},{"line_number":1486,"context_line":"                          \u0027- a controller running an ovs pod\\n\u0027"}],"source_content_type":"text/x-python","patch_set":9,"id":"8bd1cffa_bf39c142","line":1483,"range":{"start_line":1483,"start_character":20,"end_line":1483,"end_character":44},"updated":"2026-03-04 15:21:09.000000000","message":"```suggestion\n                    node[\u0027is_networker\u0027]):\n```","commit_id":"ca3bcf484cc1f42b05a5f25da82c8d5d458b7866"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"0bcc3b9423bf1ae1e7d9f97f488baccf1dbd61cd","unresolved":false,"context_lines":[{"line_number":1480,"context_line":"        for node in self.nodes:"},{"line_number":1481,"context_line":"            if not (node.get(\u0027ovs_pod\u0027) or"},{"line_number":1482,"context_line":"                    node[\u0027is_compute\u0027] or"},{"line_number":1483,"context_line":"                    node.get(\u0027is_networker\u0027)):"},{"line_number":1484,"context_line":"                LOG.debug(\u0027Traffic is not captured on node %s because it is \u0027"},{"line_number":1485,"context_line":"                          \u0027none of these:\\n\u0027"},{"line_number":1486,"context_line":"                          \u0027- a controller running an ovs pod\\n\u0027"}],"source_content_type":"text/x-python","patch_set":9,"id":"1d405790_eafab920","line":1483,"range":{"start_line":1483,"start_character":20,"end_line":1483,"end_character":44},"in_reply_to":"8bd1cffa_bf39c142","updated":"2026-03-04 15:32:07.000000000","message":"Sorry, must have missed these fixes somewhere along git rebase use.","commit_id":"ca3bcf484cc1f42b05a5f25da82c8d5d458b7866"}],"whitebox_neutron_tempest_plugin/tests/scenario/test_dvr_ovn.py":[{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"a5756b238ad5c5e6e45fad11dd3d72227b78acce","unresolved":true,"context_lines":[{"line_number":59,"context_line":"        if not cls.is_devstack:"},{"line_number":60,"context_line":"            if WB_CONF.bgp:"},{"line_number":61,"context_line":"                for node in cls.nodes:"},{"line_number":62,"context_line":"                    if node.get(\u0027is_networker\u0027) and not node[\u0027is_controller\u0027]:"},{"line_number":63,"context_line":"                        output \u003d local_utils.conf_action("},{"line_number":64,"context_line":"                            WB_CONF.bgp_agent_config,"},{"line_number":65,"context_line":"                            \u0027DEFAULT\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"6dcbd58b_b1546407","line":62,"updated":"2026-03-04 15:21:09.000000000","message":"```suggestion\n                    if node.[\u0027is_networker\u0027] and not node[\u0027is_controller\u0027]:\n```","commit_id":"ca3bcf484cc1f42b05a5f25da82c8d5d458b7866"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"f6cce56dc153b699aa7f9b9001dd66c1024ef734","unresolved":false,"context_lines":[{"line_number":59,"context_line":"        if not cls.is_devstack:"},{"line_number":60,"context_line":"            if WB_CONF.bgp:"},{"line_number":61,"context_line":"                for node in cls.nodes:"},{"line_number":62,"context_line":"                    if node.get(\u0027is_networker\u0027) and not node[\u0027is_controller\u0027]:"},{"line_number":63,"context_line":"                        output \u003d local_utils.conf_action("},{"line_number":64,"context_line":"                            WB_CONF.bgp_agent_config,"},{"line_number":65,"context_line":"                            \u0027DEFAULT\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"3a731747_39966ce4","line":62,"in_reply_to":"030a411e_585220fc","updated":"2026-03-04 16:15:17.000000000","message":"my bad. It should be node[\u0027is_networker\u0027]","commit_id":"ca3bcf484cc1f42b05a5f25da82c8d5d458b7866"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"0bcc3b9423bf1ae1e7d9f97f488baccf1dbd61cd","unresolved":false,"context_lines":[{"line_number":59,"context_line":"        if not cls.is_devstack:"},{"line_number":60,"context_line":"            if WB_CONF.bgp:"},{"line_number":61,"context_line":"                for node in cls.nodes:"},{"line_number":62,"context_line":"                    if node.get(\u0027is_networker\u0027) and not node[\u0027is_controller\u0027]:"},{"line_number":63,"context_line":"                        output \u003d local_utils.conf_action("},{"line_number":64,"context_line":"                            WB_CONF.bgp_agent_config,"},{"line_number":65,"context_line":"                            \u0027DEFAULT\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"030a411e_585220fc","line":62,"in_reply_to":"6dcbd58b_b1546407","updated":"2026-03-04 15:32:07.000000000","message":"Sorry, must have missed these fixes somewhere along git rebase use.","commit_id":"ca3bcf484cc1f42b05a5f25da82c8d5d458b7866"}]}