)]}'
{".zuul.yaml":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"95d36f4521b862f3fc4a0ee636d4fdfbd128a080","unresolved":true,"context_lines":[{"line_number":239,"context_line":"      tox_envlist: functional-py314-threading"},{"line_number":240,"context_line":""},{"line_number":241,"context_line":"- job:"},{"line_number":242,"context_line":"    name: cinder-tempest-integrated-storage-no-eventlet"},{"line_number":243,"context_line":"    parent: tempest-integrated-storage"},{"line_number":244,"context_line":"    description: |"},{"line_number":245,"context_line":"      Run tempest integration tests with Cinder using native threading"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"4fc30930_b5b3089d","line":242,"updated":"2026-06-02 14:16:52.000000000","message":"tempest-integrated-storage doesn\u0027t run cinder-tempest-plugin -- we should base this on the lio-barbican job instead.  (Will probably want a cinder-plugin-ceph-tempest variant later as well.)","commit_id":"f4129fdcf0f93269873b269b6bb6730e86c778ee"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"efa0cc6948cac8a263bf6f654b0bf29faef04613","unresolved":true,"context_lines":[{"line_number":239,"context_line":"      tox_envlist: functional-py314-threading"},{"line_number":240,"context_line":""},{"line_number":241,"context_line":"- job:"},{"line_number":242,"context_line":"    name: cinder-tempest-integrated-storage-no-eventlet"},{"line_number":243,"context_line":"    parent: tempest-integrated-storage"},{"line_number":244,"context_line":"    description: |"},{"line_number":245,"context_line":"      Run tempest integration tests with Cinder using native threading"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5eeaf3e8_8b952799","line":242,"in_reply_to":"4fc30930_b5b3089d","updated":"2026-06-03 08:47:02.000000000","message":"ack, updated to use cinder-tempest-plugin-lvm-lio-barbican as parent.","commit_id":"f4129fdcf0f93269873b269b6bb6730e86c778ee"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"41d28969d5e10b73c39f24fe24cc3594cf3bcf27","unresolved":false,"context_lines":[{"line_number":239,"context_line":"      tox_envlist: functional-py314-threading"},{"line_number":240,"context_line":""},{"line_number":241,"context_line":"- job:"},{"line_number":242,"context_line":"    name: cinder-tempest-integrated-storage-no-eventlet"},{"line_number":243,"context_line":"    parent: tempest-integrated-storage"},{"line_number":244,"context_line":"    description: |"},{"line_number":245,"context_line":"      Run tempest integration tests with Cinder using native threading"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"fa5f1608_22462f0f","line":242,"in_reply_to":"5eeaf3e8_8b952799","updated":"2026-06-11 07:27:31.000000000","message":"Done","commit_id":"f4129fdcf0f93269873b269b6bb6730e86c778ee"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"23ad0623dfc437acd0a37158bdbb403bb630ec5b","unresolved":true,"context_lines":[{"line_number":249,"context_line":"      issues as part of the OpenStack-wide eventlet removal goal."},{"line_number":250,"context_line":"    vars:"},{"line_number":251,"context_line":"      devstack_localrc:"},{"line_number":252,"context_line":"        OS_CINDER_DISABLE_EVENTLET_PATCHING: 1"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"- job:"},{"line_number":255,"context_line":"    name: cinder-rally-task"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"86927856_02b0ce37","line":252,"updated":"2026-06-04 13:57:50.000000000","message":"I don\u0027t think this works - the c-vol logs are full of oslo_service.backend._eventlet.service activity.\n\nSetting the var like this here implies that devstack knows about OS_CINDER_DISABLE_EVENTLET_PATCHING, but it does not.  See https://review.opendev.org/c/openstack/manila-tempest-plugin/+/972973 for how to do this via systemd vars.","commit_id":"96117628ed359cff8cc0a2520ed49fa35e58fb5b"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"0e13b5063d101dc1c1782bfaffdc88a98b9872cf","unresolved":true,"context_lines":[{"line_number":249,"context_line":"      issues as part of the OpenStack-wide eventlet removal goal."},{"line_number":250,"context_line":"    vars:"},{"line_number":251,"context_line":"      devstack_localrc:"},{"line_number":252,"context_line":"        OS_CINDER_DISABLE_EVENTLET_PATCHING: 1"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"- job:"},{"line_number":255,"context_line":"    name: cinder-rally-task"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1888bd01_70bebd9f","line":252,"in_reply_to":"86927856_02b0ce37","updated":"2026-06-11 13:10:54.000000000","message":"yes seystemd-env-var setting for each service works","commit_id":"96117628ed359cff8cc0a2520ed49fa35e58fb5b"},{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"612ef220426204e9f6b4d7694b60019cc24f8c69","unresolved":true,"context_lines":[{"line_number":188,"context_line":"      issues as part of the OpenStack-wide eventlet removal goal."},{"line_number":189,"context_line":"    vars:"},{"line_number":190,"context_line":"      devstack_localrc:"},{"line_number":191,"context_line":"        \u0027SYSTEMD_ENV_VARS[\"c-api\"]\u0027: OS_CINDER_DISABLE_EVENTLET_PATCHING\u003d1"},{"line_number":192,"context_line":"        \u0027SYSTEMD_ENV_VARS[\"c-sch\"]\u0027: OS_CINDER_DISABLE_EVENTLET_PATCHING\u003d1"},{"line_number":193,"context_line":"        \u0027SYSTEMD_ENV_VARS[\"c-vol\"]\u0027: OS_CINDER_DISABLE_EVENTLET_PATCHING\u003d1"},{"line_number":194,"context_line":"        \u0027SYSTEMD_ENV_VARS[\"c-bak\"]\u0027: OS_CINDER_DISABLE_EVENTLET_PATCHING\u003d1"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"c8159e7a_ceaec21b","line":191,"updated":"2026-06-16 15:46:40.000000000","message":"nit: Why not set true instead of 1 for OS_CINDER_DISABLE_EVENTLET_PATCHING? This isn\u0027t really an issue, but it\u0027s the standard for other services like Manila and Nova.","commit_id":"cc4fc51ba411ae54a21298bec75cc72fbcd4ace4"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"6a33b391e04d49ad4e0e87ecc7d350a8b98984b5","unresolved":true,"context_lines":[{"line_number":188,"context_line":"      issues as part of the OpenStack-wide eventlet removal goal."},{"line_number":189,"context_line":"    vars:"},{"line_number":190,"context_line":"      devstack_localrc:"},{"line_number":191,"context_line":"        \u0027SYSTEMD_ENV_VARS[\"c-api\"]\u0027: OS_CINDER_DISABLE_EVENTLET_PATCHING\u003d1"},{"line_number":192,"context_line":"        \u0027SYSTEMD_ENV_VARS[\"c-sch\"]\u0027: OS_CINDER_DISABLE_EVENTLET_PATCHING\u003d1"},{"line_number":193,"context_line":"        \u0027SYSTEMD_ENV_VARS[\"c-vol\"]\u0027: OS_CINDER_DISABLE_EVENTLET_PATCHING\u003d1"},{"line_number":194,"context_line":"        \u0027SYSTEMD_ENV_VARS[\"c-bak\"]\u0027: OS_CINDER_DISABLE_EVENTLET_PATCHING\u003d1"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"8b308677_326552b0","line":191,"in_reply_to":"c8159e7a_ceaec21b","updated":"2026-06-17 07:06:00.000000000","message":"yeah, so I went with existing changes,realted to unit and functional tests which I made originally which were refrenced from money_patching.py without knowing what others are using (I did refered them btw but did not noticed this)\n\nthough this does not matter much because as per monkey_patching (1, true, yes ) are valid values, I\u0027ll update this for consistancy.\nthanks for noticing.","commit_id":"cc4fc51ba411ae54a21298bec75cc72fbcd4ace4"}],"/COMMIT_MSG":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"97d6c90af7c2d58254978d1f701bc6bbeebeefe5","unresolved":true,"context_lines":[{"line_number":11,"context_line":"disable eventlet patching for all Cinder tests in tempest"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Using cinder-tempest-plugin-lvm-lio-barbican as parent ensures we run"},{"line_number":14,"context_line":"the full cinder-tempest-plugin test suit, not just basic tempest tests."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Depends-On: https://review.opendev.org/c/openstack/cinder/+/992876"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"9eff1bbc_f146f63c","line":14,"updated":"2026-06-15 10:07:28.000000000","message":"can you correct test suit to test suite? Also in commit message.","commit_id":"cc4fc51ba411ae54a21298bec75cc72fbcd4ace4"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"3d83a0a7b67f89f093b116c7921eee40f9405be6","unresolved":false,"context_lines":[{"line_number":11,"context_line":"disable eventlet patching for all Cinder tests in tempest"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Using cinder-tempest-plugin-lvm-lio-barbican as parent ensures we run"},{"line_number":14,"context_line":"the full cinder-tempest-plugin test suit, not just basic tempest tests."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Depends-On: https://review.opendev.org/c/openstack/cinder/+/992876"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"af42a570_bdd8c6df","line":14,"in_reply_to":"9eff1bbc_f146f63c","updated":"2026-06-16 06:12:19.000000000","message":"thanks for review, I\u0027ll update this if another PS required.\n\nanyway I am thinking to add codespell for spell nits.","commit_id":"cc4fc51ba411ae54a21298bec75cc72fbcd4ace4"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"3253cc1a7e5aaf735835798d98e9712146425f29","unresolved":true,"context_lines":[{"line_number":11,"context_line":"disable eventlet patching for all Cinder tests in tempest"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Using cinder-tempest-plugin-lvm-lio-barbican as parent ensures we run"},{"line_number":14,"context_line":"the full cinder-tempest-plugin test suit, not just basic tempest tests."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Depends-On: https://review.opendev.org/c/openstack/cinder/+/992876"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"86ef4945_1f9a2b56","line":14,"in_reply_to":"af42a570_bdd8c6df","updated":"2026-06-17 07:07:17.000000000","message":"fixed the typo in comit msg","commit_id":"cc4fc51ba411ae54a21298bec75cc72fbcd4ace4"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"3a1301ddd1df3916b8e63eb1dcfe26e1ade6d219","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"78df90aa_fbd4a272","updated":"2026-06-04 08:35:44.000000000","message":"recheck\nopenstack-tox-py313 timeedout","commit_id":"96117628ed359cff8cc0a2520ed49fa35e58fb5b"},{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"612ef220426204e9f6b4d7694b60019cc24f8c69","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"c78aca4e_96b39282","updated":"2026-06-16 15:46:40.000000000","message":"Hi Amit, just a small nit for you to consider.","commit_id":"cc4fc51ba411ae54a21298bec75cc72fbcd4ace4"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b6152f24ac7957fedbda4f27f50e3866e20167a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"93190c64_586cd7a9","updated":"2026-06-12 15:44:01.000000000","message":"This looks good to me, but we need to get the dependency sorted out before it can merge.","commit_id":"cc4fc51ba411ae54a21298bec75cc72fbcd4ace4"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b4c459098b49989a5fbe8178527a2683cce2769c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"3790e1df_b0cb79e4","updated":"2026-06-18 14:01:09.000000000","message":"Change LGTM, but I agree with Fernando about the commit message.  First line is supposed to be limited to 50 chars [0], maybe something like\n\n```\nAdd job using native threading only (no-eventlet)\n```\n\n[0] https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"001557719b9b3f9933fab56b2283ab8f7c7afe1a"},{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"b7f89393e3fd45eedf97a74034e37bbfaf950dea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"c0560ed3_1fc87722","updated":"2026-06-17 16:59:45.000000000","message":"Not a big problem but as the commit message well describes the change I believe you can shorten the commit title a bit. Apart of that the patch looks good to me.","commit_id":"001557719b9b3f9933fab56b2283ab8f7c7afe1a"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"c7d8f18367ed7b880ddf4feb95ef8af87832f26a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"3e498ec5_17a45def","in_reply_to":"3790e1df_b0cb79e4","updated":"2026-06-18 14:10:07.000000000","message":"ack sounds good, I respin it once 992876 is mered, to remove depends-on and fix commit msg. thanks","commit_id":"001557719b9b3f9933fab56b2283ab8f7c7afe1a"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"3800c0574f7c45b70801e6c2e94347ac72021a75","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"0d50b8f1_1ae54688","in_reply_to":"3e498ec5_17a45def","updated":"2026-06-19 06:43:38.000000000","message":"Done","commit_id":"001557719b9b3f9933fab56b2283ab8f7c7afe1a"}],"cinder/cmd/backup.py":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"41d28969d5e10b73c39f24fe24cc3594cf3bcf27","unresolved":true,"context_lines":[{"line_number":129,"context_line":"                                        service_name\u003d\u0027backup\u0027,"},{"line_number":130,"context_line":"                                        process_number\u003d1,"},{"line_number":131,"context_line":"                                        semaphore\u003dsemaphore)"},{"line_number":132,"context_line":"        service.serve(server, workers\u003d1)"},{"line_number":133,"context_line":"        service.wait()"}],"source_content_type":"text/x-python","patch_set":6,"id":"45c2a141_d9eb3fa8","line":132,"updated":"2026-06-11 07:27:31.000000000","message":"this does fix the startup issue in CI, but this is not the correct sol.\neither we should make it CONSTANT or configurable.\n\nI\u0027ll move it to other patch, so there this could be discussed in detail, also I do not want to block this patch because of this issue.","commit_id":"470a4ade4e34d5b827c8a3001535c86d489c94ce"}]}