)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"9c4881bf00952a360ff59687082a1aa6180ac4de","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"55e0a2f3_ce9a871b","updated":"2022-10-04 14:20:06.000000000","message":"Impressively big release patch :) see my comments inline.","commit_id":"c4ac4c817ae4155fa98055920f7b11fc0db8a35c"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"6bdfad47a91be48405870c1eaf01a4a0fe9a3478","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8198c167_60b968f1","updated":"2022-10-06 09:38:21.000000000","message":"Based on JayF\u0027s message on IRC [1], this patch probably gets an update soon.\n\n[1] https://meetings.opendev.org/irclogs/%23openstack-release/%23openstack-release.2022-10-04.log.html#t2022-10-04T15:58:56","commit_id":"e12226c9fffba2ecb4143a232a4928f996a29474"}],"deliverables/wallaby/ironic-inspector.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"9c4881bf00952a360ff59687082a1aa6180ac4de","unresolved":true,"context_lines":[{"line_number":27,"context_line":"  - version: 10.6.3"},{"line_number":28,"context_line":"    projects:"},{"line_number":29,"context_line":"      - repo: openstack/ironic-inspector"},{"line_number":30,"context_line":"        hash: b52f9744cbff5b8c47faa6f07ddd4a336ac99936"},{"line_number":31,"context_line":"branches:"},{"line_number":32,"context_line":"  - name: bugfix/10.5"},{"line_number":33,"context_line":"    location: 10.5.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"867722e2_e5596484","line":30,"range":{"start_line":30,"start_character":14,"end_line":30,"end_character":54},"updated":"2022-10-04 14:20:06.000000000","message":"Contains a single, test-only change. Usually it\u0027s not worth to release a package that does not have a functional change. So I\u0027d rather drop this release in case there isn\u0027t any special reason to release this.","commit_id":"c4ac4c817ae4155fa98055920f7b11fc0db8a35c"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"b920344673db014580ceebfc57451703e31dae45","unresolved":false,"context_lines":[{"line_number":27,"context_line":"  - version: 10.6.3"},{"line_number":28,"context_line":"    projects:"},{"line_number":29,"context_line":"      - repo: openstack/ironic-inspector"},{"line_number":30,"context_line":"        hash: b52f9744cbff5b8c47faa6f07ddd4a336ac99936"},{"line_number":31,"context_line":"branches:"},{"line_number":32,"context_line":"  - name: bugfix/10.5"},{"line_number":33,"context_line":"    location: 10.5.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"171af875_64792a54","line":30,"range":{"start_line":30,"start_character":14,"end_line":30,"end_character":54},"in_reply_to":"001ac1ee_fe8c3812","updated":"2022-10-04 15:38:11.000000000","message":"Done","commit_id":"c4ac4c817ae4155fa98055920f7b11fc0db8a35c"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"46a3915c3777da5cc8c203f93d6c00704fb38430","unresolved":true,"context_lines":[{"line_number":27,"context_line":"  - version: 10.6.3"},{"line_number":28,"context_line":"    projects:"},{"line_number":29,"context_line":"      - repo: openstack/ironic-inspector"},{"line_number":30,"context_line":"        hash: b52f9744cbff5b8c47faa6f07ddd4a336ac99936"},{"line_number":31,"context_line":"branches:"},{"line_number":32,"context_line":"  - name: bugfix/10.5"},{"line_number":33,"context_line":"    location: 10.5.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"001ac1ee_fe8c3812","line":30,"range":{"start_line":30,"start_character":14,"end_line":30,"end_character":54},"in_reply_to":"867722e2_e5596484","updated":"2022-10-04 14:32:09.000000000","message":"ack; will remove. Must have missed this when I was checking for changes in everything.","commit_id":"c4ac4c817ae4155fa98055920f7b11fc0db8a35c"}],"deliverables/wallaby/ironic.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"9c4881bf00952a360ff59687082a1aa6180ac4de","unresolved":true,"context_lines":[{"line_number":38,"context_line":"  - version: 17.0.5"},{"line_number":39,"context_line":"    projects:"},{"line_number":40,"context_line":"      - repo: openstack/ironic"},{"line_number":41,"context_line":"        hash: a85d689453223530b8ec867078653f58311a7c90"},{"line_number":42,"context_line":"branches:"},{"line_number":43,"context_line":"  - name: bugfix/16.1"},{"line_number":44,"context_line":"    location: 16.1.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"f1399367_1f3e0055","line":41,"range":{"start_line":41,"start_character":14,"end_line":41,"end_character":54},"updated":"2022-10-04 14:20:06.000000000","message":"This release contains a new dependency [1][2] (which is in general should be avoided in stable branches) so I think it is better to bump MINOR version instead (17.1.0; fortunately that is free as the first release on xena was 18.0.0).\n\n[1] https://zuul.opendev.org/t/openstack/build/828fea08f31740489aa9fc8a73926eb1/log/tox/validate-request-results.log#62757\n[2] https://zuul.opendev.org/t/openstack/build/20c6324b935f421db9087d5206c14a37/log/tox/list-changes-results.log#1526","commit_id":"c4ac4c817ae4155fa98055920f7b11fc0db8a35c"}],"deliverables/xena/ironic.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"9c4881bf00952a360ff59687082a1aa6180ac4de","unresolved":true,"context_lines":[{"line_number":26,"context_line":"  - version: 18.2.2"},{"line_number":27,"context_line":"    projects:"},{"line_number":28,"context_line":"      - repo: openstack/ironic"},{"line_number":29,"context_line":"        hash: 66373aadc04974de0838abc43c5f9cf113781f1f"},{"line_number":30,"context_line":"branches:"},{"line_number":31,"context_line":"  - name: bugfix/18.0"},{"line_number":32,"context_line":"    location: 18.0.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"2b99f6f9_8ba7cac1","line":29,"updated":"2022-10-04 14:20:06.000000000","message":"Same as with wallaby version, I suggest MINOR version bump instead (18.3.0) based on the same reason.","commit_id":"c4ac4c817ae4155fa98055920f7b11fc0db8a35c"}],"deliverables/yoga/ironic-inspector.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"9c4881bf00952a360ff59687082a1aa6180ac4de","unresolved":true,"context_lines":[{"line_number":23,"context_line":"  - version: 10.11.1"},{"line_number":24,"context_line":"    projects:"},{"line_number":25,"context_line":"      - repo: openstack/ironic-inspector"},{"line_number":26,"context_line":"        hash: 0f70f89fbe146866a7a390d99db6c15c22939fce"},{"line_number":27,"context_line":"branches:"},{"line_number":28,"context_line":"  - name: bugfix/10.9"},{"line_number":29,"context_line":"    location: 10.9.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"2a76d389_cdad03bb","line":26,"updated":"2022-10-04 14:20:06.000000000","message":"This release would contain only git \u0026 tox related changes [1], thus no functional changes at all. So same as with the wallaby version, this does not worth a release.\n\n[1] https://zuul.opendev.org/t/openstack/build/20c6324b935f421db9087d5206c14a37/log/tox/list-changes-results.log#6035-6041","commit_id":"c4ac4c817ae4155fa98055920f7b11fc0db8a35c"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"46a3915c3777da5cc8c203f93d6c00704fb38430","unresolved":true,"context_lines":[{"line_number":23,"context_line":"  - version: 10.11.1"},{"line_number":24,"context_line":"    projects:"},{"line_number":25,"context_line":"      - repo: openstack/ironic-inspector"},{"line_number":26,"context_line":"        hash: 0f70f89fbe146866a7a390d99db6c15c22939fce"},{"line_number":27,"context_line":"branches:"},{"line_number":28,"context_line":"  - name: bugfix/10.9"},{"line_number":29,"context_line":"    location: 10.9.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"8f530718_af6011c5","line":26,"in_reply_to":"2a76d389_cdad03bb","updated":"2022-10-04 14:32:09.000000000","message":"ack","commit_id":"c4ac4c817ae4155fa98055920f7b11fc0db8a35c"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"b920344673db014580ceebfc57451703e31dae45","unresolved":false,"context_lines":[{"line_number":23,"context_line":"  - version: 10.11.1"},{"line_number":24,"context_line":"    projects:"},{"line_number":25,"context_line":"      - repo: openstack/ironic-inspector"},{"line_number":26,"context_line":"        hash: 0f70f89fbe146866a7a390d99db6c15c22939fce"},{"line_number":27,"context_line":"branches:"},{"line_number":28,"context_line":"  - name: bugfix/10.9"},{"line_number":29,"context_line":"    location: 10.9.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"87afa720_a83bc4ca","line":26,"in_reply_to":"8f530718_af6011c5","updated":"2022-10-04 15:38:11.000000000","message":"Done","commit_id":"c4ac4c817ae4155fa98055920f7b11fc0db8a35c"}],"deliverables/yoga/ironic.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"9c4881bf00952a360ff59687082a1aa6180ac4de","unresolved":false,"context_lines":[{"line_number":22,"context_line":"  - version: 20.1.1"},{"line_number":23,"context_line":"    projects:"},{"line_number":24,"context_line":"      - repo: openstack/ironic"},{"line_number":25,"context_line":"        hash: ec4dffe98647649096ed9f832a0f585d8bab0b46"},{"line_number":26,"context_line":"branches:"},{"line_number":27,"context_line":"  - name: bugfix/19.0"},{"line_number":28,"context_line":"    location: 19.0.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ab180f03_87c76725","line":25,"updated":"2022-10-04 14:20:06.000000000","message":"The same comments would also valid for ironic yoga as for xena and wallaby versions, but unfortunately the 20.2.0 version is already taken on stable/zed...","commit_id":"c4ac4c817ae4155fa98055920f7b11fc0db8a35c"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"b920344673db014580ceebfc57451703e31dae45","unresolved":true,"context_lines":[{"line_number":22,"context_line":"  - version: 20.1.1"},{"line_number":23,"context_line":"    projects:"},{"line_number":24,"context_line":"      - repo: openstack/ironic"},{"line_number":25,"context_line":"        hash: ec4dffe98647649096ed9f832a0f585d8bab0b46"},{"line_number":26,"context_line":"branches:"},{"line_number":27,"context_line":"  - name: bugfix/19.0"},{"line_number":28,"context_line":"    location: 19.0.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"d7129da4_a4a45830","line":25,"in_reply_to":"58c83650_84185865","updated":"2022-10-04 15:38:11.000000000","message":"I can accept if the team wants to release all with only PATCH version bumps.\n(The question is, what would a consumer prefer: to have a better matching semantic version OR to have a matching bump for all the stable branches)","commit_id":"c4ac4c817ae4155fa98055920f7b11fc0db8a35c"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"46a3915c3777da5cc8c203f93d6c00704fb38430","unresolved":false,"context_lines":[{"line_number":22,"context_line":"  - version: 20.1.1"},{"line_number":23,"context_line":"    projects:"},{"line_number":24,"context_line":"      - repo: openstack/ironic"},{"line_number":25,"context_line":"        hash: ec4dffe98647649096ed9f832a0f585d8bab0b46"},{"line_number":26,"context_line":"branches:"},{"line_number":27,"context_line":"  - name: bugfix/19.0"},{"line_number":28,"context_line":"    location: 19.0.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"58c83650_84185865","line":25,"in_reply_to":"ab180f03_87c76725","updated":"2022-10-04 14:32:09.000000000","message":"So:\n- just a new added requirement, not a changed one\nand\n- we cannot consistently bump as a minor version across all branches\n\n\nI think my preference would be to just release as a patch version. I\u0027d certainly prefer that to doing a mix of minor and bugfix bumps for different branches.\n\nWhat do you think?","commit_id":"c4ac4c817ae4155fa98055920f7b11fc0db8a35c"}]}
