)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"change_message_id":"ccdcff3335d87e665417a43fefc7e93ca93d17c9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"178b9b84_6d12f84f","updated":"2021-10-11 09:52:25.000000000","message":"I agree with the comments that there are a few user-visible changes (parameters, operator...) in there  that make it worth of a 0.7.0 bump.","commit_id":"271553cdd3a60e9e4ad5709efddc6aee1c8184e2"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"07b6e0b2a5e0fe895103a6b8dc66b7b5e7e915c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b22729dc_50940e71","updated":"2021-10-11 12:55:01.000000000","message":"I understand that the team decided about 0.6.1, though the comments from the team also suggested that you are probably OK with MINOR version bump (0.7.1). On the other hand, there is requirements change as well (moreover, it\u0027s ansible [1]) and that also warrants MINOR version bump.\n\nDo you insist to the PATCH version bump (0.6.1) despite of the above things? (Giving -1 to highlight my question)\n\n[1] https://zuul.opendev.org/t/openstack/build/f3a95bd0da244a0c86007f1937402c21/log/tox/list-changes-results.log#234-242","commit_id":"271553cdd3a60e9e4ad5709efddc6aee1c8184e2"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"5943ba6c88b0a5276e3de35cc58e0edacded6ebc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8fed5a19_39a14719","in_reply_to":"178b9b84_6d12f84f","updated":"2021-10-11 10:04:09.000000000","message":"Thierry after this merges we will also publish the ansible collection with the same release number. I think sshnaidm would prefer to keep this as a .patch bump with 0.6.1 for now.\n\nSo are you +1 or -1 or neither? I mean are you blocking on this or just making an observation.","commit_id":"271553cdd3a60e9e4ad5709efddc6aee1c8184e2"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"41d1710ad5def8b059ad1e53395450647c883b77","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bd4a40ef_b8f89ef8","updated":"2021-10-11 13:03:24.000000000","message":"Because we added a new parameter it should be a minor change, found a guide in Ansible Collection docs: https://docs.ansible.com/ansible/devel/dev_guide/developing_collections_distributing.html#collection-versions","commit_id":"a56049782fe3c0ab489d7b0e729f63eddfee3407"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"a9aa34028af50a5f2a52bc9260f71b7f445f2727","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1fb05b3a_c2611ed5","updated":"2021-10-11 13:14:09.000000000","message":"Thanks for the update! Version bump looks OK to me.","commit_id":"a56049782fe3c0ab489d7b0e729f63eddfee3407"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"7fa05386abccefa326d5188ab02361d1aa1e9ba2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"13deaccd_7d53a032","updated":"2021-10-11 13:51:49.000000000","message":"Waiting for PTL approval","commit_id":"606f2243c7a6cd962785883942c6c743554d6e1c"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"01176df8a3c64a3dd3f720fe6d29bbcea881c46a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"59c0c53e_51c2ad87","updated":"2021-10-11 13:29:13.000000000","message":"Yepp, it\u0027s better to match the commit message with the actual version bump. :)","commit_id":"606f2243c7a6cd962785883942c6c743554d6e1c"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"7e50b3239a81eb4cc9d04839568a718eb150b226","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"36c878ee_2c70ed55","updated":"2021-10-13 12:28:37.000000000","message":"recheck","commit_id":"606f2243c7a6cd962785883942c6c743554d6e1c"}],"deliverables/_independent/tripleo-operator-ansible.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"c6adb80cbe0f27cbcc44c1c79ff588a49cbb7269","unresolved":true,"context_lines":[{"line_number":40,"context_line":"  - version: 0.6.1"},{"line_number":41,"context_line":"    projects:"},{"line_number":42,"context_line":"      - repo: openstack/tripleo-operator-ansible"},{"line_number":43,"context_line":"        hash: b0afe0267688a5a7d16128be68cd9b055f828772"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5fa778f4_53a0530b","line":43,"range":{"start_line":43,"start_character":0,"end_line":43,"end_character":4},"updated":"2021-10-07 13:55:37.000000000","message":"lgtm that is the topmost commit at https://github.com/openstack/tripleo-operator-ansible/commits/master right now\n\n12 commits since the last tag ... i think 0.6.1 is OK but you could probably also justify a 0.7.0 if you prefer it (looking especially at [1] and [2] for example)\n\nhttps://github.com/openstack/tripleo-operator-ansible/compare/d7be1b9a8c97dbcde44eaf65f38dfd9634bcea26...b0afe0267688a5a7d16128be68cd9b055f828772\n\n\n[1] https://github.com/openstack/tripleo-operator-ansible/commit/3e07830f0833c1c3aafb65f13913cd0f8677b7dc\n[2]  https://github.com/openstack/tripleo-operator-ansible/commit/d02a391f62264f98981627f15a07aa6a3598f300","commit_id":"271553cdd3a60e9e4ad5709efddc6aee1c8184e2"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"7975a0fbccf3084e4fbfa2598defc09650e0f4f6","unresolved":true,"context_lines":[{"line_number":40,"context_line":"  - version: 0.6.1"},{"line_number":41,"context_line":"    projects:"},{"line_number":42,"context_line":"      - repo: openstack/tripleo-operator-ansible"},{"line_number":43,"context_line":"        hash: b0afe0267688a5a7d16128be68cd9b055f828772"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b86a7399_e44096a6","line":43,"range":{"start_line":43,"start_character":0,"end_line":43,"end_character":4},"in_reply_to":"5fa778f4_53a0530b","updated":"2021-10-07 14:57:25.000000000","message":"I think it may be considered as minor changes, probably adding new operator would be a major one. But I don\u0027t have a strong opinion about that.","commit_id":"271553cdd3a60e9e4ad5709efddc6aee1c8184e2"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"4357e0ba99dc02df5c6695da651106f2c209c69f","unresolved":true,"context_lines":[{"line_number":40,"context_line":"  - version: 0.6.1"},{"line_number":41,"context_line":"    projects:"},{"line_number":42,"context_line":"      - repo: openstack/tripleo-operator-ansible"},{"line_number":43,"context_line":"        hash: b0afe0267688a5a7d16128be68cd9b055f828772"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9666f1ce_d615a7d1","line":43,"range":{"start_line":43,"start_character":0,"end_line":43,"end_character":4},"in_reply_to":"b86a7399_e44096a6","updated":"2021-10-11 10:06:32.000000000","message":"for the record, the main reason I added these comments is that sagi and I were on a call earlier and discussed how these releases are made and how you check the diff/compare between the different tags before posting the release review. I was mainly highlighting what we spoke about.","commit_id":"271553cdd3a60e9e4ad5709efddc6aee1c8184e2"}]}