)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"2c90d6ef07cbe1ed252528881f1ddb5c0025e750","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Jan Gutter \u003cgithub@jangutter.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-09-26 21:48:12 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"zun: Deprecate Zun provisionally"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"* Zun currently has a dependency on a combination of Docker and"},{"line_number":10,"context_line":"  etcd that is no longer installable (or supported) in newer"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3e9c52bb_2e8d5e42","line":7,"range":{"start_line":7,"start_character":5,"end_line":7,"end_character":14},"updated":"2023-09-27 13:26:03.000000000","message":"disable, not deprecate","commit_id":"f8d4db54db8f2ab2c7f99f0fcbc456177d33ddad"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"559faaac0c28aa3d13e9a6704863d68057a6bfe2","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Jan Gutter \u003cgithub@jangutter.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-09-26 21:48:12 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"zun: Deprecate Zun provisionally"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"* Zun currently has a dependency on a combination of Docker and"},{"line_number":10,"context_line":"  etcd that is no longer installable (or supported) in newer"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"00225c23_57f9ddfd","line":7,"range":{"start_line":7,"start_character":5,"end_line":7,"end_character":14},"in_reply_to":"3e9c52bb_2e8d5e42","updated":"2023-09-27 14:06:19.000000000","message":"where is this disabled? it is failing the precheck, but you can easily ignore that","commit_id":"f8d4db54db8f2ab2c7f99f0fcbc456177d33ddad"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"2c90d6ef07cbe1ed252528881f1ddb5c0025e750","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"991995b2_8a5cb11b","updated":"2023-09-27 13:26:03.000000000","message":"apart from the wording LGTM.","commit_id":"f8d4db54db8f2ab2c7f99f0fcbc456177d33ddad"}],"ansible/roles/prechecks/tasks/service_checks.yml":[{"author":{"_account_id":34579,"name":"Juan Pablo Suazo","email":"jsuazo@whitestack.com","username":"jsuazo"},"change_message_id":"10ac423e9742fafe8ac493590e9b3fb66e673973","unresolved":true,"context_lines":[{"line_number":67,"context_line":"  fail:"},{"line_number":68,"context_line":"    msg: |"},{"line_number":69,"context_line":"      Apologies, Zun is currently not supported for this release."},{"line_number":70,"context_line":"      Please consult the release notes."},{"line_number":71,"context_line":"  when:"},{"line_number":72,"context_line":"    - (enable_zun | default()) | bool"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"50f09bc1_766a7f49","line":70,"range":{"start_line":70,"start_character":25,"end_line":70,"end_character":38},"updated":"2023-09-27 13:27:08.000000000","message":"Maybe add a reference to the specific release note so its easier to find.","commit_id":"f8d4db54db8f2ab2c7f99f0fcbc456177d33ddad"},{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"c58d35d60eb743e8c517bdb23cd7150263a83060","unresolved":true,"context_lines":[{"line_number":67,"context_line":"  fail:"},{"line_number":68,"context_line":"    msg: |"},{"line_number":69,"context_line":"      Apologies, Zun is currently not supported for this release."},{"line_number":70,"context_line":"      Please consult the release notes."},{"line_number":71,"context_line":"  when:"},{"line_number":72,"context_line":"    - (enable_zun | default()) | bool"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"51decf3e_3ee53566","line":70,"range":{"start_line":70,"start_character":25,"end_line":70,"end_character":38},"in_reply_to":"50f09bc1_766a7f49","updated":"2023-09-27 13:31:06.000000000","message":"I\u0027ll see if I can find a way to make a stable link, if you have an idea that would be awesome :-p","commit_id":"f8d4db54db8f2ab2c7f99f0fcbc456177d33ddad"},{"author":{"_account_id":34579,"name":"Juan Pablo Suazo","email":"jsuazo@whitestack.com","username":"jsuazo"},"change_message_id":"09c9aa91f75b6b94c279c1f9e525b64b34434e1b","unresolved":true,"context_lines":[{"line_number":67,"context_line":"  fail:"},{"line_number":68,"context_line":"    msg: |"},{"line_number":69,"context_line":"      Apologies, Zun is currently not supported for this release."},{"line_number":70,"context_line":"      Please consult the release notes."},{"line_number":71,"context_line":"  when:"},{"line_number":72,"context_line":"    - (enable_zun | default()) | bool"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7b3b234d_06060958","line":70,"range":{"start_line":70,"start_character":25,"end_line":70,"end_character":38},"in_reply_to":"51decf3e_3ee53566","updated":"2023-09-27 13:46:01.000000000","message":"I was thinking more along the lines of mentioning the release note \"uuid\" 1db1295f733ec3fe so its easier to search for with a `find` command. Or add the path, but that seems messier.\n\nEither that or being a bit more detailed on the fail message. Adding a short line explaining why its disabled would remove the need to see the notes.","commit_id":"f8d4db54db8f2ab2c7f99f0fcbc456177d33ddad"},{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"bf55c270805158b6288c374d65f95fe495019ba4","unresolved":true,"context_lines":[{"line_number":67,"context_line":"  fail:"},{"line_number":68,"context_line":"    msg: |"},{"line_number":69,"context_line":"      Apologies, Zun is currently not supported for this release."},{"line_number":70,"context_line":"      Please consult the release notes."},{"line_number":71,"context_line":"  when:"},{"line_number":72,"context_line":"    - (enable_zun | default()) | bool"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"dc3abb94_09947a8a","line":70,"range":{"start_line":70,"start_character":25,"end_line":70,"end_character":38},"in_reply_to":"7b3b234d_06060958","updated":"2023-09-27 13:56:04.000000000","message":"Now that it\u0027s merging, there\u0027s going to be a stable link. I\u0027ll send up a followup cleaning up the language.","commit_id":"f8d4db54db8f2ab2c7f99f0fcbc456177d33ddad"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"559faaac0c28aa3d13e9a6704863d68057a6bfe2","unresolved":true,"context_lines":[{"line_number":67,"context_line":"  fail:"},{"line_number":68,"context_line":"    msg: |"},{"line_number":69,"context_line":"      Apologies, Zun is currently not supported for this release."},{"line_number":70,"context_line":"      Please consult the release notes."},{"line_number":71,"context_line":"  when:"},{"line_number":72,"context_line":"    - (enable_zun | default()) | bool"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"39770184_cbc2ca73","line":70,"range":{"start_line":70,"start_character":25,"end_line":70,"end_character":38},"in_reply_to":"dc3abb94_09947a8a","updated":"2023-09-27 14:06:19.000000000","message":"I think the general reference is good enough","commit_id":"f8d4db54db8f2ab2c7f99f0fcbc456177d33ddad"}],"releasenotes/notes/zun-provisional-deprecation-1db1295f733ec3fe.yaml":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"2c90d6ef07cbe1ed252528881f1ddb5c0025e750","unresolved":true,"context_lines":[{"line_number":10,"context_line":"    requirement. Please also consult the deprecation notes."},{"line_number":11,"context_line":"deprecations:"},{"line_number":12,"context_line":"  - |"},{"line_number":13,"context_line":"    Zun is currently provisionally deprecated but not removed. If Zun regains"},{"line_number":14,"context_line":"    compatibility within the next release cycle, backports to this version of"},{"line_number":15,"context_line":"    Kolla and Kolla-Ansible will be considered to provide a smooth upgrade"},{"line_number":16,"context_line":"    path."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5bb838fa_ee631330","line":13,"range":{"start_line":13,"start_character":35,"end_line":13,"end_character":45},"updated":"2023-09-27 13:26:03.000000000","message":"it\u0027s not deprecated but disabled, a deprecated item can still be used - most of the time with a warning, a disabled item can not be used.\n\nthat\u0027s a difference 😊","commit_id":"f8d4db54db8f2ab2c7f99f0fcbc456177d33ddad"},{"author":{"_account_id":25733,"name":"Jan Gutter","email":"github@jangutter.com","username":"jangutter"},"change_message_id":"c58d35d60eb743e8c517bdb23cd7150263a83060","unresolved":true,"context_lines":[{"line_number":10,"context_line":"    requirement. Please also consult the deprecation notes."},{"line_number":11,"context_line":"deprecations:"},{"line_number":12,"context_line":"  - |"},{"line_number":13,"context_line":"    Zun is currently provisionally deprecated but not removed. If Zun regains"},{"line_number":14,"context_line":"    compatibility within the next release cycle, backports to this version of"},{"line_number":15,"context_line":"    Kolla and Kolla-Ansible will be considered to provide a smooth upgrade"},{"line_number":16,"context_line":"    path."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"a7bf770b_242bbddd","line":13,"range":{"start_line":13,"start_character":35,"end_line":13,"end_character":45},"in_reply_to":"5bb838fa_ee631330","updated":"2023-09-27 13:31:06.000000000","message":"Thanks, will make the change, gathering a couple of comments and will respin tomorrow.","commit_id":"f8d4db54db8f2ab2c7f99f0fcbc456177d33ddad"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"559faaac0c28aa3d13e9a6704863d68057a6bfe2","unresolved":true,"context_lines":[{"line_number":10,"context_line":"    requirement. Please also consult the deprecation notes."},{"line_number":11,"context_line":"deprecations:"},{"line_number":12,"context_line":"  - |"},{"line_number":13,"context_line":"    Zun is currently provisionally deprecated but not removed. If Zun regains"},{"line_number":14,"context_line":"    compatibility within the next release cycle, backports to this version of"},{"line_number":15,"context_line":"    Kolla and Kolla-Ansible will be considered to provide a smooth upgrade"},{"line_number":16,"context_line":"    path."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3b35aecc_23cf6c3f","line":13,"range":{"start_line":13,"start_character":35,"end_line":13,"end_character":45},"in_reply_to":"a7bf770b_242bbddd","updated":"2023-09-27 14:06:19.000000000","message":"if it wasn\u0027t deprecated, it wouldn\u0027t fit the deprecation section. I would prefer to stick to this wording","commit_id":"f8d4db54db8f2ab2c7f99f0fcbc456177d33ddad"}]}
