)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":33451,"name":"Yushiro Furukawa","display_name":"Yushiro Furukawa","email":"yushiro.furukawa@lycorp.co.jp","username":"yushiro2"},"change_message_id":"904981cd3dcc1265cfb42c91411a9e50b2388f48","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"19dd2643_7740eaac","updated":"2026-09-08 16:42:38.000000000","message":"These comments related to https://review.opendev.org/c/openstack/oslo.messaging/+/997470/24.  Please check it.  If it\u0027s fine, I\u0027ll fix it.","commit_id":"bda4c6802537f6cf831a7f58b5d3cb61e0013a7d"}],"oslo_messaging/_drivers/http_driver/tooz_broker.py":[{"author":{"_account_id":33451,"name":"Yushiro Furukawa","display_name":"Yushiro Furukawa","email":"yushiro.furukawa@lycorp.co.jp","username":"yushiro2"},"change_message_id":"904981cd3dcc1265cfb42c91411a9e50b2388f48","unresolved":true,"context_lines":[{"line_number":840,"context_line":"            self._prober_started \u003d True"},{"line_number":841,"context_line":""},{"line_number":842,"context_line":"        http_conf \u003d self._conf.oslo_messaging_http"},{"line_number":843,"context_line":"        proxy_endpoint \u003d http_conf.proxy_endpoint"},{"line_number":844,"context_line":"        for service in probe_services:"},{"line_number":845,"context_line":"            probe_url \u003d proxy_endpoint or service.host"},{"line_number":846,"context_line":"            tls_server_name \u003d ("}],"source_content_type":"text/x-python","patch_set":29,"id":"c95b6e95_e01006ed","line":843,"updated":"2026-09-08 16:42:38.000000000","message":"```python\n  route_proxy \u003d (\n      proxy_endpoint\n      or service.reverse_proxy_endpoint\n  )\n  probe_url \u003d route_proxy or service.host\n  tls_server_name \u003d (\n      None if route_proxy\n      else service.tls_server_name\n  )\n```\nCould you also consider proxy or reverse_proxy_endpoint like impl_http.py?","commit_id":"bda4c6802537f6cf831a7f58b5d3cb61e0013a7d"},{"author":{"_account_id":33451,"name":"Yushiro Furukawa","display_name":"Yushiro Furukawa","email":"yushiro.furukawa@lycorp.co.jp","username":"yushiro2"},"change_message_id":"f58265d49c01f90fb22d4e54b87267a94a217ed8","unresolved":false,"context_lines":[{"line_number":840,"context_line":"            self._prober_started \u003d True"},{"line_number":841,"context_line":""},{"line_number":842,"context_line":"        http_conf \u003d self._conf.oslo_messaging_http"},{"line_number":843,"context_line":"        proxy_endpoint \u003d http_conf.proxy_endpoint"},{"line_number":844,"context_line":"        for service in probe_services:"},{"line_number":845,"context_line":"            probe_url \u003d proxy_endpoint or service.host"},{"line_number":846,"context_line":"            tls_server_name \u003d ("}],"source_content_type":"text/x-python","patch_set":29,"id":"cb7a0fe0_f65cea44","line":843,"in_reply_to":"c95b6e95_e01006ed","updated":"2026-09-09 13:44:18.000000000","message":"Acknowledged","commit_id":"bda4c6802537f6cf831a7f58b5d3cb61e0013a7d"}],"oslo_messaging/tests/drivers/test_tooz_broker.py":[{"author":{"_account_id":33451,"name":"Yushiro Furukawa","display_name":"Yushiro Furukawa","email":"yushiro.furukawa@lycorp.co.jp","username":"yushiro2"},"change_message_id":"904981cd3dcc1265cfb42c91411a9e50b2388f48","unresolved":true,"context_lines":[{"line_number":1064,"context_line":"        self.assertEqual(1, len(self._groups[scheduler_group]))"},{"line_number":1065,"context_line":"        self.assertIs(replacement, broker._coordinator)"},{"line_number":1066,"context_line":""},{"line_number":1067,"context_line":"    def test_service_object_structure(self):"},{"line_number":1068,"context_line":"        broker \u003d self._make_broker()"},{"line_number":1069,"context_line":"        group \u003d tooz_broker.GROUP_PREFIX + b\u0027ironic.conductor\u0027"},{"line_number":1070,"context_line":"        self.fake_coord._groups[group] \u003d {"}],"source_content_type":"text/x-python","patch_set":29,"id":"dc21f95a_003e824b","line":1067,"updated":"2026-09-08 16:42:38.000000000","message":"Please keep `Service.host` as the direct backend URL and `Service.reverse_proxy_endpoint` as separate routing metadata. The HTTP client and endpoint prober should be responsible for selecting the effective route.","commit_id":"bda4c6802537f6cf831a7f58b5d3cb61e0013a7d"},{"author":{"_account_id":33451,"name":"Yushiro Furukawa","display_name":"Yushiro Furukawa","email":"yushiro.furukawa@lycorp.co.jp","username":"yushiro2"},"change_message_id":"904981cd3dcc1265cfb42c91411a9e50b2388f48","unresolved":true,"context_lines":[{"line_number":1313,"context_line":""},{"line_number":1314,"context_line":"        self.assertEqual(2, observe.call_count)"},{"line_number":1315,"context_line":""},{"line_number":1316,"context_line":"    def test_prober_uses_client_proxy_and_rpc_routing_headers(self):"},{"line_number":1317,"context_line":"        self.fake_coord.provides_endpoint_health \u003d False"},{"line_number":1318,"context_line":"        self.conf.set_override(\u0027endpoint_probe_interval\u0027, 15,"},{"line_number":1319,"context_line":"                               group\u003d\u0027oslo_messaging_http\u0027)"}],"source_content_type":"text/x-python","patch_set":29,"id":"b322c96f_ec890461","line":1316,"updated":"2026-09-08 16:42:38.000000000","message":"This test covers the client-proxy path, but not the target-proxy fallback. Could you add a case with no client `proxy_endpoint` and a populated `service.reverse_proxy_endpoint`, and verify that the prober uses the target proxy URL, passes no backend SNI override, and preserves the RPC routing headers?","commit_id":"bda4c6802537f6cf831a7f58b5d3cb61e0013a7d"},{"author":{"_account_id":33451,"name":"Yushiro Furukawa","display_name":"Yushiro Furukawa","email":"yushiro.furukawa@lycorp.co.jp","username":"yushiro2"},"change_message_id":"f58265d49c01f90fb22d4e54b87267a94a217ed8","unresolved":false,"context_lines":[{"line_number":1313,"context_line":""},{"line_number":1314,"context_line":"        self.assertEqual(2, observe.call_count)"},{"line_number":1315,"context_line":""},{"line_number":1316,"context_line":"    def test_prober_uses_client_proxy_and_rpc_routing_headers(self):"},{"line_number":1317,"context_line":"        self.fake_coord.provides_endpoint_health \u003d False"},{"line_number":1318,"context_line":"        self.conf.set_override(\u0027endpoint_probe_interval\u0027, 15,"},{"line_number":1319,"context_line":"                               group\u003d\u0027oslo_messaging_http\u0027)"}],"source_content_type":"text/x-python","patch_set":29,"id":"4ea19787_1ec1bd88","line":1316,"in_reply_to":"b322c96f_ec890461","updated":"2026-09-09 13:44:18.000000000","message":"Acknowledged","commit_id":"bda4c6802537f6cf831a7f58b5d3cb61e0013a7d"}]}