)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"dfa378c115287547f561527913c09a2c152835d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a9feba2d_76eb5956","updated":"2023-04-28 08:53:26.000000000","message":"Giving -1 to highlight my comment.","commit_id":"8f06a5ffafc7246c15597dd4f5f759527d87f9b8"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"c90b9ed80894a840bafdc77cdf3939baeafc130e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e43241d4_0cd8156b","updated":"2023-04-28 12:13:18.000000000","message":"comments inline","commit_id":"8f06a5ffafc7246c15597dd4f5f759527d87f9b8"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"e88bf50453e9a02f4d9ef92937d6bf863ce32cc4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"6b309fea_01eeaf3b","updated":"2023-04-28 12:45:45.000000000","message":"Thanks for the review \u0026 for the update. Agreeing with Rajat, LGTM.","commit_id":"f88738e68e89ee82fd12639d7e5b46c2ade611a2"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"c893bd0ffb5407119d64b7f0a5469c4209cfd0f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5109b0d0_d42d5607","updated":"2023-05-03 07:52:01.000000000","message":"forgot to re-review this but looks good.","commit_id":"f88738e68e89ee82fd12639d7e5b46c2ade611a2"}],"deliverables/antelope/cinder.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"dfa378c115287547f561527913c09a2c152835d4","unresolved":true,"context_lines":[{"line_number":38,"context_line":"  - version: 22.1.0"},{"line_number":39,"context_line":"    projects:"},{"line_number":40,"context_line":"      - repo: openstack/cinder"},{"line_number":41,"context_line":"        hash: 63e6dfdb081685e6955409082ed0fadd73b5166b"},{"line_number":42,"context_line":"branches:"},{"line_number":43,"context_line":"  - name: stable/2023.1"},{"line_number":44,"context_line":"    location: 22.0.0.0rc1"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"19fd5340_f80d4261","line":41,"updated":"2023-04-28 08:53:26.000000000","message":"I don\u0027t see why do we need a MINOR version jump here, since the only change is a documentation change. I don\u0027t even think this is worth a release, to be honest.","commit_id":"8f06a5ffafc7246c15597dd4f5f759527d87f9b8"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"c90b9ed80894a840bafdc77cdf3939baeafc130e","unresolved":true,"context_lines":[{"line_number":38,"context_line":"  - version: 22.1.0"},{"line_number":39,"context_line":"    projects:"},{"line_number":40,"context_line":"      - repo: openstack/cinder"},{"line_number":41,"context_line":"        hash: 63e6dfdb081685e6955409082ed0fadd73b5166b"},{"line_number":42,"context_line":"branches:"},{"line_number":43,"context_line":"  - name: stable/2023.1"},{"line_number":44,"context_line":"    location: 22.0.0.0rc1"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ee0062be_f577e3ce","line":41,"in_reply_to":"19fd5340_f80d4261","updated":"2023-04-28 12:13:18.000000000","message":"I agree with Elod. the documentation change is also for a driver and not a core cinder feature so let\u0027s avoid releasing cinder here.\nAntelope has recently released and it\u0027s early in bobcat so less backports so it\u0027s understandable not to release cinder.","commit_id":"8f06a5ffafc7246c15597dd4f5f759527d87f9b8"}],"deliverables/antelope/os-brick.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"dfa378c115287547f561527913c09a2c152835d4","unresolved":false,"context_lines":[{"line_number":14,"context_line":"  - version: 6.3.0"},{"line_number":15,"context_line":"    projects:"},{"line_number":16,"context_line":"      - repo: openstack/os-brick"},{"line_number":17,"context_line":"        hash: 9bcd759e987aa2d28a79bdd6d9d0e2fac85f7eea"},{"line_number":18,"context_line":"branches:"},{"line_number":19,"context_line":"  - name: stable/2023.1"},{"line_number":20,"context_line":"    location: 6.2.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"63724f91_a9a53605","line":17,"updated":"2023-04-28 08:53:26.000000000","message":"The only change here is to add py310 classifier to setup.cfg. This is again somewhere on the border to be a valid backport, but it has no downside to backport it so why not... technically this can be considered as a \u0027feature\u0027, hence MINOR version bump is good here (though, only because we don\u0027t have a release yet on Bobcat).","commit_id":"8f06a5ffafc7246c15597dd4f5f759527d87f9b8"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"c90b9ed80894a840bafdc77cdf3939baeafc130e","unresolved":false,"context_lines":[{"line_number":14,"context_line":"  - version: 6.3.0"},{"line_number":15,"context_line":"    projects:"},{"line_number":16,"context_line":"      - repo: openstack/os-brick"},{"line_number":17,"context_line":"        hash: 9bcd759e987aa2d28a79bdd6d9d0e2fac85f7eea"},{"line_number":18,"context_line":"branches:"},{"line_number":19,"context_line":"  - name: stable/2023.1"},{"line_number":20,"context_line":"    location: 6.2.0"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"05993d47_736453d4","line":17,"in_reply_to":"63724f91_a9a53605","updated":"2023-04-28 12:13:18.000000000","message":"The backport was done because python 3.10 was a supported runtime for antelope[1] but we missed to make this change during 2023.1 Antelope cycle. I\u0027m not sure if it justifies the backport but just stating the reason of doing it.\n\nAlso for libraries, we had discussions in the past that even for features, security issues etc we prefer patch version bump and not minor. I still think we should only do 6.2.1 and use 6.3.0 for 2023.2 Bobcat maintaining the same convention.\n\n[1] https://governance.openstack.org/tc/reference/runtimes/2023.1.html","commit_id":"8f06a5ffafc7246c15597dd4f5f759527d87f9b8"}]}
