)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"change_message_id":"f33c54100f7226ff4ce9cd0469de7b29de2a488d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ff9e490a_8fdc4102","updated":"2022-12-08 10:41:29.000000000","message":"LGTM...","commit_id":"3b4694b9df12d8851f499daa2f352a2667905a43"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"efad48ed2b98a6dd7c51bf15b93f980e43f8c682","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fc3ed1a0_c1da777b","updated":"2022-12-08 12:17:52.000000000","message":"please remove the wallaby release","commit_id":"3b4694b9df12d8851f499daa2f352a2667905a43"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"98fea1dd5213defa86f01e49fb4795b5d4657c51","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9ae8059f_4628306d","updated":"2023-01-04 13:11:36.000000000","message":"Hi, sorry for not reviewing this earlier. I see some versioning problems. Can you please have a look at my comments? (Giving -1 to highlight my questions)","commit_id":"96cb135e87c60a7ffe667166b303b086f8e4608a"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"7781bd5fd71d67470a99cdcdbfbc7081f1f073c3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"340776f3_c359223d","updated":"2023-01-09 08:54:49.000000000","message":"Thanks for the updates Jake! Release patch looks good to me now.","commit_id":"014450c7d4aeddde34a34693fad570babd4a67be"},{"author":{"_account_id":8064,"name":"Jake Yip","email":"jake.yip@ardc.edu.au","username":"jake"},"change_message_id":"6bb07d8ccde268b93ae00d35dbd25baeea7ab8d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0ad3378a_0bb7c8a2","updated":"2023-01-06 01:45:28.000000000","message":"Thanks for the valuable feedback Elod, it is much appreciated.\n\nI\u0027ve amended accordingly. Please see replies to your comments and let me know if anything still needs improvement.","commit_id":"014450c7d4aeddde34a34693fad570babd4a67be"}],"deliverables/wallaby/magnum.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"efad48ed2b98a6dd7c51bf15b93f980e43f8c682","unresolved":true,"context_lines":[{"line_number":38,"context_line":"    projects:"},{"line_number":39,"context_line":"      - repo: openstack/magnum"},{"line_number":40,"context_line":"        hash: 5870994fcd51fc2695105968b708b862c4ae9d94"},{"line_number":41,"context_line":"  - version: 12.1.2"},{"line_number":42,"context_line":"    projects:"},{"line_number":43,"context_line":"      - repo: openstack/magnum"},{"line_number":44,"context_line":"        hash: 02ee44b839ed3a2182f2c039ae6caa1f31815d45"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ee7d23ee_f6bdb0c9","line":41,"range":{"start_line":41,"start_character":13,"end_line":41,"end_character":19},"updated":"2022-12-08 12:17:52.000000000","message":"Wallaby is in Extended Maintenance already, which means no official releases are accepted. (It\u0027s strange that our validator did not fail on this)","commit_id":"3b4694b9df12d8851f499daa2f352a2667905a43"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"7781bd5fd71d67470a99cdcdbfbc7081f1f073c3","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    projects:"},{"line_number":39,"context_line":"      - repo: openstack/magnum"},{"line_number":40,"context_line":"        hash: 5870994fcd51fc2695105968b708b862c4ae9d94"},{"line_number":41,"context_line":"  - version: 12.1.2"},{"line_number":42,"context_line":"    projects:"},{"line_number":43,"context_line":"      - repo: openstack/magnum"},{"line_number":44,"context_line":"        hash: 02ee44b839ed3a2182f2c039ae6caa1f31815d45"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1982363c_b79777f7","line":41,"range":{"start_line":41,"start_character":13,"end_line":41,"end_character":19},"in_reply_to":"ee7d23ee_f6bdb0c9","updated":"2023-01-09 08:54:49.000000000","message":"Done","commit_id":"3b4694b9df12d8851f499daa2f352a2667905a43"}],"deliverables/xena/magnum.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"98fea1dd5213defa86f01e49fb4795b5d4657c51","unresolved":false,"context_lines":[{"line_number":15,"context_line":"      - repo: openstack/magnum"},{"line_number":16,"context_line":"        hash: 6951ac4fc6b6178637fac7bd5d05096adb5c6bdc"},{"line_number":17,"context_line":"    diff-start: 12.0.0.0rc1"},{"line_number":18,"context_line":"  - version: 13.1.0"},{"line_number":19,"context_line":"    projects:"},{"line_number":20,"context_line":"      - repo: openstack/magnum"},{"line_number":21,"context_line":"        hash: 5a32e40d0497f68b834e4a5fb6fed3901a85054d"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"50010392_ab5e0018","line":18,"range":{"start_line":18,"start_character":13,"end_line":18,"end_character":19},"updated":"2023-01-04 13:11:36.000000000","message":"This is probably OK, as there are many changes, some can be considered with bigger impacts:\n* \u0027Fix the default volume api version\u0027 - this changes default config value for cinder api version\n* \u0027Optimize cluster list api\u0027 - this looks \"feature-ish\"\n\n(Note: on stable branches it is best to avoid feature backports and default config value changes. The best is to backport bugfixes only. Though I know that magnum is not listed amongst projects that follow stable policy)","commit_id":"96cb135e87c60a7ffe667166b303b086f8e4608a"},{"author":{"_account_id":8064,"name":"Jake Yip","email":"jake.yip@ardc.edu.au","username":"jake"},"change_message_id":"6bb07d8ccde268b93ae00d35dbd25baeea7ab8d0","unresolved":false,"context_lines":[{"line_number":15,"context_line":"      - repo: openstack/magnum"},{"line_number":16,"context_line":"        hash: 6951ac4fc6b6178637fac7bd5d05096adb5c6bdc"},{"line_number":17,"context_line":"    diff-start: 12.0.0.0rc1"},{"line_number":18,"context_line":"  - version: 13.1.0"},{"line_number":19,"context_line":"    projects:"},{"line_number":20,"context_line":"      - repo: openstack/magnum"},{"line_number":21,"context_line":"        hash: 5a32e40d0497f68b834e4a5fb6fed3901a85054d"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"fb5e1f2f_e255844c","line":18,"range":{"start_line":18,"start_character":13,"end_line":18,"end_character":19},"in_reply_to":"50010392_ab5e0018","updated":"2023-01-06 01:45:28.000000000","message":"Apologies for that, we will try to limit backports to bugfixes in the future.\n\n(Please see my comments for yoga too.)","commit_id":"96cb135e87c60a7ffe667166b303b086f8e4608a"}],"deliverables/yoga/magnum.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"98fea1dd5213defa86f01e49fb4795b5d4657c51","unresolved":true,"context_lines":[{"line_number":19,"context_line":"      - repo: openstack/magnum"},{"line_number":20,"context_line":"        hash: 2ae6cd95f18a1675ec6059e6434ee58a1868b07f"},{"line_number":21,"context_line":"    diff-start: 13.0.0"},{"line_number":22,"context_line":"  - version: 14.1.0"},{"line_number":23,"context_line":"    projects:"},{"line_number":24,"context_line":"      - repo: openstack/magnum"},{"line_number":25,"context_line":"        hash: f77471678d13a13dddfc54dbdf41f050f43e2660"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"bc7edf41_090fca5e","line":22,"updated":"2023-01-04 13:11:36.000000000","message":"Only 3 changes [1] are listed here, out of which 1 is purely CI related. I think we don\u0027t need here the MINOR version bump, 14.0.1 could be enough. Do you insist to use 14.1.0?\n\n[1] \nf7747167 2022-11-23 12:03:55 -0300 Make configure-agent-env.service idempotent.\n4d30bef6 2022-10-14 09:06:36 +0000 Allow update cluster status with admin context\n46e10625 2022-10-14 05:17:28 +0000 [CI] move queue setting to project level","commit_id":"96cb135e87c60a7ffe667166b303b086f8e4608a"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"7781bd5fd71d67470a99cdcdbfbc7081f1f073c3","unresolved":false,"context_lines":[{"line_number":19,"context_line":"      - repo: openstack/magnum"},{"line_number":20,"context_line":"        hash: 2ae6cd95f18a1675ec6059e6434ee58a1868b07f"},{"line_number":21,"context_line":"    diff-start: 13.0.0"},{"line_number":22,"context_line":"  - version: 14.1.0"},{"line_number":23,"context_line":"    projects:"},{"line_number":24,"context_line":"      - repo: openstack/magnum"},{"line_number":25,"context_line":"        hash: f77471678d13a13dddfc54dbdf41f050f43e2660"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"f58d5e07_173654f5","line":22,"in_reply_to":"1d81048f_a8366582","updated":"2023-01-09 08:54:49.000000000","message":"Ack","commit_id":"96cb135e87c60a7ffe667166b303b086f8e4608a"},{"author":{"_account_id":8064,"name":"Jake Yip","email":"jake.yip@ardc.edu.au","username":"jake"},"change_message_id":"6bb07d8ccde268b93ae00d35dbd25baeea7ab8d0","unresolved":true,"context_lines":[{"line_number":19,"context_line":"      - repo: openstack/magnum"},{"line_number":20,"context_line":"        hash: 2ae6cd95f18a1675ec6059e6434ee58a1868b07f"},{"line_number":21,"context_line":"    diff-start: 13.0.0"},{"line_number":22,"context_line":"  - version: 14.1.0"},{"line_number":23,"context_line":"    projects:"},{"line_number":24,"context_line":"      - repo: openstack/magnum"},{"line_number":25,"context_line":"        hash: f77471678d13a13dddfc54dbdf41f050f43e2660"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"1d81048f_a8366582","line":22,"in_reply_to":"bc7edf41_090fca5e","updated":"2023-01-06 01:45:28.000000000","message":"I think Rico is concerned with Change I9cc9a0e654fb26ebec517e3413a592ac6613777c which is a significant bug fix in terms of users\u0027 interaction, and it is fine to bump MINOR to reflect that. What do you think?","commit_id":"96cb135e87c60a7ffe667166b303b086f8e4608a"}],"deliverables/zed/magnum.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"98fea1dd5213defa86f01e49fb4795b5d4657c51","unresolved":true,"context_lines":[{"line_number":19,"context_line":"      - repo: openstack/magnum"},{"line_number":20,"context_line":"        hash: b5a36932e9013308b5732c6bc62af375d62be979"},{"line_number":21,"context_line":"    diff-start: 14.0.0.0rc1"},{"line_number":22,"context_line":"  - version: 15.1.0"},{"line_number":23,"context_line":"    projects:"},{"line_number":24,"context_line":"      - repo: openstack/magnum"},{"line_number":25,"context_line":"        hash: 11348ce7a818b24e34ea1d861f22e65af5592e3b"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"d4d6c381_ea82c0f4","line":22,"range":{"start_line":22,"start_character":13,"end_line":22,"end_character":19},"updated":"2023-01-04 13:11:36.000000000","message":"This release contains only 1 patch: \u0027Make configure-agent-env.service idempotent.\u0027\n\nI don\u0027t think that MINOR version bump is needed here. Can 15.0.1 used here instead?","commit_id":"96cb135e87c60a7ffe667166b303b086f8e4608a"},{"author":{"_account_id":8064,"name":"Jake Yip","email":"jake.yip@ardc.edu.au","username":"jake"},"change_message_id":"6bb07d8ccde268b93ae00d35dbd25baeea7ab8d0","unresolved":false,"context_lines":[{"line_number":19,"context_line":"      - repo: openstack/magnum"},{"line_number":20,"context_line":"        hash: b5a36932e9013308b5732c6bc62af375d62be979"},{"line_number":21,"context_line":"    diff-start: 14.0.0.0rc1"},{"line_number":22,"context_line":"  - version: 15.1.0"},{"line_number":23,"context_line":"    projects:"},{"line_number":24,"context_line":"      - repo: openstack/magnum"},{"line_number":25,"context_line":"        hash: 11348ce7a818b24e34ea1d861f22e65af5592e3b"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7e87003d_631f5d7a","line":22,"range":{"start_line":22,"start_character":13,"end_line":22,"end_character":19},"in_reply_to":"d4d6c381_ea82c0f4","updated":"2023-01-06 01:45:28.000000000","message":"Done","commit_id":"96cb135e87c60a7ffe667166b303b086f8e4608a"}]}
