)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"8ee43167df73b8c8f65ff00c0ff65a5d68555a29","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Sorting, limiting and pagination support for agent list"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Support already exists on `neutron-lib`."},{"line_number":10,"context_line":"This patch addresses the missing support on Neutron extensions by"},{"line_number":11,"context_line":"creating and using a `agent_sort_key` extension."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7d22d591_563bf2ae","line":9,"updated":"2026-04-08 09:00:42.000000000","message":"yeah since Victoria as I see (https://review.opendev.org/c/openstack/neutron-lib/+/738471 )","commit_id":"27e4ac27a19f895beb7df22e961c103ddf8167a7"},{"author":{"_account_id":36824,"name":"Isaac Silva","display_name":"Isaac Vicente","email":"isaacvicentsocial@gmail.com","username":"isaacvicente","status":"Cloud Engineer working with Openstack on Laboratory of Distributed System at LSD, Brazil"},"change_message_id":"bdbdcc835fe3f9082bff3305dc29f5206b1844fa","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Sorting, limiting and pagination support for agent list"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Support already exists on `neutron-lib`."},{"line_number":10,"context_line":"This patch addresses the missing support on Neutron extensions by"},{"line_number":11,"context_line":"creating and using a `agent_sort_key` extension."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"6cd1e4fe_8bd03f65","line":9,"in_reply_to":"66fcbe7c_967ff3c0","updated":"2026-04-17 20:10:32.000000000","message":"Done","commit_id":"27e4ac27a19f895beb7df22e961c103ddf8167a7"},{"author":{"_account_id":36824,"name":"Isaac Silva","display_name":"Isaac Vicente","email":"isaacvicentsocial@gmail.com","username":"isaacvicente","status":"Cloud Engineer working with Openstack on Laboratory of Distributed System at LSD, Brazil"},"change_message_id":"3211529ae07b291b21add711fc6cb7b610083ccc","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Sorting, limiting and pagination support for agent list"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Support already exists on `neutron-lib`."},{"line_number":10,"context_line":"This patch addresses the missing support on Neutron extensions by"},{"line_number":11,"context_line":"creating and using a `agent_sort_key` extension."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ef616ab9_64ea7cc1","line":9,"in_reply_to":"7d22d591_563bf2ae","updated":"2026-04-08 16:56:36.000000000","message":"do you think it should be mentioned in commit message?","commit_id":"27e4ac27a19f895beb7df22e961c103ddf8167a7"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"cba4142958989658861b70626e1c112cf80594e1","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Sorting, limiting and pagination support for agent list"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Support already exists on `neutron-lib`."},{"line_number":10,"context_line":"This patch addresses the missing support on Neutron extensions by"},{"line_number":11,"context_line":"creating and using a `agent_sort_key` extension."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"66fcbe7c_967ff3c0","line":9,"in_reply_to":"ef616ab9_64ea7cc1","updated":"2026-04-14 08:47:12.000000000","message":"No, sorry, just to have it written as comment, I don\u0027t think such historical precision is necessary, but if you anyway need to push a new version it can be included :)","commit_id":"27e4ac27a19f895beb7df22e961c103ddf8167a7"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":36824,"name":"Isaac Silva","display_name":"Isaac Vicente","email":"isaacvicentsocial@gmail.com","username":"isaacvicente","status":"Cloud Engineer working with Openstack on Laboratory of Distributed System at LSD, Brazil"},"change_message_id":"049c2d2e3a6184d43d78d447e4170560d2109415","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d52b0f05_1e96675b","updated":"2026-04-05 03:27:49.000000000","message":"Pep8 failing in pylint with widespread W0101 warnings seems to be unrelated to this patch, being more likely related to setuptools\u0027 `packages` configuration, as per output of the build [1]. For this reason, I will call for a recheck once more to see if this behaviour changes.\n\n[1] https://zuul.opendev.org/t/openstack/build/def9255bd4fe4a3facc11cd067cc5063","commit_id":"a0bcd31e0358241205df0f54ad112b81dc6dacd4"},{"author":{"_account_id":36824,"name":"Isaac Silva","display_name":"Isaac Vicente","email":"isaacvicentsocial@gmail.com","username":"isaacvicente","status":"Cloud Engineer working with Openstack on Laboratory of Distributed System at LSD, Brazil"},"change_message_id":"eb50e3857200b77c0dc1d2c5b75154e56074ce5c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"33c440e3_0d2d80cf","updated":"2026-04-05 00:59:46.000000000","message":"recheck","commit_id":"a0bcd31e0358241205df0f54ad112b81dc6dacd4"},{"author":{"_account_id":36824,"name":"Isaac Silva","display_name":"Isaac Vicente","email":"isaacvicentsocial@gmail.com","username":"isaacvicente","status":"Cloud Engineer working with Openstack on Laboratory of Distributed System at LSD, Brazil"},"change_message_id":"79d21fcf523b74cdb6be07b76f68668e9458c1cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8c1f8204_9f8def09","updated":"2026-04-05 03:28:15.000000000","message":"recheck","commit_id":"a0bcd31e0358241205df0f54ad112b81dc6dacd4"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"51c35fb108c2f2beda6eed1736296d2a34cef40c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c63e6be1_9e2d8db0","updated":"2026-04-08 09:01:39.000000000","message":"Thanks, please check my comments","commit_id":"27e4ac27a19f895beb7df22e961c103ddf8167a7"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"8ee43167df73b8c8f65ff00c0ff65a5d68555a29","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7dff1ac6_bd4c0cb8","updated":"2026-04-08 09:00:42.000000000","message":"looks ok","commit_id":"27e4ac27a19f895beb7df22e961c103ddf8167a7"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"b668c4c49853f35b937b9bf579e89cb3db9bddfb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"0bf53869_3e918b9b","updated":"2026-04-17 13:07:59.000000000","message":"You need a release note. If you are not adding the ML2/OVN, add a parity gap and mention it in the release note.","commit_id":"94928c390d0f654d29376312f6dc447bbbedaeaa"},{"author":{"_account_id":36824,"name":"Isaac Silva","display_name":"Isaac Vicente","email":"isaacvicentsocial@gmail.com","username":"isaacvicente","status":"Cloud Engineer working with Openstack on Laboratory of Distributed System at LSD, Brazil"},"change_message_id":"e89d6f1fe033c492ec7d9a9b00e7d0bf8eaf88be","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"41314a05_ae8f2d0c","updated":"2026-04-18 19:19:44.000000000","message":"recheck","commit_id":"fc20b17b24da232ec5792c3234de15f0115748f0"},{"author":{"_account_id":36824,"name":"Isaac Silva","display_name":"Isaac Vicente","email":"isaacvicentsocial@gmail.com","username":"isaacvicente","status":"Cloud Engineer working with Openstack on Laboratory of Distributed System at LSD, Brazil"},"change_message_id":"9ff6f0a6a2f73847f55bbb9e6d0cc3cb2a27253a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"4235a045_33eb7020","updated":"2026-04-19 22:47:16.000000000","message":"recheck","commit_id":"fc20b17b24da232ec5792c3234de15f0115748f0"},{"author":{"_account_id":36824,"name":"Isaac Silva","display_name":"Isaac Vicente","email":"isaacvicentsocial@gmail.com","username":"isaacvicente","status":"Cloud Engineer working with Openstack on Laboratory of Distributed System at LSD, Brazil"},"change_message_id":"2844c11f1127cc9e4440967b8c66b219cffedac3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f0da16c6_192843c0","updated":"2026-04-18 14:59:28.000000000","message":"recheck","commit_id":"fc20b17b24da232ec5792c3234de15f0115748f0"},{"author":{"_account_id":36824,"name":"Isaac Silva","display_name":"Isaac Vicente","email":"isaacvicentsocial@gmail.com","username":"isaacvicente","status":"Cloud Engineer working with Openstack on Laboratory of Distributed System at LSD, Brazil"},"change_message_id":"ace43bb1b8050f283020f7069a520aa2fcc7bd89","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"e12874b4_342fd9ba","updated":"2026-04-20 19:12:41.000000000","message":"recheck Only one job failed. It seems that not all compute workers have booted.","commit_id":"fc20b17b24da232ec5792c3234de15f0115748f0"},{"author":{"_account_id":36824,"name":"Isaac Silva","display_name":"Isaac Vicente","email":"isaacvicentsocial@gmail.com","username":"isaacvicente","status":"Cloud Engineer working with Openstack on Laboratory of Distributed System at LSD, Brazil"},"change_message_id":"2ba2941f41fc9db1824d4557ddefa4d9423ef8a4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"91fd2404_bce38934","in_reply_to":"0ae764bd_4e012a6b","updated":"2026-04-20 00:47:05.000000000","message":"Hi Brian, sorry for the rechecks. This one I triggered because of the message of opendevstatus in openstack-neutron IRC channel, about the `POST-FAILURE` job status. Is there any bug tracking this issue with the CI so I can follow up on?","commit_id":"fc20b17b24da232ec5792c3234de15f0115748f0"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"f01b431dee47be4ff22e262f797f61573f6fbbe4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"0ae764bd_4e012a6b","in_reply_to":"4235a045_33eb7020","updated":"2026-04-20 00:19:07.000000000","message":"There seems to be an issue with CI, let\u0027s not try any more until we know it\u0027s fixed. One of the core reviewers will do it when/if they approve.","commit_id":"fc20b17b24da232ec5792c3234de15f0115748f0"}],"neutron/db/agents_db.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"8ee43167df73b8c8f65ff00c0ff65a5d68555a29","unresolved":true,"context_lines":[{"line_number":293,"context_line":"        pager \u003d base_obj.Pager("},{"line_number":294,"context_line":"            sorts\u003dsorts, limit\u003dlimit, page_reverse\u003dpage_reverse, marker\u003dmarker"},{"line_number":295,"context_line":"        )"},{"line_number":296,"context_line":"        alive \u003d filters and filters.pop(\u0027alive\u0027, None)"},{"line_number":297,"context_line":"        agents \u003d agent_obj.Agent.get_objects(context, _pager\u003dpager, **filters)"},{"line_number":298,"context_line":"        if alive:"},{"line_number":299,"context_line":"            alive \u003d converters.convert_to_boolean(alive[0])"}],"source_content_type":"text/x-python","patch_set":4,"id":"3b5f26c3_1b72ac90","line":296,"range":{"start_line":296,"start_character":8,"end_line":296,"end_character":54},"updated":"2026-04-08 09:00:42.000000000","message":"not sure what happens if alive is there and it will be applied only on the previously limited list of agents, so perhaps have to apply first alive and than the filter with limit etc... ?","commit_id":"27e4ac27a19f895beb7df22e961c103ddf8167a7"},{"author":{"_account_id":36824,"name":"Isaac Silva","display_name":"Isaac Vicente","email":"isaacvicentsocial@gmail.com","username":"isaacvicente","status":"Cloud Engineer working with Openstack on Laboratory of Distributed System at LSD, Brazil"},"change_message_id":"3211529ae07b291b21add711fc6cb7b610083ccc","unresolved":true,"context_lines":[{"line_number":293,"context_line":"        pager \u003d base_obj.Pager("},{"line_number":294,"context_line":"            sorts\u003dsorts, limit\u003dlimit, page_reverse\u003dpage_reverse, marker\u003dmarker"},{"line_number":295,"context_line":"        )"},{"line_number":296,"context_line":"        alive \u003d filters and filters.pop(\u0027alive\u0027, None)"},{"line_number":297,"context_line":"        agents \u003d agent_obj.Agent.get_objects(context, _pager\u003dpager, **filters)"},{"line_number":298,"context_line":"        if alive:"},{"line_number":299,"context_line":"            alive \u003d converters.convert_to_boolean(alive[0])"}],"source_content_type":"text/x-python","patch_set":4,"id":"aa70d1c9_dffca63d","line":296,"range":{"start_line":296,"start_character":8,"end_line":296,"end_character":54},"in_reply_to":"3b5f26c3_1b72ac90","updated":"2026-04-08 16:56:36.000000000","message":"Thanks for the feedback. The filters are being applied on the next line with the argument `**filters`, so all agents are being filtered and sorted and/or limited. It seems that \"alive\" is a additional filter not handled by the `get_objects` method.","commit_id":"27e4ac27a19f895beb7df22e961c103ddf8167a7"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"cba4142958989658861b70626e1c112cf80594e1","unresolved":true,"context_lines":[{"line_number":293,"context_line":"        pager \u003d base_obj.Pager("},{"line_number":294,"context_line":"            sorts\u003dsorts, limit\u003dlimit, page_reverse\u003dpage_reverse, marker\u003dmarker"},{"line_number":295,"context_line":"        )"},{"line_number":296,"context_line":"        alive \u003d filters and filters.pop(\u0027alive\u0027, None)"},{"line_number":297,"context_line":"        agents \u003d agent_obj.Agent.get_objects(context, _pager\u003dpager, **filters)"},{"line_number":298,"context_line":"        if alive:"},{"line_number":299,"context_line":"            alive \u003d converters.convert_to_boolean(alive[0])"}],"source_content_type":"text/x-python","patch_set":4,"id":"aed71469_12141bf8","line":296,"range":{"start_line":296,"start_character":8,"end_line":296,"end_character":54},"in_reply_to":"aa70d1c9_dffca63d","updated":"2026-04-14 08:47:12.000000000","message":"yeah, the logic is interesting for alive, after looking again look ok","commit_id":"27e4ac27a19f895beb7df22e961c103ddf8167a7"},{"author":{"_account_id":36824,"name":"Isaac Silva","display_name":"Isaac Vicente","email":"isaacvicentsocial@gmail.com","username":"isaacvicente","status":"Cloud Engineer working with Openstack on Laboratory of Distributed System at LSD, Brazil"},"change_message_id":"bdbdcc835fe3f9082bff3305dc29f5206b1844fa","unresolved":false,"context_lines":[{"line_number":293,"context_line":"        pager \u003d base_obj.Pager("},{"line_number":294,"context_line":"            sorts\u003dsorts, limit\u003dlimit, page_reverse\u003dpage_reverse, marker\u003dmarker"},{"line_number":295,"context_line":"        )"},{"line_number":296,"context_line":"        alive \u003d filters and filters.pop(\u0027alive\u0027, None)"},{"line_number":297,"context_line":"        agents \u003d agent_obj.Agent.get_objects(context, _pager\u003dpager, **filters)"},{"line_number":298,"context_line":"        if alive:"},{"line_number":299,"context_line":"            alive \u003d converters.convert_to_boolean(alive[0])"}],"source_content_type":"text/x-python","patch_set":4,"id":"5661713f_0dcec59c","line":296,"range":{"start_line":296,"start_character":8,"end_line":296,"end_character":54},"in_reply_to":"aed71469_12141bf8","updated":"2026-04-17 20:10:32.000000000","message":"Done","commit_id":"27e4ac27a19f895beb7df22e961c103ddf8167a7"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"8ee43167df73b8c8f65ff00c0ff65a5d68555a29","unresolved":true,"context_lines":[{"line_number":1593,"context_line":""},{"line_number":1594,"context_line":""},{"line_number":1595,"context_line":"def get_agents(self, context, filters\u003dNone, fields\u003dNone, _driver\u003dNone,"},{"line_number":1596,"context_line":"               sorts\u003dNone, limit\u003dNone, marker\u003dNone, page_reverse\u003dFalse):"},{"line_number":1597,"context_line":"    _driver.ping_all_chassis()"},{"line_number":1598,"context_line":"    filters \u003d filters or {}"},{"line_number":1599,"context_line":"    agent_list \u003d n_agent.AgentCache().get_agents(filters)"}],"source_content_type":"text/x-python","patch_set":4,"id":"6c753a66_68746a8e","line":1596,"range":{"start_line":1596,"start_character":15,"end_line":1596,"end_character":70},"updated":"2026-04-08 09:00:42.000000000","message":"these are ignored here, is that intentional?","commit_id":"27e4ac27a19f895beb7df22e961c103ddf8167a7"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"b668c4c49853f35b937b9bf579e89cb3db9bddfb","unresolved":true,"context_lines":[{"line_number":1593,"context_line":""},{"line_number":1594,"context_line":""},{"line_number":1595,"context_line":"def get_agents(self, context, filters\u003dNone, fields\u003dNone, _driver\u003dNone,"},{"line_number":1596,"context_line":"               sorts\u003dNone, limit\u003dNone, marker\u003dNone, page_reverse\u003dFalse):"},{"line_number":1597,"context_line":"    _driver.ping_all_chassis()"},{"line_number":1598,"context_line":"    filters \u003d filters or {}"},{"line_number":1599,"context_line":"    agent_list \u003d n_agent.AgentCache().get_agents(filters)"}],"source_content_type":"text/x-python","patch_set":4,"id":"52908c98_f0d28eb9","line":1596,"range":{"start_line":1596,"start_character":15,"end_line":1596,"end_character":70},"in_reply_to":"03c8efc0_daea0246","updated":"2026-04-17 13:07:59.000000000","message":"I was going to comment that, precisely. The ML2/OVN agent cache works different from other (SQL) agents registers.\n\nIf you are not addressing the OVN code part, add a parity gap for ML2/OVN.","commit_id":"27e4ac27a19f895beb7df22e961c103ddf8167a7"},{"author":{"_account_id":36824,"name":"Isaac Silva","display_name":"Isaac Vicente","email":"isaacvicentsocial@gmail.com","username":"isaacvicente","status":"Cloud Engineer working with Openstack on Laboratory of Distributed System at LSD, Brazil"},"change_message_id":"bdbdcc835fe3f9082bff3305dc29f5206b1844fa","unresolved":true,"context_lines":[{"line_number":1593,"context_line":""},{"line_number":1594,"context_line":""},{"line_number":1595,"context_line":"def get_agents(self, context, filters\u003dNone, fields\u003dNone, _driver\u003dNone,"},{"line_number":1596,"context_line":"               sorts\u003dNone, limit\u003dNone, marker\u003dNone, page_reverse\u003dFalse):"},{"line_number":1597,"context_line":"    _driver.ping_all_chassis()"},{"line_number":1598,"context_line":"    filters \u003d filters or {}"},{"line_number":1599,"context_line":"    agent_list \u003d n_agent.AgentCache().get_agents(filters)"}],"source_content_type":"text/x-python","patch_set":4,"id":"e4c68d69_756b853a","line":1596,"range":{"start_line":1596,"start_character":15,"end_line":1596,"end_character":70},"in_reply_to":"52908c98_f0d28eb9","updated":"2026-04-17 20:10:32.000000000","message":"Done. Can you can see if everything is ok?","commit_id":"27e4ac27a19f895beb7df22e961c103ddf8167a7"},{"author":{"_account_id":36824,"name":"Isaac Silva","display_name":"Isaac Vicente","email":"isaacvicentsocial@gmail.com","username":"isaacvicente","status":"Cloud Engineer working with Openstack on Laboratory of Distributed System at LSD, Brazil"},"change_message_id":"3211529ae07b291b21add711fc6cb7b610083ccc","unresolved":true,"context_lines":[{"line_number":1593,"context_line":""},{"line_number":1594,"context_line":""},{"line_number":1595,"context_line":"def get_agents(self, context, filters\u003dNone, fields\u003dNone, _driver\u003dNone,"},{"line_number":1596,"context_line":"               sorts\u003dNone, limit\u003dNone, marker\u003dNone, page_reverse\u003dFalse):"},{"line_number":1597,"context_line":"    _driver.ping_all_chassis()"},{"line_number":1598,"context_line":"    filters \u003d filters or {}"},{"line_number":1599,"context_line":"    agent_list \u003d n_agent.AgentCache().get_agents(filters)"}],"source_content_type":"text/x-python","patch_set":4,"id":"bedcd1a8_a3e91c1d","line":1596,"range":{"start_line":1596,"start_character":15,"end_line":1596,"end_character":70},"in_reply_to":"6c753a66_68746a8e","updated":"2026-04-08 16:56:36.000000000","message":"As per this comment[1] on this patch[2], these arguments are used to overwrite the `get_agents` method[3]. To be fair, I think that the arguments need to be used in `get_agents` method of AgentCache object. For that, maybe a lot of code would change. For this, I suggest that the support for sorting, paging, etc for OVN driver would be in a different patch. What do you think?\n\n[1] https://review.opendev.org/c/openstack/neutron/+/743953/comment/9f560f44_44ae4f2e/\n[2] https://review.opendev.org/c/openstack/neutron/+/743953\n[3] https://review.opendev.org/c/openstack/neutron/+/983337/4/neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py#435","commit_id":"27e4ac27a19f895beb7df22e961c103ddf8167a7"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"cba4142958989658861b70626e1c112cf80594e1","unresolved":true,"context_lines":[{"line_number":1593,"context_line":""},{"line_number":1594,"context_line":""},{"line_number":1595,"context_line":"def get_agents(self, context, filters\u003dNone, fields\u003dNone, _driver\u003dNone,"},{"line_number":1596,"context_line":"               sorts\u003dNone, limit\u003dNone, marker\u003dNone, page_reverse\u003dFalse):"},{"line_number":1597,"context_line":"    _driver.ping_all_chassis()"},{"line_number":1598,"context_line":"    filters \u003d filters or {}"},{"line_number":1599,"context_line":"    agent_list \u003d n_agent.AgentCache().get_agents(filters)"}],"source_content_type":"text/x-python","patch_set":4,"id":"03c8efc0_daea0246","line":1596,"range":{"start_line":1596,"start_character":15,"end_line":1596,"end_character":70},"in_reply_to":"bedcd1a8_a3e91c1d","updated":"2026-04-14 08:47:12.000000000","message":"I am fine with a follow-up patch for sorting/paging etc, let\u0027s see how others see it","commit_id":"27e4ac27a19f895beb7df22e961c103ddf8167a7"}],"neutron/tests/unit/extensions/test_agent.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"6eb06db2806299cc22309bbbd0b54d98f6450ea5","unresolved":true,"context_lines":[{"line_number":198,"context_line":"                expected_code\u003dexc.HTTPBadRequest.code)"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":"        expected_error_message \u003d ("},{"line_number":201,"context_line":"            \"The number of sort_keys and sort_dirs must be same\""},{"line_number":202,"context_line":"        )"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"        self.assertEqual("}],"source_content_type":"text/x-python","patch_set":6,"id":"dd6b68f8_fe79de12","line":201,"range":{"start_line":201,"start_character":56,"end_line":201,"end_character":63},"updated":"2026-04-27 19:48:23.000000000","message":"hah, guess there\u0027s a typo in the message - \"be the same\". Don\u0027t need to fix unless you re-spin.","commit_id":"fc20b17b24da232ec5792c3234de15f0115748f0"},{"author":{"_account_id":36824,"name":"Isaac Silva","display_name":"Isaac Vicente","email":"isaacvicentsocial@gmail.com","username":"isaacvicente","status":"Cloud Engineer working with Openstack on Laboratory of Distributed System at LSD, Brazil"},"change_message_id":"0f138825a7786fe8441ff219d5fe1dc3688a9492","unresolved":false,"context_lines":[{"line_number":198,"context_line":"                expected_code\u003dexc.HTTPBadRequest.code)"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":"        expected_error_message \u003d ("},{"line_number":201,"context_line":"            \"The number of sort_keys and sort_dirs must be same\""},{"line_number":202,"context_line":"        )"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"        self.assertEqual("}],"source_content_type":"text/x-python","patch_set":6,"id":"ed1b855b_1e3ea3cc","line":201,"range":{"start_line":201,"start_character":56,"end_line":201,"end_character":63},"in_reply_to":"dd6b68f8_fe79de12","updated":"2026-05-04 18:05:21.000000000","message":"yeah, I missed this one. The message is in api_common.","commit_id":"fc20b17b24da232ec5792c3234de15f0115748f0"}]}