)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"01def9c76a73216e596ed7fed257d0d08c400dcb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9ffae82c_63e68a1b","updated":"2026-08-12 15:05:08.000000000","message":"I\u0027ll drop my -1, let\u0027s see what Rodolfo thinks","commit_id":"ef09ac8ed8e14d9ccbf18c4db2602a0b62e7ea00"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"0ffc352318055b0d66e3946286e9b16bb6f94fa2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"902f7994_e44815bc","updated":"2026-07-31 18:29:24.000000000","message":"This effectively backports https://review.opendev.org/c/openstack/neutron/+/983868 without relying on the other changes.","commit_id":"ef09ac8ed8e14d9ccbf18c4db2602a0b62e7ea00"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"ea36460298d7954db7cb6730e5eb1ee17fa9678e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a9e80ce7_6f04cb08","updated":"2026-08-13 22:36:37.000000000","message":"Nova servers rally test has an incorrect number of arguments for a function.","commit_id":"017469463f8e91cba4c4681b8374f7c449d2b09c"}],"neutron/tests/functional/services/ovn_l3/test_plugin.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"f9dc2d8e9ee32de479a0f256729ce0537834ea5a","unresolved":true,"context_lines":[{"line_number":272,"context_line":"        # The GW network is tunnelled, no physnet defined --\u003e no possible"},{"line_number":273,"context_line":"        # candidates."},{"line_number":274,"context_line":"        self._check_gateway_chassis_candidates(None, physnet\u003dNone)"},{"line_number":275,"context_line":""},{"line_number":276,"context_line":"    def _create_router_with_tunnelled_gw_and_internal_net(self):"},{"line_number":277,"context_line":"        ext1 \u003d self._create_ext_network("},{"line_number":278,"context_line":"            \u0027ext1\u0027, \u0027geneve\u0027, None, None, \"10.0.0.1\", \"10.0.0.0/24\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"0d8bb3ef_a8018a2c","line":275,"updated":"2026-08-07 15:46:59.000000000","message":"So I think my main issue with this isn\u0027t the one-liner code change, but that none of these tests exist on the master branch. It seems they would all apply there and help with coverage? I\u0027ll add Rodolfo to get his opinion as well.","commit_id":"ef09ac8ed8e14d9ccbf18c4db2602a0b62e7ea00"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"3461e34db76ebaa7c1699f09bb506d1de92470a7","unresolved":true,"context_lines":[{"line_number":272,"context_line":"        # The GW network is tunnelled, no physnet defined --\u003e no possible"},{"line_number":273,"context_line":"        # candidates."},{"line_number":274,"context_line":"        self._check_gateway_chassis_candidates(None, physnet\u003dNone)"},{"line_number":275,"context_line":""},{"line_number":276,"context_line":"    def _create_router_with_tunnelled_gw_and_internal_net(self):"},{"line_number":277,"context_line":"        ext1 \u003d self._create_ext_network("},{"line_number":278,"context_line":"            \u0027ext1\u0027, \u0027geneve\u0027, None, None, \"10.0.0.1\", \"10.0.0.0/24\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"90f95389_8a12e516","line":275,"in_reply_to":"0d8bb3ef_a8018a2c","updated":"2026-08-07 20:17:50.000000000","message":"So you\u0027re right those tests don\u0027t exist in master. In fact the behaviors aren\u0027t really tested in master at all. The change to the behaviors that I\u0027m looking to backport are from I8bfcc2b77acb89b78b13563aa4ca9fe3f19cb076 and what I\u0027m doing are adding tests. I started with having the LLM take my logs and troubleshooting and craft me some tests but then I couldn\u0027t reproduce the issue on master. So I switched to 2026.1, which is what I\u0027m running. That led me to that change but its a larger change with other dependencies so I just took the change that mattered. So what I\u0027ve done is made https://review.opendev.org/c/openstack/neutron/+/1000150 to bring the tests to master first and then we can re-consider this?","commit_id":"ef09ac8ed8e14d9ccbf18c4db2602a0b62e7ea00"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"30682e8667db592d35c14a8d492646653f71551d","unresolved":false,"context_lines":[{"line_number":272,"context_line":"        # The GW network is tunnelled, no physnet defined --\u003e no possible"},{"line_number":273,"context_line":"        # candidates."},{"line_number":274,"context_line":"        self._check_gateway_chassis_candidates(None, physnet\u003dNone)"},{"line_number":275,"context_line":""},{"line_number":276,"context_line":"    def _create_router_with_tunnelled_gw_and_internal_net(self):"},{"line_number":277,"context_line":"        ext1 \u003d self._create_ext_network("},{"line_number":278,"context_line":"            \u0027ext1\u0027, \u0027geneve\u0027, None, None, \"10.0.0.1\", \"10.0.0.0/24\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"104dc6a6_bb80d87d","line":275,"in_reply_to":"6b2cc73f_85f2338a","updated":"2026-08-13 16:55:21.000000000","message":"Done","commit_id":"ef09ac8ed8e14d9ccbf18c4db2602a0b62e7ea00"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"a7225632c633ff6c6d295ccc84c8ea3e6c9b4619","unresolved":true,"context_lines":[{"line_number":272,"context_line":"        # The GW network is tunnelled, no physnet defined --\u003e no possible"},{"line_number":273,"context_line":"        # candidates."},{"line_number":274,"context_line":"        self._check_gateway_chassis_candidates(None, physnet\u003dNone)"},{"line_number":275,"context_line":""},{"line_number":276,"context_line":"    def _create_router_with_tunnelled_gw_and_internal_net(self):"},{"line_number":277,"context_line":"        ext1 \u003d self._create_ext_network("},{"line_number":278,"context_line":"            \u0027ext1\u0027, \u0027geneve\u0027, None, None, \"10.0.0.1\", \"10.0.0.0/24\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"6b2cc73f_85f2338a","line":275,"in_reply_to":"90f95389_8a12e516","updated":"2026-08-10 21:04:30.000000000","message":"Yes I think that\u0027s fine.","commit_id":"ef09ac8ed8e14d9ccbf18c4db2602a0b62e7ea00"}]}