)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"3aabd5891c0def8f6089d7a449be0d09169a147d","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Katarina Strenkova \u003ckstrenko@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-08-01 14:37:31 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Replace\tdeprecated terms"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Following up on\tthe usage of inclusive jargon [1] and to ensure"},{"line_number":10,"context_line":"consistency in code, there is a need to replace deprecated term"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f66f6dc_f92d3e8c","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":5},"updated":"2023-08-14 09:05:03.000000000","message":"is there a tab between Replace and deprecated? o.O","commit_id":"4ba37cb674e91a4479cd6f0629a20fd52e8a63fe"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"13c447d114624d89647db39264397705de30e085","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Katarina Strenkova \u003ckstrenko@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-08-01 14:37:31 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Replace\tdeprecated terms"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Following up on\tthe usage of inclusive jargon [1] and to ensure"},{"line_number":10,"context_line":"consistency in code, there is a need to replace deprecated term"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"e65d404f_f2e00f81","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":5},"in_reply_to":"7f66f6dc_f92d3e8c","updated":"2023-08-14 10:29:23.000000000","message":"o.O 😄","commit_id":"4ba37cb674e91a4479cd6f0629a20fd52e8a63fe"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"a3bd925b7f8d7e4948cbad1ae274030ab8e9db61","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ba6c414f_e9ef8020","updated":"2023-08-03 06:46:46.000000000","message":"We probably do not need to change the file here. My assumption is that run-tempest role is not used here. Let\u0027s wait for someone else\u0027s opinion on this.","commit_id":"4ba37cb674e91a4479cd6f0629a20fd52e8a63fe"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"3b65d93146e2522c8766a30bc6ce36f2564279f4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"18c35c59_afa7343d","updated":"2023-08-02 08:17:17.000000000","message":"recheck\nFailure not caused by the patch \u003d\u003e openstack-ansible-deploy-aio_metal-ubuntu-jammy","commit_id":"4ba37cb674e91a4479cd6f0629a20fd52e8a63fe"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"5ed41735be0b746730994031f72d950ce9c53a9f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4eb26113_00c92f8e","in_reply_to":"ba6c414f_e9ef8020","updated":"2023-08-12 08:07:08.000000000","message":"run-tempest is not used, everything suggests they use os_tempest","commit_id":"4ba37cb674e91a4479cd6f0629a20fd52e8a63fe"},{"author":{"_account_id":36227,"name":"Katarina Strenkova","email":"kstrenko@redhat.com","username":"kstrenko"},"change_message_id":"1e70a2ef5945f97cca69c08b9b1e56de93d9bae0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d1a7ca12_99297759","updated":"2023-08-15 09:17:38.000000000","message":"I have added the suggested change from whitelist to include_list.","commit_id":"e241be1d0e4ca4f0aaace6a3d817d651fc2fcc56"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"699832331a2a59f84fe59d72955bb48664eb4042","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b509684f_ef68e000","updated":"2023-08-15 10:05:28.000000000","message":"I wanna thank for the contribution, but... Having this file at all is kinda a technical debt. We don\u0027t run any functional tests in CI for quite a while as using solely integrated tests. We\u0027re short on capacity to finally drop tests folder at all...\n\nOfc we can land that, but it is not used in any way or form as of today.","commit_id":"e241be1d0e4ca4f0aaace6a3d817d651fc2fcc56"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"94313cbe782ede3341eec44bbabc8b242d16374c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b72ce1eb_32613d49","updated":"2023-08-16 09:27:30.000000000","message":"one detail, otherwise lgtm","commit_id":"e241be1d0e4ca4f0aaace6a3d817d651fc2fcc56"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"c582017b817086d00e1d04b70689133a0e8f6204","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"77fe7dd7_8364062e","in_reply_to":"b509684f_ef68e000","updated":"2023-08-15 10:06:03.000000000","message":"(as well as any similar matches in other OSA repositories)","commit_id":"e241be1d0e4ca4f0aaace6a3d817d651fc2fcc56"}],"tests/os_manila-overrides.yml":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"5ed41735be0b746730994031f72d950ce9c53a9f","unresolved":true,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"tempest_run: yes"},{"line_number":17,"context_line":"tempest_test_whitelist:"},{"line_number":18,"context_line":"  - smoke"},{"line_number":19,"context_line":"  - manila_tempest_tests.tests.scenario.test_share_basic_ops"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1b1075cd_01a497df","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":22},"updated":"2023-08-12 08:07:08.000000000","message":"let\u0027s change also this","commit_id":"4ba37cb674e91a4479cd6f0629a20fd52e8a63fe"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"ea8ca39ca982b6b8bb2f02284ca530718faf477a","unresolved":true,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"tempest_run: yes"},{"line_number":17,"context_line":"tempest_test_whitelist:"},{"line_number":18,"context_line":"  - smoke"},{"line_number":19,"context_line":"  - manila_tempest_tests.tests.scenario.test_share_basic_ops"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"001cbb8d_179e1a4e","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":22},"in_reply_to":"0d304d28_8b40c13e","updated":"2023-08-17 08:14:03.000000000","message":"I\u0027ve proposed https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/891664 as alternative.","commit_id":"4ba37cb674e91a4479cd6f0629a20fd52e8a63fe"},{"author":{"_account_id":36227,"name":"Katarina Strenkova","email":"kstrenko@redhat.com","username":"kstrenko"},"change_message_id":"c23f06e91a0e0cdcceb1e7c8d907e0ca0536c1fe","unresolved":true,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"tempest_run: yes"},{"line_number":17,"context_line":"tempest_test_whitelist:"},{"line_number":18,"context_line":"  - smoke"},{"line_number":19,"context_line":"  - manila_tempest_tests.tests.scenario.test_share_basic_ops"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"732eb1d5_dc0fa652","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":22},"in_reply_to":"0d304d28_8b40c13e","updated":"2023-08-17 08:20:43.000000000","message":"Okay, I get where you are coming from. On that note, should I abandon this patch or keep it open?","commit_id":"4ba37cb674e91a4479cd6f0629a20fd52e8a63fe"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"e238d4d1fc6d9142973d844164f64e7b1ba00305","unresolved":true,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"tempest_run: yes"},{"line_number":17,"context_line":"tempest_test_whitelist:"},{"line_number":18,"context_line":"  - smoke"},{"line_number":19,"context_line":"  - manila_tempest_tests.tests.scenario.test_share_basic_ops"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7704e1c6_8d5e9180","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":22},"in_reply_to":"1b1075cd_01a497df","updated":"2023-08-14 07:59:55.000000000","message":"Why should this be changed when run-tempest is not used here?","commit_id":"4ba37cb674e91a4479cd6f0629a20fd52e8a63fe"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"13c447d114624d89647db39264397705de30e085","unresolved":true,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"tempest_run: yes"},{"line_number":17,"context_line":"tempest_test_whitelist:"},{"line_number":18,"context_line":"  - smoke"},{"line_number":19,"context_line":"  - manila_tempest_tests.tests.scenario.test_share_basic_ops"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"6580f3cc_2c797ae9","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":22},"in_reply_to":"1cc32371_b0add392","updated":"2023-08-14 10:29:23.000000000","message":"Yes, this is a good point. I got all stuck up in the run-tempest cleanup that I forgot that we can do the clean up in other repos as well.\n\nhttps://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/784885","commit_id":"4ba37cb674e91a4479cd6f0629a20fd52e8a63fe"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"4560c7e7cf7e139e2b307e92777d181bde8d8132","unresolved":true,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"tempest_run: yes"},{"line_number":17,"context_line":"tempest_test_whitelist:"},{"line_number":18,"context_line":"  - smoke"},{"line_number":19,"context_line":"  - manila_tempest_tests.tests.scenario.test_share_basic_ops"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"0d304d28_8b40c13e","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":22},"in_reply_to":"6580f3cc_2c797ae9","updated":"2023-08-17 08:00:21.000000000","message":"there is actually another patch to os_tempest that fully drop old var names:\nhttps://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/891567\n\nBut again, I\u0027m inlcined to do clean-up and just drop this file instead of fixing it.","commit_id":"4ba37cb674e91a4479cd6f0629a20fd52e8a63fe"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"3aabd5891c0def8f6089d7a449be0d09169a147d","unresolved":true,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"tempest_run: yes"},{"line_number":17,"context_line":"tempest_test_whitelist:"},{"line_number":18,"context_line":"  - smoke"},{"line_number":19,"context_line":"  - manila_tempest_tests.tests.scenario.test_share_basic_ops"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1cc32371_b0add392","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":22},"in_reply_to":"7704e1c6_8d5e9180","updated":"2023-08-14 09:05:03.000000000","message":"I don\u0027t understand why it should matter if run-tempest is used or not. The commit message states: \"Replace deprecated terms\"","commit_id":"4ba37cb674e91a4479cd6f0629a20fd52e8a63fe"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"94313cbe782ede3341eec44bbabc8b242d16374c","unresolved":true,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"tempest_run: yes"},{"line_number":17,"context_line":"tempest_test_include_list:"},{"line_number":18,"context_line":"  - smoke"},{"line_number":19,"context_line":"  - manila_tempest_tests.tests.scenario.test_share_basic_ops"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"539b04ba_0bc6f896","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":25},"updated":"2023-08-16 09:27:30.000000000","message":"tempest_test_includelist \nhttps://opendev.org/openstack/openstack-ansible-os_tempest/src/commit/687dcb30bf0a8ed60b41dcb1f9e45c16aee74af7/defaults/main.yml#L175","commit_id":"e241be1d0e4ca4f0aaace6a3d817d651fc2fcc56"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"94313cbe782ede3341eec44bbabc8b242d16374c","unresolved":true,"context_lines":[{"line_number":18,"context_line":"  - smoke"},{"line_number":19,"context_line":"  - manila_tempest_tests.tests.scenario.test_share_basic_ops"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"tempest_test_exclude_list:"},{"line_number":22,"context_line":"  - test: tempest.api.identity"},{"line_number":23,"context_line":"    reason: Reducing run time and required resources"},{"line_number":24,"context_line":"  - test: tempest.api.image"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"45d34722_3d84adab","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":25},"updated":"2023-08-16 09:27:30.000000000","message":"tempest_test_excludelist\nhttps://opendev.org/openstack/openstack-ansible-os_tempest/src/commit/687dcb30bf0a8ed60b41dcb1f9e45c16aee74af7/defaults/main.yml#L199","commit_id":"e241be1d0e4ca4f0aaace6a3d817d651fc2fcc56"}]}