)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"c2bae8d1ee7a33e56d3527c725eca099521e6827","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c706864b_39f2feef","updated":"2023-08-16 07:04:01.000000000","message":"Matt - is this going forward or should we abandon this?","commit_id":"19cf80b0f970e72b7d6dbb33ccb55ec9a9b23d74"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"1d5577aaeb5a9dc0598499f5eb3b7240bcc28dd8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"259233b4_10ff44d6","updated":"2023-04-24 16:05:01.000000000","message":"Thanks 😊","commit_id":"19cf80b0f970e72b7d6dbb33ccb55ec9a9b23d74"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"6b8883bf727563f51a0fb404d60c97f922af8d15","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9b9dab05_6d5eef34","updated":"2023-04-25 11:07:19.000000000","message":"The problem with backporting this is that we don\u0027t know whether the image kolla-ansible is using has this patch or not.","commit_id":"19cf80b0f970e72b7d6dbb33ccb55ec9a9b23d74"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"dab14ed73342d0abdeafb75d1cb0b69d9baed5b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f4d8cf3f_15b179d5","in_reply_to":"62b0e9a6_53f23620","updated":"2023-08-22 09:22:59.000000000","message":"Done","commit_id":"19cf80b0f970e72b7d6dbb33ccb55ec9a9b23d74"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"4da6bcca5364ab72b58a38a6d84a507ba1a4c517","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"62b0e9a6_53f23620","in_reply_to":"9b9dab05_6d5eef34","updated":"2023-05-09 09:12:08.000000000","message":"Should we add a label to the image and check for it in K-A?","commit_id":"19cf80b0f970e72b7d6dbb33ccb55ec9a9b23d74"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"a2b0e85f36e0134117d61e3f3b37c083cdfe335f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"eaf12fcc_3a503ab6","in_reply_to":"c706864b_39f2feef","updated":"2023-08-16 08:13:40.000000000","message":"I think this should still go forward.","commit_id":"19cf80b0f970e72b7d6dbb33ccb55ec9a9b23d74"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"4df595b7a502cbfce58154ead6dc83a6b0389b81","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6cd1400f_9d76a89a","updated":"2023-08-22 09:48:56.000000000","message":"this is not cherry picked from commit 48b5980c6352ad4e5cf0ac6da369140910c8f389","commit_id":"3e2afe2f6ecebaa6bd45fcedc25663c7c3780872"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"ebc7951d7636431c78647d43b41e5fab47a2bbbd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"03891db2_fb222935","in_reply_to":"1c3d4280_397b87a1","updated":"2023-08-26 21:15:22.000000000","message":"Ack","commit_id":"3e2afe2f6ecebaa6bd45fcedc25663c7c3780872"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"e19a6657d9807743d774dab25e78261727087428","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1c3d4280_397b87a1","in_reply_to":"6cd1400f_9d76a89a","updated":"2023-08-22 09:57:35.000000000","message":"The behaviour in extend_start was cherry-picked. But the backport needs an additional label so we can confirm that the image used in a given KA deployment containers this behaviour.\n\nImo this would still class as a cherry-pick, but I\u0027ll accept if this is not the case.","commit_id":"3e2afe2f6ecebaa6bd45fcedc25663c7c3780872"}],"docker/nova/nova-api/Dockerfile.j2":[{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"4df595b7a502cbfce58154ead6dc83a6b0389b81","unresolved":true,"context_lines":[{"line_number":1,"context_line":"FROM {{ namespace }}/{{ image_prefix }}nova-base:{{ tag }}"},{"line_number":2,"context_line":"{% block labels %}"},{"line_number":3,"context_line":"LABEL maintainer\u003d\"{{ maintainer }}\" name\u003d\"{{ image_name }}\" build-date\u003d\"{{ build_date }}\" upgrade_check\u003dtrue"},{"line_number":4,"context_line":"{% endblock %}"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"{% block nova_api_header %}{% endblock %}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"a279b789_ada4cfa9","line":3,"range":{"start_line":3,"start_character":90,"end_line":3,"end_character":108},"updated":"2023-08-22 09:48:56.000000000","message":"not certain how this would set KOLLA_UPGRADE_CHECK environment variable.","commit_id":"3e2afe2f6ecebaa6bd45fcedc25663c7c3780872"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"e19a6657d9807743d774dab25e78261727087428","unresolved":true,"context_lines":[{"line_number":1,"context_line":"FROM {{ namespace }}/{{ image_prefix }}nova-base:{{ tag }}"},{"line_number":2,"context_line":"{% block labels %}"},{"line_number":3,"context_line":"LABEL maintainer\u003d\"{{ maintainer }}\" name\u003d\"{{ image_name }}\" build-date\u003d\"{{ build_date }}\" upgrade_check\u003dtrue"},{"line_number":4,"context_line":"{% endblock %}"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"{% block nova_api_header %}{% endblock %}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"da4c1844_0ad1f130","line":3,"range":{"start_line":3,"start_character":90,"end_line":3,"end_character":108},"in_reply_to":"a279b789_ada4cfa9","updated":"2023-08-22 09:57:35.000000000","message":"This doesn\u0027t set the variable, it is set in the KA patch. It is just a label so that we can check for the presence of the new extend_start behaviour in Kolla-Ansible. Solution was suggested in this comment chain: https://review.opendev.org/c/openstack/kolla/+/881328/comments/9b9dab05_6d5eef34","commit_id":"3e2afe2f6ecebaa6bd45fcedc25663c7c3780872"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"b02e5ba18cea16d7355aa26d7edea4f999f42b32","unresolved":true,"context_lines":[{"line_number":1,"context_line":"FROM {{ namespace }}/{{ image_prefix }}nova-base:{{ tag }}"},{"line_number":2,"context_line":"{% block labels %}"},{"line_number":3,"context_line":"LABEL maintainer\u003d\"{{ maintainer }}\" name\u003d\"{{ image_name }}\" build-date\u003d\"{{ build_date }}\" upgrade_check\u003dtrue"},{"line_number":4,"context_line":"{% endblock %}"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"{% block nova_api_header %}{% endblock %}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"4bbac29a_760c3c17","line":3,"range":{"start_line":3,"start_character":90,"end_line":3,"end_character":108},"in_reply_to":"da4c1844_0ad1f130","updated":"2023-08-22 10:09:14.000000000","message":"I see. Thanks. We should document this here in K-A as well.","commit_id":"3e2afe2f6ecebaa6bd45fcedc25663c7c3780872"}]}