)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"8539e91013a58910c7e12aeeed4515c0d9f3195b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"75c15c64_35df051f","updated":"2026-04-09 21:23:45.000000000","message":"Ping reviewers, just needs a +W","commit_id":"16de9de481921aa0ede28774b50c5b994f57bcf5"}],"neutron_tempest_plugin/api/admin/test_routers_ha.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"f7430420f5a83f2287ae36dc68f8c8188e8bd9a6","unresolved":true,"context_lines":[{"line_number":14,"context_line":"from tempest.lib.common.utils import data_utils"},{"line_number":15,"context_line":"from tempest.lib import decorators"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from neutron_lib import constants"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from neutron_tempest_plugin.api import base_routers as base"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"a75a94ae_317d358e","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":33},"updated":"2026-04-01 08:01:52.000000000","message":"I have mixed feelings with things when the test code imports from the tested code in integration test suites, but I see that it can make life easier.","commit_id":"16de9de481921aa0ede28774b50c5b994f57bcf5"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"9cb8e8486760e8346d5e51f0bd97e5e15975a720","unresolved":true,"context_lines":[{"line_number":14,"context_line":"from tempest.lib.common.utils import data_utils"},{"line_number":15,"context_line":"from tempest.lib import decorators"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from neutron_lib import constants"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from neutron_tempest_plugin.api import base_routers as base"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"d79fe166_809a5f11","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":33},"in_reply_to":"212aac0b_ad3b8025","updated":"2026-04-01 16:17:08.000000000","message":"after thinking on it, I am fine with it here, but still keep that it was good to think longer on this topic 😊","commit_id":"16de9de481921aa0ede28774b50c5b994f57bcf5"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"fe03ed998ded6316128c717a6e4f5d843d310fc4","unresolved":true,"context_lines":[{"line_number":14,"context_line":"from tempest.lib.common.utils import data_utils"},{"line_number":15,"context_line":"from tempest.lib import decorators"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from neutron_lib import constants"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from neutron_tempest_plugin.api import base_routers as base"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"212aac0b_ad3b8025","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":33},"in_reply_to":"a75a94ae_317d358e","updated":"2026-04-01 13:15:07.000000000","message":"Right. In this case I think it\u0027s Ok since we\u0027re just using the constant to construct the name of the network, then verifying the segment is correct. But let me know what you want to do.","commit_id":"16de9de481921aa0ede28774b50c5b994f57bcf5"}]}
