)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"058d631573ad39b6c119a588bd9e615c996c30bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"35f8ee09_615734ac","updated":"2022-11-18 15:40:43.000000000","message":"@fungi: answerd your question inline.","commit_id":"e83e05d8e1bbf3a03f6d4ddcb1e9aa646b9e7b1d"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"ff94c4ab52eb0cbb351280c8e7f3c4e1938876e7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"33bf4d38_29608e45","updated":"2022-11-18 10:28:18.000000000","message":"LGTM:\n* MINOR version bump is necessary (1st release on cycle)\n* approved by release liaison\n* no issues found in test job logs","commit_id":"e83e05d8e1bbf3a03f6d4ddcb1e9aa646b9e7b1d"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"fa38a0c0945e229e7001e4684316dabcc32afabf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"32243f0f_d42778a9","updated":"2022-11-24 10:29:53.000000000","message":"Personally I\u0027d prefer to go with a minor version as the one given by this patch set.\nI agree with fungi concerning the policy topic and the case related to the major version.","commit_id":"e83e05d8e1bbf3a03f6d4ddcb1e9aa646b9e7b1d"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"3bd079e4e4bfa28bb4ba29bf633c165c5e29512d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d4be630c_b82ccc6e","updated":"2022-11-17 14:15:12.000000000","message":"The only question is have: do we need/want to make releases of other\ncinder subprojects at this time?\n","commit_id":"e83e05d8e1bbf3a03f6d4ddcb1e9aa646b9e7b1d"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"939d5cda4e645a4824620b630065bc600b3ba59c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"033d27ce_f95a60e9","updated":"2022-11-17 14:13:45.000000000","message":"This is the same patch that I would have submitted so no complaints from\nme, I think this looks good.","commit_id":"e83e05d8e1bbf3a03f6d4ddcb1e9aa646b9e7b1d"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"0d3f3062bc68cd9b7e361053bffe471ebdec093d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"24b8da74_eaf77a83","updated":"2022-11-18 14:54:22.000000000","message":"one concern inline.","commit_id":"e83e05d8e1bbf3a03f6d4ddcb1e9aa646b9e7b1d"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"ca9ca963d064830d5d28f64e1462711235217741","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"aef38d4e_de872b87","in_reply_to":"32243f0f_d42778a9","updated":"2022-11-24 13:21:53.000000000","message":"I remembered wrongly probably by thinking we are not that strict for cwi deliverables, anyway, we discussed it on IRC [1] and based on the opinions there, let\u0027s stick to MINOR version bump then. I don\u0027t +W this now, but let the team to ACK from their side before we proceed.\n\n[1] https://meetings.opendev.org/irclogs/%23openstack-release/%23openstack-release.2022-11-18.log.html#t2022-11-18T15:09:09","commit_id":"e83e05d8e1bbf3a03f6d4ddcb1e9aa646b9e7b1d"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"1949d0f490c5b6267302faa8d145f2227590f18c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"706ec8c8_81d6b9a8","in_reply_to":"aef38d4e_de872b87","updated":"2022-11-25 15:09:13.000000000","message":"If for the mentioned scenarios (like requirement change in case of security issues), we are good with patch bump then I\u0027m OK with this release.","commit_id":"e83e05d8e1bbf3a03f6d4ddcb1e9aa646b9e7b1d"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"ff94c4ab52eb0cbb351280c8e7f3c4e1938876e7","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f5e32581_aeb7a2e8","in_reply_to":"d4be630c_b82ccc6e","updated":"2022-11-18 10:28:18.000000000","message":"Feel free to propose any release patch for other cycle-with-intermediary deliverables. Hervé generated release patches only for those libraries that have any code change since zed release (omitting CI / test / tox / etc. patches).","commit_id":"e83e05d8e1bbf3a03f6d4ddcb1e9aa646b9e7b1d"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"0d3f3062bc68cd9b7e361053bffe471ebdec093d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"aa68bbbc_dc81f71e","in_reply_to":"f5e32581_aeb7a2e8","updated":"2022-11-18 14:54:22.000000000","message":"Apart from cinderclient, we\u0027ve os-brick[1] and python-brick-cinderclient-ext[2] which had last commits on 23rd September so nothing new after Zed release and doesn\u0027t seem to be a benefit by releasing them.\n\n[1] https://github.com/openstack/os-brick/commits/master\n[2]  https://github.com/openstack/python-brick-cinderclient-ext/commits/master","commit_id":"e83e05d8e1bbf3a03f6d4ddcb1e9aa646b9e7b1d"}],"deliverables/antelope/python-cinderclient.yaml":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"c33cd2564495b8524a9876a07f31c64321b6120c","unresolved":false,"context_lines":[{"line_number":7,"context_line":"repository-settings:"},{"line_number":8,"context_line":"  openstack/python-cinderclient: {}"},{"line_number":9,"context_line":"releases:"},{"line_number":10,"context_line":"  - version: 9.2.0"},{"line_number":11,"context_line":"    projects:"},{"line_number":12,"context_line":"      - repo: openstack/python-cinderclient"},{"line_number":13,"context_line":"        hash: 9df653571d4da06c25222189be27e87a6da75628"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"0a912013_3f5e3b27","line":10,"updated":"2022-11-18 14:58:58.000000000","message":"Minor (Y) version increases signal feature additions or changes in external dependencies, which aren\u0027t something I\u0027d expect to see merged to a stable branch. When in the past was it needed?","commit_id":"e83e05d8e1bbf3a03f6d4ddcb1e9aa646b9e7b1d"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"d65d509a404495ae1e90e49ee61e036970901851","unresolved":false,"context_lines":[{"line_number":7,"context_line":"repository-settings:"},{"line_number":8,"context_line":"  openstack/python-cinderclient: {}"},{"line_number":9,"context_line":"releases:"},{"line_number":10,"context_line":"  - version: 9.2.0"},{"line_number":11,"context_line":"    projects:"},{"line_number":12,"context_line":"      - repo: openstack/python-cinderclient"},{"line_number":13,"context_line":"        hash: 9df653571d4da06c25222189be27e87a6da75628"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7be2ae87_ae43460b","line":10,"updated":"2022-11-18 17:33:17.000000000","message":"That\u0027s interesting. I thought by policy we didn\u0027t change external dependency versions in stable constraints, even when they announced security vulnerabilities. Doing so is a very slippery slope and implies to users that we think installing the versions of dependencies in the stable constraints lists is production safe (it\u0027s definitely not, that\u0027s not at all what it\u0027s for and we have things in there with widely-known vulnerabilities which we intentionally ignore because we\u0027re trying to stick with contemporary versions of them which lack those fixes).\n\nAnyway, it was pointed out in IRC that there was a case in the past where a stable branch policy violation was intentionally merged to os-brick and the maintainers wanted to be able to signal to consumers that they had done so. I suppose if this is going to be commonplace, we should change the policy to recommend major version increases after branching any library, since there\u0027s no way to predict whether or not they will actually follow policy over the course of the cycle. I\u0027m sure that problem is not limited to the Cinder team\u0027s deliverables alone.","commit_id":"e83e05d8e1bbf3a03f6d4ddcb1e9aa646b9e7b1d"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"0d3f3062bc68cd9b7e361053bffe471ebdec093d","unresolved":true,"context_lines":[{"line_number":7,"context_line":"repository-settings:"},{"line_number":8,"context_line":"  openstack/python-cinderclient: {}"},{"line_number":9,"context_line":"releases:"},{"line_number":10,"context_line":"  - version: 9.2.0"},{"line_number":11,"context_line":"    projects:"},{"line_number":12,"context_line":"      - repo: openstack/python-cinderclient"},{"line_number":13,"context_line":"        hash: 9df653571d4da06c25222189be27e87a6da75628"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"f68a69e7_ee67e56c","line":10,"range":{"start_line":10,"start_character":13,"end_line":10,"end_character":18},"updated":"2022-11-18 14:54:22.000000000","message":"Zed was 9.1.0 and now we are proposing 9.2.0 for antelope.\nThe concern which I\u0027ve faced in the past with this type of versioning is we\u0027ve very less room for stable releases when we\u0027ve bugfix backport and even for bugfixes we need to do the patch version bump Z in X.Y.Z and not minor bump because we are updating the minor version here and not the major version.","commit_id":"e83e05d8e1bbf3a03f6d4ddcb1e9aa646b9e7b1d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"058d631573ad39b6c119a588bd9e615c996c30bc","unresolved":true,"context_lines":[{"line_number":7,"context_line":"repository-settings:"},{"line_number":8,"context_line":"  openstack/python-cinderclient: {}"},{"line_number":9,"context_line":"releases:"},{"line_number":10,"context_line":"  - version: 9.2.0"},{"line_number":11,"context_line":"    projects:"},{"line_number":12,"context_line":"      - repo: openstack/python-cinderclient"},{"line_number":13,"context_line":"        hash: 9df653571d4da06c25222189be27e87a6da75628"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ac6bb8d4_5515cfbe","line":10,"in_reply_to":"0a912013_3f5e3b27","updated":"2022-11-18 15:40:43.000000000","message":"We\u0027ve needed this in the past when an external library has a security fix, and we must update the requirements.txt to raise the min version, and sometimes, modify our code that consumes the library.  We\u0027ve tried to use minor version bumps to signal that it\u0027s not just a patch release, but it\u0027s a serious change that operators should upgrade to.","commit_id":"e83e05d8e1bbf3a03f6d4ddcb1e9aa646b9e7b1d"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"3385fa103e1676153aad9e891f7d7ed6c2382d60","unresolved":true,"context_lines":[{"line_number":7,"context_line":"repository-settings:"},{"line_number":8,"context_line":"  openstack/python-cinderclient: {}"},{"line_number":9,"context_line":"releases:"},{"line_number":10,"context_line":"  - version: 9.2.0"},{"line_number":11,"context_line":"    projects:"},{"line_number":12,"context_line":"      - repo: openstack/python-cinderclient"},{"line_number":13,"context_line":"        hash: 9df653571d4da06c25222189be27e87a6da75628"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"c442555b_3bb5f81c","line":10,"range":{"start_line":10,"start_character":13,"end_line":10,"end_character":18},"in_reply_to":"8624d40e_34066549","updated":"2022-11-18 15:05:40.000000000","message":"Ack, I think using major versions would handle the case I\u0027ve mentioned above and also allows us to easily track which tag is associated to which branch like\n8 -\u003e Yoga\n9 -\u003e Zed\n10 -\u003e Antelope\n\nSo I would prefer the major bump. thanks.","commit_id":"e83e05d8e1bbf3a03f6d4ddcb1e9aa646b9e7b1d"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"9062126d3ee1e955e9fc0c183b8dc20119a6273d","unresolved":true,"context_lines":[{"line_number":7,"context_line":"repository-settings:"},{"line_number":8,"context_line":"  openstack/python-cinderclient: {}"},{"line_number":9,"context_line":"releases:"},{"line_number":10,"context_line":"  - version: 9.2.0"},{"line_number":11,"context_line":"    projects:"},{"line_number":12,"context_line":"      - repo: openstack/python-cinderclient"},{"line_number":13,"context_line":"        hash: 9df653571d4da06c25222189be27e87a6da75628"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"8624d40e_34066549","line":10,"range":{"start_line":10,"start_character":13,"end_line":10,"end_character":18},"in_reply_to":"f68a69e7_ee67e56c","updated":"2022-11-18 14:59:19.000000000","message":"cycle-with-intermediary model requires only MINOR version bump between cycles, but I think we can accept if a team want a MAJOR bumnp instead.","commit_id":"e83e05d8e1bbf3a03f6d4ddcb1e9aa646b9e7b1d"}]}