)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"86f24f88cae742cb20630ba6afdacd46f33a5db3","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Add setuptools\u003c82 to requirements for doc job so"},{"line_number":16,"context_line":"that the openstackdocstheme works that needs"},{"line_number":17,"context_line":"pkg_resources available."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Remove devstack jobs because unmaintained branches"},{"line_number":20,"context_line":"no longer works."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"4f4eab0d_be9c3bf3","line":17,"updated":"2026-06-10 11:43:12.000000000","message":"setuptools pinning can be done via tox.ini, however if mistral-lib is using pkg_resources, then it\u0027s probably OK to add it to requirements.","commit_id":"2f663fb63211711814ea5cf62bf504498bdcc1c4"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"86f24f88cae742cb20630ba6afdacd46f33a5db3","unresolved":true,"context_lines":[{"line_number":16,"context_line":"that the openstackdocstheme works that needs"},{"line_number":17,"context_line":"pkg_resources available."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Remove devstack jobs because unmaintained branches"},{"line_number":20,"context_line":"no longer works."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Remove requirements check for unmaintained branch"},{"line_number":23,"context_line":"because we will need to pin things that conflicts"},{"line_number":24,"context_line":"with global requirements (such as setuptools) for"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"d67f91b3_4ed1dc09","line":21,"range":{"start_line":19,"start_character":0,"end_line":21,"end_character":1},"updated":"2026-06-10 11:43:12.000000000","message":"the devstack jobs are not working because those jobs are using mistral-tempest-plugins from master branch. That probably could be fixed by pinning tempest and the plugin to 2024.1-last tagged versions.\nWe say that unmaintained branches are not fully covered by CI jobs, but I\u0027d rather not throw away all these jobs. because then it\u0027s more easy to introduce regression :-/","commit_id":"2f663fb63211711814ea5cf62bf504498bdcc1c4"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"86f24f88cae742cb20630ba6afdacd46f33a5db3","unresolved":false,"context_lines":[{"line_number":22,"context_line":"Remove requirements check for unmaintained branch"},{"line_number":23,"context_line":"because we will need to pin things that conflicts"},{"line_number":24,"context_line":"with global requirements (such as setuptools) for"},{"line_number":25,"context_line":"older versions testing to work."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Remove non-voting jobs for unmaintained branches"},{"line_number":28,"context_line":"as they just waste CI resources."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"2c3128fd_f2adbe66","line":25,"updated":"2026-06-10 11:43:12.000000000","message":"yepp, if we change something in requirements file, then this is needed :/","commit_id":"2f663fb63211711814ea5cf62bf504498bdcc1c4"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"86f24f88cae742cb20630ba6afdacd46f33a5db3","unresolved":false,"context_lines":[{"line_number":25,"context_line":"older versions testing to work."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Remove non-voting jobs for unmaintained branches"},{"line_number":28,"context_line":"as they just waste CI resources."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Backport race condition fixes [1] [2] to the"},{"line_number":31,"context_line":"unmaintained/2024.1 branch."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"f46a3337_634d3a90","line":28,"updated":"2026-06-10 11:43:12.000000000","message":"✅","commit_id":"2f663fb63211711814ea5cf62bf504498bdcc1c4"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"86f24f88cae742cb20630ba6afdacd46f33a5db3","unresolved":false,"context_lines":[{"line_number":28,"context_line":"as they just waste CI resources."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Backport race condition fixes [1] [2] to the"},{"line_number":31,"context_line":"unmaintained/2024.1 branch."},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"Remove mysql and postgresl unit tests as they"},{"line_number":34,"context_line":"are error prone to race conditions and let\u0027s"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"5e24eb61_5ac5d336","line":31,"updated":"2026-06-10 11:43:12.000000000","message":"✅","commit_id":"2f663fb63211711814ea5cf62bf504498bdcc1c4"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"0ff7205d4288e779ddc916aaa86d38fe216eb27d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"338bfdb4_34b07fc7","updated":"2026-06-04 12:13:25.000000000","message":"recheck","commit_id":"3c88001af6ca8d80fa2446f6d6f5774efe9ecf13"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"86f24f88cae742cb20630ba6afdacd46f33a5db3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bbc8f9e1_9fb65bd6","updated":"2026-06-10 11:43:12.000000000","message":"this mostly looks OK, however, i\u0027d rather try keep the tempest jobs to have a meaningful gate. i\u0027ll prepare a DNM patch with the pinnings and if that works then I\u0027ll update this patch.","commit_id":"2f663fb63211711814ea5cf62bf504498bdcc1c4"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"87bc0adf3b7e74c48d70b06f5db9bc30cf797fb9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8356d306_307242bb","in_reply_to":"bbc8f9e1_9fb65bd6","updated":"2026-06-10 15:23:16.000000000","message":"so the DNM patch [1] seems to did the trick, i\u0027ve squashed it to this patch.\n\n[1] https://review.opendev.org/c/openstack/mistral/+/992654","commit_id":"2f663fb63211711814ea5cf62bf504498bdcc1c4"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"d4cb4786595e14558adb800fa25579b5612a1e99","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"355ba8c0_34b1288b","updated":"2026-06-11 07:49:13.000000000","message":"LGTM","commit_id":"8619378fdd5db378e234716709aeaeb4bb8cce50"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"b64c05aa9dd7a0c3d3834441ba499d690f3934d7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"eb6e86a0_8478c313","in_reply_to":"355ba8c0_34b1288b","updated":"2026-06-11 07:56:49.000000000","message":"ACK, let\u0027s fix the gate then. Thanks Arnaud!","commit_id":"8619378fdd5db378e234716709aeaeb4bb8cce50"}]}
