)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"065487b56953fbba86c4ca02158a3c900d597007","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"086bd796_ae1e6ddf","updated":"2025-06-17 15:34:38.000000000","message":"I\u0027ve skimmed through some of the files, and I like some of the changes that it does (removing old style class defintions with object, removing old-style usages of super with the class name, introducing f-strings). Realistically, the only way this will get done is through an automated tool like this, but a change so big is worrisome because of the size and the not fully reliable test coverage we have in watcher. Overall I think I am in favor of the switch, unless we see significat breakages in CI","commit_id":"b39906fb5eac4453708fcb28fff554762dd5dbf9"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"04407626a357ddb2434e3c453b606b3e6f563633","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c91c6b40_2a682f14","updated":"2025-06-24 04:10:55.000000000","message":"If we fix the merge conflict, we will try to do cherry-pick downstream while testing to confirm that it does not break anything in downstream.\n\nIf all works fine, we can merge that.","commit_id":"b39906fb5eac4453708fcb28fff554762dd5dbf9"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"52be7a40ad5e60c5d245d8de076adc0fab752f6a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"741a3d23_87c84300","updated":"2025-04-24 11:05:04.000000000","message":"ok i need to split this.\n\nthis is failign becuase of the way we are current packaging the wsgi script that we use and how it invoked form devstack\n\nthat not unexpsect as to make this work proeprly we need to swap to using a wsgi module in a slighlty diffent way. we have don the same change in other porject so ill do that in a separate commit first to make it clear what changes are require to swap to haveing a pyproject.toml\n\nnot this is alos needed fo rdevstack on fedora/centos to work.","commit_id":"b39906fb5eac4453708fcb28fff554762dd5dbf9"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"9700391a6c284c4d35d4ae1af07456194dfb0471","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"9c9d5c42_fd63f11f","updated":"2025-07-15 11:44:58.000000000","message":"It is a large change. If someone cherry-pick .pre-commit-config.yaml changes to apply ruff downstream. It brings lots of changes to the codebase which are hard to fix automatically. It requires manual intervention case by case.\n\nThe only benefit of this change it improves readability apart from that I am not seeing any other value. That\u0027s why I am putting -1 on this.\n\nThank you Sean once again putting the time to adding this pr.\n\nBased on discussion with Sean, we can bring this cr during PTG to know what other reviewers think about that.","commit_id":"7f44d54a00f16e94a2811db4a30c505edb7386c0"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"759967f7624f041afeb44ec1d5faa7b9816156c0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"9a395c0b_cc4e12fd","updated":"2025-07-04 07:27:51.000000000","message":"check experimental","commit_id":"7f44d54a00f16e94a2811db4a30c505edb7386c0"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"7882cb64691698f8b7587ca1e2e591427717970a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b20a3949_d8985cf0","updated":"2025-07-04 07:22:07.000000000","message":"https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_483/openstack/483312d2b1954c89843e1b565181fa18/testr_results.html\n```\nTraceback (most recent call last):\n  File \"/opt/stack/tempest/.tox/tempest/lib/python3.12/site-packages/watcher_tempest_plugin/tests/scenario/test_execute_workload_balance.py\", line 83, in test_execute_workload_balance_strategy_cpu\n    self.execute_strategy(goal_name, strategy_name,\n  File \"/opt/stack/tempest/.tox/tempest/lib/python3.12/site-packages/watcher_tempest_plugin/tests/scenario/base.py\", line 929, in execute_strategy\n    self.fail(\"The audit has found action types %s when expecting \"\n  File \"/usr/lib/python3.12/unittest/case.py\", line 715, in fail\n    raise self.failureException(msg)\nAssertionError: The audit has found action types set() when expecting [\u0027migrate\u0027]\n\n```","commit_id":"7f44d54a00f16e94a2811db4a30c505edb7386c0"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"484d187dbed068fa811e73e32ed86096215bd4f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"222e1f19_04360354","updated":"2025-07-04 07:22:15.000000000","message":"recheck","commit_id":"7f44d54a00f16e94a2811db4a30c505edb7386c0"}]}