)]}'
{"neutron/tests/functional/agent/ovn/agent/test_ovn_neutron_agent.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"8422e472446652cc4af4a15027366bb164a8fb25","unresolved":true,"context_lines":[{"line_number":145,"context_line":"                   }"},{"line_number":146,"context_line":"        n_utils.wait_until_true("},{"line_number":147,"context_line":"            lambda: check_extids(ext_ids, self.chassis_name),"},{"line_number":148,"context_line":"            timeout\u003d10)"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"        # Check Unix proxy is running."},{"line_number":151,"context_line":"        metadata_extension \u003d self.ovn_agent[METADATA_EXTENSION]"}],"source_content_type":"text/x-python","patch_set":1,"id":"fca3eb81_be723045","line":148,"updated":"2025-07-15 15:48:18.000000000","message":"This change from stephenfin is getting rid of a lot of lambdas (currently in gate queue):\n\nhttps://review.opendev.org/c/openstack/neutron/+/950582/\n\nWith that in mind, if you move the function just below where ext_ids is defined, you should be able to remove the lambda. Just mentioning since I think this will fail pep8 if that merges first.\n\n        def check_extids():\n            ch_private \u003d self.sb_api.lookup(\u0027Chassis_Private\u0027, self.chassis_name)\n            return ext_ids \u003d\u003d ch_private.external_ids\n\n        n_utils.wait_until_true(check_extids(), timeout\u003d10)\n\n\n(I actually just checked and your original code will not fail the ruff check, but i\u0027ll still leave my comment)","commit_id":"d8697facb0d95f15644d49052570e80f9f866eab"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"40f3be959720ccc8eed21164ecba03c6f40bbc5b","unresolved":false,"context_lines":[{"line_number":145,"context_line":"                   }"},{"line_number":146,"context_line":"        n_utils.wait_until_true("},{"line_number":147,"context_line":"            lambda: check_extids(ext_ids, self.chassis_name),"},{"line_number":148,"context_line":"            timeout\u003d10)"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"        # Check Unix proxy is running."},{"line_number":151,"context_line":"        metadata_extension \u003d self.ovn_agent[METADATA_EXTENSION]"}],"source_content_type":"text/x-python","patch_set":1,"id":"1553cce0_fb3ade26","line":148,"in_reply_to":"fca3eb81_be723045","updated":"2025-07-17 06:41:36.000000000","message":"We can\u0027t pass directly `check_extids()` because that will execute the method once only, the first time is called. The lambda expression will perform the execution any time is called.\n\nThe upper patch is removing the unnecessary lambdas but in this case it is neccessary.","commit_id":"d8697facb0d95f15644d49052570e80f9f866eab"}]}