)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"9b20e0a3e9e3e03f3c0f0462de7e90e37a87b4c2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"48acf45a_777def87","updated":"2024-09-04 14:22:59.000000000","message":"-1 for the question related to https://review.opendev.org/c/openstack/tempest/+/927831/3/tox.ini","commit_id":"819042aae80c931d18dbc88bcef407bbbb2ca797"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"d5105d0408ef3367e206570b769df2261fae4f13","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ddbcc323_b4741606","updated":"2024-09-03 14:16:42.000000000","message":"waiting for the https://review.opendev.org/c/openstack/tempest/+/927831/ to get merged","commit_id":"819042aae80c931d18dbc88bcef407bbbb2ca797"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"7a1239b9bc0a0a416729d9c68f7638c821ef755d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"471dbd35_837ccc22","updated":"2024-09-06 09:58:39.000000000","message":"Giving -1 to highlight my comment.","commit_id":"6894fd2e66fa6ce7eb966f5354e41d311d820101"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"e58345b80e2579d7d4e2c8c659b08eaac5221a18","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"50926902_699f1dd7","updated":"2024-09-05 00:15:19.000000000","message":"lgtm","commit_id":"6894fd2e66fa6ce7eb966f5354e41d311d820101"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"10b7bdcdded2612709db0aa62d782e18aa508b2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b7c60264_4e837412","updated":"2024-09-06 18:44:29.000000000","message":"Thanks, LGTM.","commit_id":"5033adfb52e13b2ba8d3008ce28aa2aef691f054"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9f46fe6eaec44e16bf65d96359797fac969a7d45","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d65d5173_98558687","updated":"2024-09-06 18:07:17.000000000","message":"thanks","commit_id":"5033adfb52e13b2ba8d3008ce28aa2aef691f054"}],"deliverables/dalmatian/tempest.yaml":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"d5105d0408ef3367e206570b769df2261fae4f13","unresolved":true,"context_lines":[{"line_number":15,"context_line":"    version: 39.0.0"},{"line_number":16,"context_line":"  - projects:"},{"line_number":17,"context_line":"      - repo: openstack/tempest"},{"line_number":18,"context_line":"        hash: b711313a793c0b9726261b3c0d01405c4729207b"},{"line_number":19,"context_line":"    version: 39.1.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b9c4e8de_6cc21387","line":18,"range":{"start_line":18,"start_character":14,"end_line":18,"end_character":54},"updated":"2024-09-03 14:16:42.000000000","message":"https://review.opendev.org/c/openstack/tempest/+/927831/","commit_id":"819042aae80c931d18dbc88bcef407bbbb2ca797"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6a657f18dde7eb282930a11fd6bfae4e8838ec33","unresolved":false,"context_lines":[{"line_number":15,"context_line":"    version: 39.0.0"},{"line_number":16,"context_line":"  - projects:"},{"line_number":17,"context_line":"      - repo: openstack/tempest"},{"line_number":18,"context_line":"        hash: b711313a793c0b9726261b3c0d01405c4729207b"},{"line_number":19,"context_line":"    version: 39.1.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7e1adc75_efeb6234","line":18,"range":{"start_line":18,"start_character":14,"end_line":18,"end_character":54},"in_reply_to":"66808ce8_195916ca","updated":"2024-09-06 06:18:54.000000000","message":"Yes, I saw the patch was reverted to master. That patch was just to have a hash pointing to 2024.1.\n\nWB testing failed with the mentioned patch. For example: https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/927391 had some executions failed due to the old requirements.\n\nThanks for your explanations!","commit_id":"819042aae80c931d18dbc88bcef407bbbb2ca797"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"803016ea467521716d8cd5426a18dac8145c7213","unresolved":true,"context_lines":[{"line_number":15,"context_line":"    version: 39.0.0"},{"line_number":16,"context_line":"  - projects:"},{"line_number":17,"context_line":"      - repo: openstack/tempest"},{"line_number":18,"context_line":"        hash: b711313a793c0b9726261b3c0d01405c4729207b"},{"line_number":19,"context_line":"    version: 39.1.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"f49b6a4f_219dfc7a","line":18,"range":{"start_line":18,"start_character":14,"end_line":18,"end_character":54},"in_reply_to":"7e8d578f_5cd18ccf","updated":"2024-09-04 14:48:56.000000000","message":"yeah, i had it with 2024.2 but then gmann pointed out that 2024.1 is better as 2024.2 is not released yet .. so i changed that, see his comment: https://review.opendev.org/c/openstack/tempest/+/927831/1..3/tox.ini#b12","commit_id":"819042aae80c931d18dbc88bcef407bbbb2ca797"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"323c89c58153ff77e902685207b6b5047da7111a","unresolved":true,"context_lines":[{"line_number":15,"context_line":"    version: 39.0.0"},{"line_number":16,"context_line":"  - projects:"},{"line_number":17,"context_line":"      - repo: openstack/tempest"},{"line_number":18,"context_line":"        hash: b711313a793c0b9726261b3c0d01405c4729207b"},{"line_number":19,"context_line":"    version: 39.1.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"fe8fd41f_9a65a4bf","line":18,"range":{"start_line":18,"start_character":14,"end_line":18,"end_character":54},"in_reply_to":"98fd4859_acd27227","updated":"2024-09-05 19:08:46.000000000","message":"\u003e The commit https://review.opendev.org/c/openstack/tempest/+/927831/1..3/tox.ini#b12 broke the neutron-tempest-plugin and whitebox-neutron-tempest-plugin\n\nCan you please send me the link to the failure? We know the failure during constraints pin and that should have fixed by this https://review.opendev.org/c/openstack/tempest/+/927832/3","commit_id":"819042aae80c931d18dbc88bcef407bbbb2ca797"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"dfb22505ae584658ea75150fe23c3b6d57902f25","unresolved":true,"context_lines":[{"line_number":15,"context_line":"    version: 39.0.0"},{"line_number":16,"context_line":"  - projects:"},{"line_number":17,"context_line":"      - repo: openstack/tempest"},{"line_number":18,"context_line":"        hash: b711313a793c0b9726261b3c0d01405c4729207b"},{"line_number":19,"context_line":"    version: 39.1.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"66808ce8_195916ca","line":18,"range":{"start_line":18,"start_character":14,"end_line":18,"end_character":54},"in_reply_to":"98fd4859_acd27227","updated":"2024-09-05 19:09:30.000000000","message":"Rodolfo, please note the issue is https://review.opendev.org/c/openstack/tempest/+/927831,. which will be reverted. Blocking the release isn\u0027t going to improve the situation, just making it worse.","commit_id":"819042aae80c931d18dbc88bcef407bbbb2ca797"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"939d3a1f557d2f748ef0db1497562c90571df745","unresolved":true,"context_lines":[{"line_number":15,"context_line":"    version: 39.0.0"},{"line_number":16,"context_line":"  - projects:"},{"line_number":17,"context_line":"      - repo: openstack/tempest"},{"line_number":18,"context_line":"        hash: b711313a793c0b9726261b3c0d01405c4729207b"},{"line_number":19,"context_line":"    version: 39.1.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b9074a86_e6983446","line":18,"range":{"start_line":18,"start_character":14,"end_line":18,"end_character":54},"in_reply_to":"aad6b736_fc4f42bf","updated":"2024-09-05 07:48:07.000000000","message":"I\u0027ve resumed that in https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/OQ4AS74MLMRUEXCLJWY27G3YYXPZOB7N/","commit_id":"819042aae80c931d18dbc88bcef407bbbb2ca797"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"1383e5d7549eba3c1eaa7b04161dac10d8c38c23","unresolved":true,"context_lines":[{"line_number":15,"context_line":"    version: 39.0.0"},{"line_number":16,"context_line":"  - projects:"},{"line_number":17,"context_line":"      - repo: openstack/tempest"},{"line_number":18,"context_line":"        hash: b711313a793c0b9726261b3c0d01405c4729207b"},{"line_number":19,"context_line":"    version: 39.1.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"98fd4859_acd27227","line":18,"range":{"start_line":18,"start_character":14,"end_line":18,"end_character":54},"in_reply_to":"b9074a86_e6983446","updated":"2024-09-05 19:06:07.000000000","message":"I replied in that email. I think you are confused with intermediate release vs final release of Tempest. This is intermediate release of Tempest and that is why we need to use stable/2024.1 constraints instead of master. Past intermediate ref:\n\nTempest 34.2.0 intermediate release in bobcat (2023.2), constrained used is stable/2023.1:\n- https://review.opendev.org/q/topic:%22tempest-34-2-0-release%22\n\nTempest 37.0.0 intermediate release in Caracal (2024.1), constrained used is stable/2023.2:\n- https://review.opendev.org/c/openstack/releases/+/908346\n- https://review.opendev.org/c/openstack/tempest/+/908345","commit_id":"819042aae80c931d18dbc88bcef407bbbb2ca797"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"9b20e0a3e9e3e03f3c0f0462de7e90e37a87b4c2","unresolved":true,"context_lines":[{"line_number":15,"context_line":"    version: 39.0.0"},{"line_number":16,"context_line":"  - projects:"},{"line_number":17,"context_line":"      - repo: openstack/tempest"},{"line_number":18,"context_line":"        hash: b711313a793c0b9726261b3c0d01405c4729207b"},{"line_number":19,"context_line":"    version: 39.1.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7e8d578f_5cd18ccf","line":18,"range":{"start_line":18,"start_character":14,"end_line":18,"end_character":54},"in_reply_to":"b9c4e8de_6cc21387","updated":"2024-09-04 14:22:59.000000000","message":"qq: should [1] point to 2024.1 or to 2024.2?\n\n[1]https://review.opendev.org/c/openstack/tempest/+/927831/3/tox.ini","commit_id":"819042aae80c931d18dbc88bcef407bbbb2ca797"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"31eec2bcd56d6802fafe353b90901504eb34934d","unresolved":true,"context_lines":[{"line_number":15,"context_line":"    version: 39.0.0"},{"line_number":16,"context_line":"  - projects:"},{"line_number":17,"context_line":"      - repo: openstack/tempest"},{"line_number":18,"context_line":"        hash: b711313a793c0b9726261b3c0d01405c4729207b"},{"line_number":19,"context_line":"    version: 39.1.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"aad6b736_fc4f42bf","line":18,"range":{"start_line":18,"start_character":14,"end_line":18,"end_character":54},"in_reply_to":"bb23a688_8b00d723","updated":"2024-09-05 06:41:45.000000000","message":"As I commented in the tempest patch, \"https://releases.openstack.org/constraints/upper/2024.2\" is a valid link despite 2024.2 doesn\u0027t exist. This link points to master and once the 2024.2 branch is created, the link will point to the stable branch.\n\nThe commit https://review.opendev.org/c/openstack/tempest/+/927831/1..3/tox.ini#b12 broke the neutron-tempest-plugin and whitebox-neutron-tempest-plugin","commit_id":"819042aae80c931d18dbc88bcef407bbbb2ca797"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"20645899d937dfa49701acdc729e40e9948a7145","unresolved":true,"context_lines":[{"line_number":15,"context_line":"    version: 39.0.0"},{"line_number":16,"context_line":"  - projects:"},{"line_number":17,"context_line":"      - repo: openstack/tempest"},{"line_number":18,"context_line":"        hash: b711313a793c0b9726261b3c0d01405c4729207b"},{"line_number":19,"context_line":"    version: 39.1.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bb23a688_8b00d723","line":18,"range":{"start_line":18,"start_character":14,"end_line":18,"end_character":54},"in_reply_to":"ccc1f856_cf0c00c5","updated":"2024-09-04 17:29:10.000000000","message":"We need to use 2024.1 only as that is the latest freeze constraint, and we tested one with this tempest release. 2024.2 constraints are still not frozen right, so we should not use them for the Tempest-released version. Once we cut the final release of Tempest for 2024.2, then we will use the stable/2024.2 constraints.","commit_id":"819042aae80c931d18dbc88bcef407bbbb2ca797"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"afe11502fbd1dabcfe28619015db7642e5e825fb","unresolved":true,"context_lines":[{"line_number":15,"context_line":"    version: 39.0.0"},{"line_number":16,"context_line":"  - projects:"},{"line_number":17,"context_line":"      - repo: openstack/tempest"},{"line_number":18,"context_line":"        hash: b711313a793c0b9726261b3c0d01405c4729207b"},{"line_number":19,"context_line":"    version: 39.1.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ccc1f856_cf0c00c5","line":18,"range":{"start_line":18,"start_character":14,"end_line":18,"end_character":54},"in_reply_to":"f49b6a4f_219dfc7a","updated":"2024-09-04 15:08:39.000000000","message":"soo , now what? do we need to do that dance in tempest again (now with 2024.2) or we may proceed with this?\nmy goal is to cut this tag so that this version of tempest can be packaged and consumed as an rpm .. this little imperfection in tox.ini doesn\u0027t cause a problem for packaging (well, as far as i know)","commit_id":"819042aae80c931d18dbc88bcef407bbbb2ca797"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"7a1239b9bc0a0a416729d9c68f7638c821ef755d","unresolved":true,"context_lines":[{"line_number":16,"context_line":"  - projects:"},{"line_number":17,"context_line":"      - repo: openstack/tempest"},{"line_number":18,"context_line":"        hash: 571959dd2b1469ada0657ae701e4699044e3d79f"},{"line_number":19,"context_line":"    version: 39.1.0"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3ce5501b_4724b504","line":19,"updated":"2024-09-06 09:58:39.000000000","message":"reno suggests MAJOR version bump as there is \u0027upgrade notes\u0027. one of them shows backward incompatibility [1], so 40.0.0 would be probably better.\n\n[1] https://zuul.opendev.org/t/openstack/build/d47f3bf5f69b4384be10af2af35e8bb2/log/tox/list-changes/list-changes-results.log#1039-1040","commit_id":"6894fd2e66fa6ce7eb966f5354e41d311d820101"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"e81c6e70b02d54ce514bf7057f49de2907074360","unresolved":true,"context_lines":[{"line_number":16,"context_line":"  - projects:"},{"line_number":17,"context_line":"      - repo: openstack/tempest"},{"line_number":18,"context_line":"        hash: 571959dd2b1469ada0657ae701e4699044e3d79f"},{"line_number":19,"context_line":"    version: 39.1.0"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"b970f12c_dfa40c23","line":19,"in_reply_to":"26bea18d_67ffcf9f","updated":"2024-09-06 17:02:31.000000000","message":"I am ok either, we can bump major version also 40.0.0 and final release can be minor version bump 40.1.0. We have done same in past release also.","commit_id":"6894fd2e66fa6ce7eb966f5354e41d311d820101"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"662cb5f77cc6ae37139222010d19d12693dc451a","unresolved":true,"context_lines":[{"line_number":16,"context_line":"  - projects:"},{"line_number":17,"context_line":"      - repo: openstack/tempest"},{"line_number":18,"context_line":"        hash: 571959dd2b1469ada0657ae701e4699044e3d79f"},{"line_number":19,"context_line":"    version: 39.1.0"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"26bea18d_67ffcf9f","line":19,"in_reply_to":"3ce5501b_4724b504","updated":"2024-09-06 13:01:46.000000000","message":"\"The attr decorator was deprecated during Pike cycle[1] and multiple cycles have passed since then.\" - https://opendev.org/openstack/tempest/commit/033f78a4bebdd93c35182b90606cf406c97c89b4\n\nIt\u0027s not a big deal, that thing was deprecated for years.\n\nFirst, I wanted to make a release now so that lots of things that changed can be consumed further (rpms etc). As we still have a couple weeks until the moment when we need to cut the major release, I figured it would be better to make a minor release now - that way we still have time to merge some other things (there is always something close to the end of cycle) that will have a chance to be a part of the major release.\n\nSecond, the preparation we did in tempest - https://opendev.org/openstack/tempest/commit/571959dd2b1469ada0657ae701e4699044e3d79f - counted with a minor release, see the long discussion above about the 2024.1 vs 2024.2 , sooo if we don\u0027t release this, all that was for nothing.","commit_id":"6894fd2e66fa6ce7eb966f5354e41d311d820101"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9f46fe6eaec44e16bf65d96359797fac969a7d45","unresolved":false,"context_lines":[{"line_number":16,"context_line":"  - projects:"},{"line_number":17,"context_line":"      - repo: openstack/tempest"},{"line_number":18,"context_line":"        hash: 571959dd2b1469ada0657ae701e4699044e3d79f"},{"line_number":19,"context_line":"    version: 39.1.0"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"eb89cd16_d46916ec","line":19,"in_reply_to":"b970f12c_dfa40c23","updated":"2024-09-06 18:07:17.000000000","message":"Done","commit_id":"6894fd2e66fa6ce7eb966f5354e41d311d820101"}]}
