)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f76be02795230445ae411efd4438f7db5fc19d31","unresolved":true,"context_lines":[{"line_number":11,"context_line":"This is not enabled by default and need a backend_url to be set in"},{"line_number":12,"context_line":"config for this feature to be enabled."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"During the eventlet removal, we identified that some drivers in tooz"},{"line_number":15,"context_line":"rely on the fact eventlet monkeypatching is enabled."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Since we want to get rid of eventlet, and because we think this services"},{"line_number":18,"context_line":"endpoint is not widely used, let\u0027s deprecate that feature so we can move"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"216de81c_9bbed0ca","line":15,"range":{"start_line":14,"start_character":29,"end_line":15,"end_character":52},"updated":"2025-02-25 00:45:45.000000000","message":"I\u0027m curious to know which drivers are you referring to ?","commit_id":"c095c3715d5506e6a7bee3aa56ca1fd8cf08b551"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"7880cd898fb317d862f71500f6a36d89796a08f0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d4a95ccb_61125005","updated":"2025-03-04 15:30:39.000000000","message":"Anyway, let\u0027s go with this deprecation.\nMaybe we will reintroduce something later for /v2/services\nBut it\u0027s not worth it for now","commit_id":"c095c3715d5506e6a7bee3aa56ca1fd8cf08b551"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"46ff9be0b29c00926ee8c4ebe056f28ffd04f7eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"16537d2a_c17ff2fd","updated":"2025-02-25 06:51:54.000000000","message":"hey!\nthe file driver at least.\nI have not dig that much into it.\nIt seems that the executor thread got stuck at some point.\n\nthe initial patch introducing coordination in mistral is also explicitely enabling eventlet for the same reason, and it seems that etcd driver was used in that situation.\n\nanyway, I was really aurprised to see that coordination is used only for services api, which seems pretty useless to me.\nDo you have an idea about uage of this?","commit_id":"c095c3715d5506e6a7bee3aa56ca1fd8cf08b551"},{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"287eaa2510c5ae88eb19b23095d14c1d121f269c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"10e4bf1d_718e3aad","updated":"2025-03-04 22:23:24.000000000","message":"recheck","commit_id":"c095c3715d5506e6a7bee3aa56ca1fd8cf08b551"},{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"b3c4c1990e026e41847c9ed7181a8d79ac24bb8e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"486d84e5_43b1fb68","updated":"2025-03-04 18:19:14.000000000","message":"recheck","commit_id":"c095c3715d5506e6a7bee3aa56ca1fd8cf08b551"},{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"6ed7ea79f6ff747f208d5df794e8781f981120d9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c3540211_6dcd12b1","updated":"2025-03-04 21:24:33.000000000","message":"recheck","commit_id":"c095c3715d5506e6a7bee3aa56ca1fd8cf08b551"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f9a134e119f0d828f72a362f45bc1dc7d52ae31b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a9766fd6_7adc6051","in_reply_to":"16537d2a_c17ff2fd","updated":"2025-02-25 07:20:58.000000000","message":"I personally think having service list API is useful to monitor service status (for example in Puppet we use this API for saniy check to ensure all services are deployed and running), but I agree it sounds odd to use coordination for the API. Probably you may want to implement the service management using DB, though I know it requires additional handling (like purging old service records) and am not too sure if API is worth implementing with that complexity as trade-off.","commit_id":"c095c3715d5506e6a7bee3aa56ca1fd8cf08b551"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"28447852d95abb57f5c690eb54a6d2e84635bb8a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"65129dee_6911cde1","in_reply_to":"a9766fd6_7adc6051","updated":"2025-02-25 19:53:50.000000000","message":"I have been a little bit deeper into the file driver and it seems that mistral code got stuck because of the max_worker\u003d1 given to the ThreadPoolExecutor [1].\nIf I switch the value to 2, it works.\n\n[1] https://opendev.org/openstack/tooz/src/commit/379a3bd2cc9ebc999ed583fad76597c225635685/tooz/utils.py#L54","commit_id":"c095c3715d5506e6a7bee3aa56ca1fd8cf08b551"}]}
