)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"35191d7641de4dfdef7c3968ee56aaedfe2b116d","unresolved":true,"context_lines":[{"line_number":21,"context_line":"/usr/lib64/python3.13/unittest/case.py:597: RuntimeWarning: TestResult has no addDuration method"},{"line_number":22,"context_line":"  warnings.warn(\"TestResult has no addDuration method\","},{"line_number":23,"context_line":"{0} unittest.loader._FailedTest.watcher_tempest_plugin.tests.client_functional.v1.test_action [0.000071s] ... FAILED"},{"line_number":24,"context_line":"```"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Related-Bug: 2100741"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"f5c37da7_27fe86a2","line":24,"updated":"2025-03-20 12:54:58.000000000","message":"a deprecation shoudl not cause the test to fail\n\nyou seam to have had a diffent bug in your patch wehn you generate this\n\ncan you update it to a fixed example","commit_id":"750377d879dc765cd26106fd89dab900e3a6a1bc"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"0f3c76e4aab308803e96b7301e05b31a64174e55","unresolved":true,"context_lines":[{"line_number":21,"context_line":"/usr/lib64/python3.13/unittest/case.py:597: RuntimeWarning: TestResult has no addDuration method"},{"line_number":22,"context_line":"  warnings.warn(\"TestResult has no addDuration method\","},{"line_number":23,"context_line":"{0} unittest.loader._FailedTest.watcher_tempest_plugin.tests.client_functional.v1.test_action [0.000071s] ... FAILED"},{"line_number":24,"context_line":"```"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Related-Bug: 2100741"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"22c598cb_da1004b4","line":24,"in_reply_to":"f5c37da7_27fe86a2","updated":"2025-03-25 08:25:45.000000000","message":"It was failing because I was running the tests without openrc file.","commit_id":"750377d879dc765cd26106fd89dab900e3a6a1bc"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"0cb9d49fc3dde4584d5e118e1b8f27d2901b00cf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"76b336cd_f5d88cf9","updated":"2025-03-04 10:04:17.000000000","message":"recheck","commit_id":"882a0463ecf90a4de0f02102f26377205266f20c"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"b52351539b334a58993559b3de56b2f72fcdab52","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"57aae60f_8ea24040","updated":"2025-03-05 13:07:40.000000000","message":"recheck","commit_id":"ff4a0301f1f16ea119007a2d69b0e604f9bab7de"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"bc7215305703adac0b69bd1eb5452a66bfa412b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"92b5b84f_76e23f51","updated":"2025-03-04 13:13:47.000000000","message":"recheck","commit_id":"ff4a0301f1f16ea119007a2d69b0e604f9bab7de"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"8c1d211abc68ff3100478d36d59c3ae99597813b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c6802512_566e97e8","updated":"2025-03-04 10:22:56.000000000","message":"recheck","commit_id":"ff4a0301f1f16ea119007a2d69b0e604f9bab7de"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"8b1186e4049e4de6827e9f7563d0cb749414986a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e47df268_3507f09a","updated":"2025-03-04 13:02:27.000000000","message":"recheck","commit_id":"ff4a0301f1f16ea119007a2d69b0e604f9bab7de"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"3a8fa653a116bb0388d21aac7fc7c2fdb3cf4d34","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f6c893f5_e7f34832","updated":"2025-03-06 06:38:37.000000000","message":"recheck","commit_id":"ff4a0301f1f16ea119007a2d69b0e604f9bab7de"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"ff2333485174f0784db9cfdc88aa41a2e5391fe1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7fed9dc8_7103ae82","updated":"2025-03-14 11:58:04.000000000","message":"We can drop tests here since this is a branchless repo, so all watcher stable branches depends on this repo to run tests.\nWe can only drop if we move all maintained stable branches to the new job too","commit_id":"ba7e387dfef81a8032ce4f446bd8abbb03de68e8"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"6bd72bf68550c867a05edbe2b9cac3e9a4566eb1","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d3dd28c5_33e3728d","in_reply_to":"03c15947_7c2991bd","updated":"2025-03-17 07:26:44.000000000","message":"First options sounds good. I will propose the rest of the patches.","commit_id":"ba7e387dfef81a8032ce4f446bd8abbb03de68e8"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"d86d3eaacc76bccaaeef4b5134d7a0665760ec64","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"555e87b8_604e4f0e","in_reply_to":"11add8ce_835463d7","updated":"2025-03-26 17:53:14.000000000","message":"I do not think we need the deprecation warning for tests moving but if we do there is no harm also.\n\nOnce you move stable branches to use the test from pythonclient then you can remove the tests from this repo instead of waiting for them to be EOL/unmaintained.","commit_id":"ba7e387dfef81a8032ce4f446bd8abbb03de68e8"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"9b9b41b93c026638270c308c8ddc25e11f9f4465","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e88bf5de_2793e7a1","in_reply_to":"7fed9dc8_7103ae82","updated":"2025-03-14 11:58:57.000000000","message":"s/we can/we can´t","commit_id":"ba7e387dfef81a8032ce4f446bd8abbb03de68e8"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"86a2b58dc4ef6d53c1d52fe6ae3c41744f517e78","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"dcd1c645_55b731e5","in_reply_to":"d15afde2_4d286bef","updated":"2025-03-19 00:41:56.000000000","message":"backporting the test move would not really be inline wiht stable policy.\ni think we are btter just keepign the test here and removing them in 18 months when 2024.2 is eol","commit_id":"ba7e387dfef81a8032ce4f446bd8abbb03de68e8"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"7504be8d9300c181e89af64a4f9f74842c2b6574","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f2afafc0_19ef5720","in_reply_to":"d3dd28c5_33e3728d","updated":"2025-03-17 07:39:47.000000000","message":"Below are the backports\nhttps://review.opendev.org/c/openstack/python-watcherclient/+/944623?usp\u003dcherry-pick\nhttps://review.opendev.org/c/openstack/python-watcherclient/+/944624?usp\u003dcherry-pick\nhttps://review.opendev.org/c/openstack/python-watcherclient/+/944625?usp\u003dcherry-pick\n\nDo we also need to backport it to stable/2023.2 release ?","commit_id":"ba7e387dfef81a8032ce4f446bd8abbb03de68e8"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"826269ded472b7e07a0eead03b7253e25e8a449f","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"11add8ce_835463d7","in_reply_to":"dcd1c645_55b731e5","updated":"2025-03-19 04:54:49.000000000","message":"Thank you Sean mooney for the review. I am fine with keeping the tests here as it is. is it ok If i mark these tests deprecated with a release note and remove zuul job from here in this patch?","commit_id":"ba7e387dfef81a8032ce4f446bd8abbb03de68e8"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"c01794bfe018031b644435e70265edbf649c2ab3","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"03c15947_7c2991bd","in_reply_to":"e88bf5de_2793e7a1","updated":"2025-03-14 14:50:15.000000000","message":"Some options are:\n1) Drop this content and also remove watcherclient tests from stable branches\n2) Maintain this code here as deprecated for a couple of releases, to not break stable branch job.","commit_id":"ba7e387dfef81a8032ce4f446bd8abbb03de68e8"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"129fa322af2cbd805ed947bb6be0f61d954c4784","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d15afde2_4d286bef","in_reply_to":"f2afafc0_19ef5720","updated":"2025-03-17 10:28:24.000000000","message":"@viroel@gmail.com Instead of dropping the job `watcherclient-tempest-functional`. I have backported the functional tests to stable release. This will make sure it does not break anything from branchless watcher-tempest-plugin repo.\n\nAm I going in correct direction?","commit_id":"ba7e387dfef81a8032ce4f446bd8abbb03de68e8"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"4ac182c03d1f74c7c0a9698355f3828bb3ac0667","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"65f4774b_5c908aff","updated":"2025-03-26 15:00:34.000000000","message":"v7 was correct v8 is wrong","commit_id":"932b6cb2619debb2f19b672177a2e65cd0753d56"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"d86d3eaacc76bccaaeef4b5134d7a0665760ec64","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"1680a995_8121192a","updated":"2025-03-26 17:53:14.000000000","message":"warning is logged correctly https://zuul.opendev.org/t/openstack/build/175578d652e44550b22f85b62e692291/log/job-output.txt#28236","commit_id":"1c9537921139b0b8ad67a182d97dbd369e1107b3"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"510b025833a2a0233db314f46fb6c5ef4d402ab5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"2380a633_890037ce","updated":"2025-04-02 10:41:50.000000000","message":"recheck","commit_id":"70186c84b4c193e3f09328fc93d11f094f61176d"},{"author":{"_account_id":7186,"name":"Matthieu Huin","email":"mhuin@redhat.com","username":"mhu"},"change_message_id":"19034908195ca0cf5b7de1d8031a69af9812458e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"da1353cc_4b47267b","updated":"2025-03-28 10:50:27.000000000","message":"recheck","commit_id":"70186c84b4c193e3f09328fc93d11f094f61176d"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"13378b5bf751e7214a1bbb9fff66f2a4d17b57cf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"ce5cc01b_23be253c","updated":"2025-03-26 17:54:11.000000000","message":"restoring my +2, fixed import order only","commit_id":"70186c84b4c193e3f09328fc93d11f094f61176d"}],"watcher_tempest_plugin/tests/client_functional/__init__.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"35191d7641de4dfdef7c3968ee56aaedfe2b116d","unresolved":true,"context_lines":[{"line_number":4,"context_line":"    \"The watcher_tempest_plugin.tests.client_functional module (client \""},{"line_number":5,"context_line":"    \"functional module) is moved to python-watcherclient repo and is \""},{"line_number":6,"context_line":"    \"deprecated. It will  be removed y 2024.2 EOL release.\""},{"line_number":7,"context_line":"    \"\", DeprecationWarning)"}],"source_content_type":"text/x-python","patch_set":4,"id":"e27e8a48_4703d6f4","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":2},"updated":"2025-03-20 12:54:58.000000000","message":"DeprecationWarning  is not in scope\n\nyou should use the debth collectior module deprecation funcitons instead.\n\n\nhttps://github.com/openstack/debtcollector/blob/master/debtcollector/removals.py#L298","commit_id":"750377d879dc765cd26106fd89dab900e3a6a1bc"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"8833760ddf7d2ebb4bcfeebdbeffd0317470676b","unresolved":true,"context_lines":[{"line_number":4,"context_line":"    \"The watcher_tempest_plugin.tests.client_functional module (client \""},{"line_number":5,"context_line":"    \"functional module) is moved to python-watcherclient repo and is \""},{"line_number":6,"context_line":"    \"deprecated. It will  be removed y 2024.2 EOL release.\""},{"line_number":7,"context_line":"    \"\", DeprecationWarning)"}],"source_content_type":"text/x-python","patch_set":4,"id":"ca48e0ca_15cce55b","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":2},"in_reply_to":"6706066a_120a26e1","updated":"2025-03-25 08:33:39.000000000","message":"I might be doing something wrong here https://a3fd8c27c0382f01ec6b-02618f010e74d581319c83aa0d27e1a8.ssl.cf5.rackcdn.com/openstack/bbed1a09d95b4bf591525cfe939536d6/job-output.txt.\n```\nall run-test: commands[1] | tempest run --regex watcher_tempest_plugin.tests.client_functional --concurrency\u003d1\n2025-03-25 07:57:05.730380 | controller | {0} watcher_tempest_plugin.tests.client_functional.v1.test_action.ActionTests.test_action_detailed_list [2.040132s] ... ok\n2025-03-25 07:57:07.806799 | controller | {0} watcher_tempest_plugin.tests.client_functional.v1.test_action.ActionTests.test_action_list [2.076841s] ... ok\n2025-03-25 07:57:11.989970 | controller | {0} watcher_tempest_plugin.tests.client_functional.v1.test_action.ActionTests.test_action_show [4.181223s] ... ok\n```\nIt should have printed some deprecation message based on https://github.com/openstack/debtcollector/blob/master/debtcollector/removals.py#L312 .\nBut it is not showing anything.","commit_id":"750377d879dc765cd26106fd89dab900e3a6a1bc"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"af8e68f96d40117091883d0e5a30a2a0ef813b25","unresolved":true,"context_lines":[{"line_number":4,"context_line":"    \"The watcher_tempest_plugin.tests.client_functional module (client \""},{"line_number":5,"context_line":"    \"functional module) is moved to python-watcherclient repo and is \""},{"line_number":6,"context_line":"    \"deprecated. It will  be removed y 2024.2 EOL release.\""},{"line_number":7,"context_line":"    \"\", DeprecationWarning)"}],"source_content_type":"text/x-python","patch_set":4,"id":"6706066a_120a26e1","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":2},"in_reply_to":"e27e8a48_4703d6f4","updated":"2025-03-25 07:28:37.000000000","message":"Thank you Sean, I updated the same.","commit_id":"750377d879dc765cd26106fd89dab900e3a6a1bc"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b46b7471fe559271cf8077344a00f063572057f0","unresolved":true,"context_lines":[{"line_number":19,"context_line":"removals.removed_module("},{"line_number":20,"context_line":"    __name__,"},{"line_number":21,"context_line":"    replacement\u003d\"watcherclient.tests.client_functional\","},{"line_number":22,"context_line":"    removal_version\u003d\"2024.2-eol\","},{"line_number":23,"context_line":"    message\u003d("},{"line_number":24,"context_line":"        \"The \u0027watcher_tempest_plugin.tests.client_functional\u0027 module is \""},{"line_number":25,"context_line":"        \"deprecated and will be removed in version 2024.2-eol. \""}],"source_content_type":"text/x-python","patch_set":6,"id":"9ba7677c_9b0aec6c","line":22,"range":{"start_line":22,"start_character":21,"end_line":22,"end_character":31},"updated":"2025-03-26 13:20:48.000000000","message":"this is not really correct. it would be 2026.1","commit_id":"ddca84e0257c62586306034d2b70f7cd3086b8ca"}]}