)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"5c7977bd6aa9878b9c52532848fe34ae2e96e51a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b4bdedd8_97db87b9","updated":"2023-08-02 07:45:43.000000000","message":"Looks good to me!","commit_id":"f5eb742616bdb3e74aedc9e2f4b993028699793d"}],"zuul.d/jobs.yaml":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"d3369ff16c908c534bdf26c6959d94dc49cdf597","unresolved":true,"context_lines":[{"line_number":345,"context_line":"        ovn-northd: false"},{"line_number":346,"context_line":"      tempest_test_regex: ^trove_tempest_plugin\\.tests"},{"line_number":347,"context_line":"      tempest_test_timeout: 3600"},{"line_number":348,"context_line":"      tempest_test_exclude_list: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/trove-tempest-plugin\"].src_dir }}/tempest_exclude_list.txt\u0027"},{"line_number":349,"context_line":"      zuul_copy_output:"},{"line_number":350,"context_line":"        \u0027/var/log/guest-agent-logs/\u0027: \u0027logs\u0027"},{"line_number":351,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"2b0f1015_7fa283a4","line":348,"range":{"start_line":348,"start_character":131,"end_line":348,"end_character":151},"updated":"2023-07-27 21:39:51.000000000","message":"note: the content of the file is the same","commit_id":"7f574c7fe46229ebbdeedcfb68a05aa2e443962a"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"d3369ff16c908c534bdf26c6959d94dc49cdf597","unresolved":true,"context_lines":[{"line_number":345,"context_line":"        ovn-northd: false"},{"line_number":346,"context_line":"      tempest_test_regex: ^trove_tempest_plugin\\.tests"},{"line_number":347,"context_line":"      tempest_test_timeout: 3600"},{"line_number":348,"context_line":"      tempest_test_exclude_list: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/trove-tempest-plugin\"].src_dir }}/tempest_exclude_list.txt\u0027"},{"line_number":349,"context_line":"      zuul_copy_output:"},{"line_number":350,"context_line":"        \u0027/var/log/guest-agent-logs/\u0027: \u0027logs\u0027"},{"line_number":351,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ed2bef41_36e55589","line":348,"range":{"start_line":348,"start_character":6,"end_line":348,"end_character":31},"updated":"2023-07-27 21:39:51.000000000","message":"we can use this variable as long as the job runs tempest\u003e\u003d27.0.0, which is the case for all stable releases older than ussuri","commit_id":"7f574c7fe46229ebbdeedcfb68a05aa2e443962a"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"a4a16120c3b58e1a7e294f375030af0818e43e00","unresolved":true,"context_lines":[{"line_number":345,"context_line":"        ovn-northd: false"},{"line_number":346,"context_line":"      tempest_test_regex: ^trove_tempest_plugin\\.tests"},{"line_number":347,"context_line":"      tempest_test_timeout: 3600"},{"line_number":348,"context_line":"      tempest_test_exclude_list: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/trove-tempest-plugin\"].src_dir }}/tempest_exclude_list.txt\u0027"},{"line_number":349,"context_line":"      zuul_copy_output:"},{"line_number":350,"context_line":"        \u0027/var/log/guest-agent-logs/\u0027: \u0027logs\u0027"},{"line_number":351,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"6316670e_0aebcc11","line":348,"range":{"start_line":348,"start_character":131,"end_line":348,"end_character":151},"updated":"2023-07-28 01:45:15.000000000","message":"we don\u0027t have this file in the trove-tempest-plugin repository, so keep the tempest_blacklist like before.","commit_id":"7f574c7fe46229ebbdeedcfb68a05aa2e443962a"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"a4a16120c3b58e1a7e294f375030af0818e43e00","unresolved":true,"context_lines":[{"line_number":345,"context_line":"        ovn-northd: false"},{"line_number":346,"context_line":"      tempest_test_regex: ^trove_tempest_plugin\\.tests"},{"line_number":347,"context_line":"      tempest_test_timeout: 3600"},{"line_number":348,"context_line":"      tempest_test_exclude_list: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/trove-tempest-plugin\"].src_dir }}/tempest_exclude_list.txt\u0027"},{"line_number":349,"context_line":"      zuul_copy_output:"},{"line_number":350,"context_line":"        \u0027/var/log/guest-agent-logs/\u0027: \u0027logs\u0027"},{"line_number":351,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"e62a6ee9_c0c897a1","line":348,"range":{"start_line":348,"start_character":131,"end_line":348,"end_character":151},"in_reply_to":"2b0f1015_7fa283a4","updated":"2023-07-28 01:45:15.000000000","message":"yes, they are same.","commit_id":"7f574c7fe46229ebbdeedcfb68a05aa2e443962a"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"e3d69d226cb312ec797d635bff50e6483cd8bfea","unresolved":true,"context_lines":[{"line_number":345,"context_line":"        ovn-northd: false"},{"line_number":346,"context_line":"      tempest_test_regex: ^trove_tempest_plugin\\.tests"},{"line_number":347,"context_line":"      tempest_test_timeout: 3600"},{"line_number":348,"context_line":"      tempest_test_exclude_list: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/trove-tempest-plugin\"].src_dir }}/tempest_exclude_list.txt\u0027"},{"line_number":349,"context_line":"      zuul_copy_output:"},{"line_number":350,"context_line":"        \u0027/var/log/guest-agent-logs/\u0027: \u0027logs\u0027"},{"line_number":351,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"56c65dda_ffc22edf","line":348,"range":{"start_line":348,"start_character":131,"end_line":348,"end_character":151},"in_reply_to":"3d315d0c_8decffac","updated":"2023-08-01 01:07:06.000000000","message":"yes, this file only used by trove zuul jobs.","commit_id":"7f574c7fe46229ebbdeedcfb68a05aa2e443962a"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"bed57601bc22e057228dc69c9460a6ffbe765ae9","unresolved":true,"context_lines":[{"line_number":345,"context_line":"        ovn-northd: false"},{"line_number":346,"context_line":"      tempest_test_regex: ^trove_tempest_plugin\\.tests"},{"line_number":347,"context_line":"      tempest_test_timeout: 3600"},{"line_number":348,"context_line":"      tempest_test_exclude_list: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/trove-tempest-plugin\"].src_dir }}/tempest_exclude_list.txt\u0027"},{"line_number":349,"context_line":"      zuul_copy_output:"},{"line_number":350,"context_line":"        \u0027/var/log/guest-agent-logs/\u0027: \u0027logs\u0027"},{"line_number":351,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b69b5147_7c17b4a7","line":348,"range":{"start_line":348,"start_character":131,"end_line":348,"end_character":151},"in_reply_to":"51fe1afd_df352483","updated":"2023-07-28 14:05:05.000000000","message":"that file you provided link for is a symlink and it points to this file - https://opendev.org/openstack/tempest/src/branch/master/tools/tempest-integrated-gate-storage-exclude-list.txt\n\nAnyway, the main goal here is to get rid of any deprecated terms and use the new alternatives instead.\n\nE.g.:\ntempest_test_blacklist -\u003e tempest_test_exclude_list\ntempest-integrated-gate-storage-blacklist.txt -\u003e tempest-integrated-gate-storage-exclude-list.txt\ntempest_blacklist.txt (in trove projects) -\u003e ? (seems like trove doesn\u0027t have the alternative yet, that\u0027s what we\u0027re discussing now)","commit_id":"7f574c7fe46229ebbdeedcfb68a05aa2e443962a"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"1aaf08842e632f5df0912773bb29a86c70d59636","unresolved":true,"context_lines":[{"line_number":345,"context_line":"        ovn-northd: false"},{"line_number":346,"context_line":"      tempest_test_regex: ^trove_tempest_plugin\\.tests"},{"line_number":347,"context_line":"      tempest_test_timeout: 3600"},{"line_number":348,"context_line":"      tempest_test_exclude_list: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/trove-tempest-plugin\"].src_dir }}/tempest_exclude_list.txt\u0027"},{"line_number":349,"context_line":"      zuul_copy_output:"},{"line_number":350,"context_line":"        \u0027/var/log/guest-agent-logs/\u0027: \u0027logs\u0027"},{"line_number":351,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"d1faf9c6_4d9bb49a","line":348,"range":{"start_line":348,"start_character":131,"end_line":348,"end_character":151},"in_reply_to":"6316670e_0aebcc11","updated":"2023-07-28 07:58:32.000000000","message":"right, I see, our primary goal was to replace usage of this file - https://opendev.org/openstack/tempest/src/branch/master/tools/tempest-integrated-gate-storage-blacklist.txt - all over upstream, I didn\u0027t notice, this one is different and not consumed from tempest.\nThat being said we can still do one of the 2 things and continue with this patch:\n1. rename the file in trove-tempest-plugin\n2. create a symlink of the file in trove-tempest-plugin so that we can rename the file here and get some time for other consumers (if there are any) to switch (as 2 files with the same content would coexist for now)","commit_id":"7f574c7fe46229ebbdeedcfb68a05aa2e443962a"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"1b72824f1d5ca362256ff2f4e9edb0e576b3f09f","unresolved":true,"context_lines":[{"line_number":345,"context_line":"        ovn-northd: false"},{"line_number":346,"context_line":"      tempest_test_regex: ^trove_tempest_plugin\\.tests"},{"line_number":347,"context_line":"      tempest_test_timeout: 3600"},{"line_number":348,"context_line":"      tempest_test_exclude_list: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/trove-tempest-plugin\"].src_dir }}/tempest_exclude_list.txt\u0027"},{"line_number":349,"context_line":"      zuul_copy_output:"},{"line_number":350,"context_line":"        \u0027/var/log/guest-agent-logs/\u0027: \u0027logs\u0027"},{"line_number":351,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3d315d0c_8decffac","line":348,"range":{"start_line":348,"start_character":131,"end_line":348,"end_character":151},"in_reply_to":"915b2e8b_897305ec","updated":"2023-07-31 08:25:25.000000000","message":"I think the renaming of the file should be enough here, if I\u0027m not missing something. The tempest_blacklist.txt should be consumed only by these jobs [1].\n\n[1] https://codesearch.opendev.org/?q\u003dzuul.projects%5C%5B%22opendev.org%2Fopenstack%2Ftrove-tempest-plugin%22%5C%5D.src_dir\u0026i\u003dnope\u0026literal\u003dnope\u0026files\u003d\u0026excludeFiles\u003d\u0026repos\u003d","commit_id":"7f574c7fe46229ebbdeedcfb68a05aa2e443962a"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"51d2a1a462116d266f1356766e8780e403dd5d0e","unresolved":true,"context_lines":[{"line_number":345,"context_line":"        ovn-northd: false"},{"line_number":346,"context_line":"      tempest_test_regex: ^trove_tempest_plugin\\.tests"},{"line_number":347,"context_line":"      tempest_test_timeout: 3600"},{"line_number":348,"context_line":"      tempest_test_exclude_list: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/trove-tempest-plugin\"].src_dir }}/tempest_exclude_list.txt\u0027"},{"line_number":349,"context_line":"      zuul_copy_output:"},{"line_number":350,"context_line":"        \u0027/var/log/guest-agent-logs/\u0027: \u0027logs\u0027"},{"line_number":351,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"915b2e8b_897305ec","line":348,"range":{"start_line":348,"start_character":131,"end_line":348,"end_character":151},"in_reply_to":"b69b5147_7c17b4a7","updated":"2023-07-31 07:07:15.000000000","message":"ok, got it. but i still think we don\u0027n need to create a symbolic link for the tempest_blacklist.txt. as this is just a file name, Not \"deprecated terms\", right？ thanks","commit_id":"7f574c7fe46229ebbdeedcfb68a05aa2e443962a"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"01cf0eaa1640dd10f8744cbf1e430f0681cc4b66","unresolved":true,"context_lines":[{"line_number":345,"context_line":"        ovn-northd: false"},{"line_number":346,"context_line":"      tempest_test_regex: ^trove_tempest_plugin\\.tests"},{"line_number":347,"context_line":"      tempest_test_timeout: 3600"},{"line_number":348,"context_line":"      tempest_test_exclude_list: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/trove-tempest-plugin\"].src_dir }}/tempest_exclude_list.txt\u0027"},{"line_number":349,"context_line":"      zuul_copy_output:"},{"line_number":350,"context_line":"        \u0027/var/log/guest-agent-logs/\u0027: \u0027logs\u0027"},{"line_number":351,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"51fe1afd_df352483","line":348,"range":{"start_line":348,"start_character":131,"end_line":348,"end_character":151},"in_reply_to":"d1faf9c6_4d9bb49a","updated":"2023-07-28 08:04:46.000000000","message":"I think, we just need to replace the tempest_test_black_list to tempest_test_exclude_list, this is enough? why we need change the tempest_exclude_list.txt file name? I don\u0027t understand what\u0027s meaning of this https://opendev.org/openstack/tempest/src/branch/master/tools/tempest-integrated-gate-storage-blacklist.txt","commit_id":"7f574c7fe46229ebbdeedcfb68a05aa2e443962a"}]}