)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"390fbfc84511f347a02901d179037aef840f270a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d9fe8411_4552d92b","updated":"2026-07-14 18:33:53.000000000","message":"(I know it\u0027s too early to comment on this but ) I wonder if we want to use an existing library such as retrying instead of implementing the logic by ourselves ?","commit_id":"8554c2e089935677b2298978396589c30bd8e66b"},{"author":{"_account_id":24434,"name":"Fabian Wiesel","email":"fabian.wiesel@sap.com","username":"fwiesel"},"change_message_id":"d8391c53388fe496f60d8367e241be675bf3c0c2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"c45d86f4_4d89ed8e","in_reply_to":"9ac1bdc2_2b9c0026","updated":"2026-07-15 13:15:26.000000000","message":"Done","commit_id":"8554c2e089935677b2298978396589c30bd8e66b"},{"author":{"_account_id":24434,"name":"Fabian Wiesel","email":"fabian.wiesel@sap.com","username":"fwiesel"},"change_message_id":"bd59652f464a6911fb11b3ac5a8ba268a8c67029","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"9ac1bdc2_2b9c0026","in_reply_to":"d9fe8411_4552d92b","updated":"2026-07-15 05:04:13.000000000","message":"I would be perfectly okay with doing that, I just thought it would add another dependency.\nBut both os-brick and tooz depend on tenacity, and are pulled in by glance and cinder as well.\nOnly nova relies on retrying. \n\nI\u0027d say, tenacity would then be the sensible choice.","commit_id":"8554c2e089935677b2298978396589c30bd8e66b"},{"author":{"_account_id":24434,"name":"Fabian Wiesel","email":"fabian.wiesel@sap.com","username":"fwiesel"},"change_message_id":"29447641b07aa0dcf8a27627fcbc19392faab80f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"a463b295_c7b45f91","updated":"2026-07-15 11:52:05.000000000","message":"Actually, now on second though, I wonder it if is not an issue with `oslo_service.loopingcall.RetryDecorator` as well.","commit_id":"277048adfdfeca5aecad9099e278bf2e0ac68574"},{"author":{"_account_id":24434,"name":"Fabian Wiesel","email":"fabian.wiesel@sap.com","username":"fwiesel"},"change_message_id":"d8391c53388fe496f60d8367e241be675bf3c0c2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"7769f04c_45c2132a","updated":"2026-07-15 13:15:26.000000000","message":"Switched to tenacity, as it seems to be a smaller delta and is pulled in anyway.","commit_id":"7b6be348405c1e394691a3855e5c0ce248c87eb2"}],"oslo_vmware/api.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"28599b088ebb29f2f4f14e4fccd95dd95d1b12fe","unresolved":false,"context_lines":[{"line_number":218,"context_line":"                                    exceptions.VimConnectionException))"},{"line_number":219,"context_line":""},{"line_number":220,"context_line":"        @tenacity.retry("},{"line_number":221,"context_line":"            retry\u003dtenacity.retry_if_exception(_retry_on_overload_or_connection),"},{"line_number":222,"context_line":"            wait\u003dtenacity.wait_incrementing(start\u003d0, increment\u003d10, max\u003d60),"},{"line_number":223,"context_line":"            stop\u003d(tenacity.stop_never if self._api_retry_count \u003d\u003d -1"},{"line_number":224,"context_line":"                  else tenacity.stop_after_attempt(self._api_retry_count)),"}],"source_content_type":"text/x-python","patch_set":9,"id":"e97b1d84_29180e11","line":221,"updated":"2026-07-15 06:15:50.000000000","message":"pep8: E501 line too long (80 \u003e 79 characters)","commit_id":"3a4a44d463397a42dcc0491e9b921bd1e8f81204"}],"oslo_vmware/tests/test_api.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"5ec1ccde1decb9691e1b80cc58432252b30e172f","unresolved":false,"context_lines":[{"line_number":295,"context_line":"        module \u003d mock.Mock()"},{"line_number":296,"context_line":"        module.api \u003d api"},{"line_number":297,"context_line":"        sleep_times \u003d []"},{"line_number":298,"context_line":"        with mock.patch(\u0027time.sleep\u0027, side_effect\u003dlambda t: sleep_times.append(t)):"},{"line_number":299,"context_line":"            self.assertRaises(exceptions.VimConnectionException,"},{"line_number":300,"context_line":"                              api_session.invoke_api,"},{"line_number":301,"context_line":"                              module,"}],"source_content_type":"text/x-python","patch_set":14,"id":"ef9b11ee_33034688","line":298,"updated":"2026-07-15 15:14:25.000000000","message":"pep8: E501 line too long (83 \u003e 79 characters)","commit_id":"43b7f2655c54f538069f319b5092f31f3aa886c9"}]}