)]}'
{"neutron/tests/functional/agent/ovsdb/native/test_helpers.py":[{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"34395b6724928dddafd51c4a60911fbbf69f4dd4","unresolved":true,"context_lines":[{"line_number":45,"context_line":"            ovsdb_cfg_connections.append(\u0027tcp:127.0.0.1:%s\u0027 % _port)"},{"line_number":46,"context_line":"            manager_connections.append(\u0027ptcp:%s:127.0.0.1\u0027 % _port)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"        # Wait until all managers are in the OVS DB."},{"line_number":49,"context_line":"        n_utils.wait_until_true("},{"line_number":50,"context_line":"            functools.partial(check_num_manconn, NUM_MANAGERS, ovs.ovsdb),"},{"line_number":51,"context_line":"            timeout\u003d10,"}],"source_content_type":"text/x-python","patch_set":1,"id":"ae1f2efe_1a1127ca","line":48,"updated":"2025-11-11 17:35:07.000000000","message":"Unless I\u0027m missing something, this looks like it is waiting before we call enable_connection_uri() which will create the Manager rows.\n\nAlso, the bug refers to this test as well https://d038b98109ab08a014de-0ff19eff07cf9d69f5d2cf9532cefcfc.ssl.cf5.rackcdn.com/openstack/f33554389b5e48d7b791ed3f19ef78f3/testr_results.html which this doesn\u0027t yet address.\n\nNormally, I don\u0027t like wait_until_true() for ovsdb-related waiting since we have WaitEvent which can be created to wait for the Rows we really want w/o polling. Should be possible to create one that listens for Manager create events for each row and marks them off in the match_fn() and matches when all expected rows are matched. Maybe not worth it, so don\u0027t feel like you have to do that part. 😊","commit_id":"81d81afed8a941f9ee935dc5748cd13f7b724661"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"c4c0953bcd067aa339bc33c35f54f7afe3226f73","unresolved":false,"context_lines":[{"line_number":45,"context_line":"            ovsdb_cfg_connections.append(\u0027tcp:127.0.0.1:%s\u0027 % _port)"},{"line_number":46,"context_line":"            manager_connections.append(\u0027ptcp:%s:127.0.0.1\u0027 % _port)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"        # Wait until all managers are in the OVS DB."},{"line_number":49,"context_line":"        n_utils.wait_until_true("},{"line_number":50,"context_line":"            functools.partial(check_num_manconn, NUM_MANAGERS, ovs.ovsdb),"},{"line_number":51,"context_line":"            timeout\u003d10,"}],"source_content_type":"text/x-python","patch_set":1,"id":"c5050685_d683a781","line":48,"in_reply_to":"ae1f2efe_1a1127ca","updated":"2025-11-12 13:47:39.000000000","message":"This is indeed a very bad implementation.\n\nFirst, not waiting in the correct place. Second, as you mentioned, a waitevent could be more efficient. Let me push a new PS","commit_id":"81d81afed8a941f9ee935dc5748cd13f7b724661"}]}