)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"13dce61fce0fd3c95d39946c45cc842cd382a712","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"10235889_2945983d","updated":"2026-05-14 11:53:21.000000000","message":"https://review.opendev.org/c/openstack/keystone/+/984715 has been backported to 2023.1, so we need this.","commit_id":"2d76aa9e5f194fc2a38052b8327eae9cdef056e8"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"1c100c80216a3d1b9575b47352973d06bdd8eb74","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"041bd00a_9bf33d09","updated":"2026-05-15 10:03:32.000000000","message":"thanks for backporting the patch to unmaintained branches! with a quick glance i have 2 comments for this:\n1) unmaintained/2024.1 version has to be merged first, as stable policy has to be followed on unmaintained branches as well\n2) please push a separate patch for the gate fix if possible, so that this patch will be a pure cherry pick of the bug fix\n\ngiving -1 to highlight my 2 comments","commit_id":"2d76aa9e5f194fc2a38052b8327eae9cdef056e8"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"516ea26e185848397a4f9d9909ad246dc626bc43","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"083ee1a8_c7381c97","updated":"2026-05-19 09:50:45.000000000","message":"unmaintained/2024.1 version has merged meanwhile, so this should be good to go.\n\nto note some differences:\n* .zuul.yaml and tox.ini contain changes / conflicts, due to branch specific settings.\n* heat/engine/clients/os/keystone/heat_keystoneclient.py doesn\u0027t need the bandit nosec annotation here, compared to unmaintained/2024.1 version. so it is not part of the patch.\n\nrest of the code changes are *clean cherry picks* from unmaintained/2024.1.\n\ngiven that the patch is coming from heat core team member and the fact that we are on unmaintained branches, i\u0027m fast-approving this.","commit_id":"2d76aa9e5f194fc2a38052b8327eae9cdef056e8"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"755ac4f2c02c9448334a397a4daede83b36351f2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d505ae19_3adb5df1","in_reply_to":"041bd00a_9bf33d09","updated":"2026-05-15 10:16:13.000000000","message":"2024.1 has already been proposed. I don\u0027t see any issues this patch containing the gate fix as the commit message mentions it. It\u0027s not a clean cherry-pick too.\n\nI would like also to know the guidelines for unmaintained branches and why the project stable core don\u0027t have merge rights for it please.","commit_id":"2d76aa9e5f194fc2a38052b8327eae9cdef056e8"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"371079c8193f223a27e55e4cdaa6ffaa8ca3e6be","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"643b7e56_225332c4","in_reply_to":"592d8272_e10e3f36","updated":"2026-05-15 11:20:15.000000000","message":"https://review.opendev.org/c/openstack/heat/+/988739 is for 2024.1","commit_id":"2d76aa9e5f194fc2a38052b8327eae9cdef056e8"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"3117dcc3a520beb6976d3a3d4b1b8de1b098c6a4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"592d8272_e10e3f36","in_reply_to":"d505ae19_3adb5df1","updated":"2026-05-15 11:12:37.000000000","message":"I forgot to mention that CI fixes like setuptools pin has to go with this patch for green CI. They can\u0027t be proposed separately. If we can merge without CI vote then I can remove them from this patch.","commit_id":"2d76aa9e5f194fc2a38052b8327eae9cdef056e8"}]}
