)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"45c5220f9a964021252fb5cd71a4e37537fea1c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7ad30d89_83a7f15c","updated":"2023-02-06 15:36:43.000000000","message":"I am fine with it, if there will be another round, elvira\u0027s comment should be considered","commit_id":"10adc0ba6b42ddaafe4c8ed8f3ef55bf6e0b61ae"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"b18cf7ca6fa258242d4ff61d2935b68179a329e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2305a002_c50ef79a","updated":"2023-02-06 10:01:29.000000000","message":"Just wondering: Should we have any testing for this?","commit_id":"10adc0ba6b42ddaafe4c8ed8f3ef55bf6e0b61ae"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"ec6d40c2ee5eb26052dbd7b8c33679768dad0dd8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"395523a6_c80a644f","updated":"2023-02-06 09:01:07.000000000","message":"recheck\ntest_ovs_and_ovs_events failure seems irrelevant","commit_id":"10adc0ba6b42ddaafe4c8ed8f3ef55bf6e0b61ae"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"f31e9fd28e6553bab5f4cc5bf13ce061991c14d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7284deb7_8562a502","updated":"2023-02-03 16:39:48.000000000","message":"recheck fullstack","commit_id":"10adc0ba6b42ddaafe4c8ed8f3ef55bf6e0b61ae"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"5d25af1520a498bf909f20dee6009742b728dcaa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"93d70a0e_b3642218","updated":"2023-02-03 15:13:51.000000000","message":"thanks, it seems to be ok, I am fine with fixing it here in Neutron or in networking -sfc also.","commit_id":"10adc0ba6b42ddaafe4c8ed8f3ef55bf6e0b61ae"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7369058f7e7400f56a23d4d514bbcfcaca2955bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"91753701_d3559436","in_reply_to":"2305a002_c50ef79a","updated":"2023-02-06 16:32:45.000000000","message":"The only new code added is [1]. I think it is not necessary to test it. In any case, I\u0027ve create a new UT class.\n\n[1]https://review.opendev.org/c/openstack/neutron/+/872644/2/neutron/conf/service.py","commit_id":"10adc0ba6b42ddaafe4c8ed8f3ef55bf6e0b61ae"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"edb9475e6df81679d6c828862936507eb62dbda4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fa9ada69_5b3df7b7","updated":"2023-02-08 14:00:41.000000000","message":"recheck FT error under investigation now\n","commit_id":"47fef55e25fb1b7d3c6c7d6083a916e26fd45b6d"}],"neutron/conf/service.py":[{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"3ac7fb71c427fa2d926f9ebd4105120e50f1e3ae","unresolved":true,"context_lines":[{"line_number":55,"context_line":"    conf.register_opts(opts)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"def get_rpc_workers(conf\u003dcfg.CONF):"},{"line_number":59,"context_line":"    try:"},{"line_number":60,"context_line":"        return conf.rpc_workers"},{"line_number":61,"context_line":"    except cfg.NoSuchOptError:"}],"source_content_type":"text/x-python","patch_set":2,"id":"e718395b_4b7fcc88","line":58,"range":{"start_line":58,"start_character":4,"end_line":58,"end_character":35},"updated":"2023-02-06 03:49:24.000000000","message":"Nit. I think this function should have some doc string indicating it will perform implicit register","commit_id":"10adc0ba6b42ddaafe4c8ed8f3ef55bf6e0b61ae"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a60447882d0d8bee2b3a07b6a7b9db3d6fe28e44","unresolved":false,"context_lines":[{"line_number":55,"context_line":"    conf.register_opts(opts)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"def get_rpc_workers(conf\u003dcfg.CONF):"},{"line_number":59,"context_line":"    try:"},{"line_number":60,"context_line":"        return conf.rpc_workers"},{"line_number":61,"context_line":"    except cfg.NoSuchOptError:"}],"source_content_type":"text/x-python","patch_set":2,"id":"4dfbadff_50951c7b","line":58,"range":{"start_line":58,"start_character":4,"end_line":58,"end_character":35},"in_reply_to":"e718395b_4b7fcc88","updated":"2023-02-08 14:00:14.000000000","message":"Done","commit_id":"10adc0ba6b42ddaafe4c8ed8f3ef55bf6e0b61ae"}]}