)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"b1f2b752b2ecb52236c94fa3028903c526baa249","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7d858ce1_fb672a64","updated":"2021-11-23 12:08:55.000000000","message":"How does that work with other distributions or build types than binary?","commit_id":"051dcbe5a23f65c5827f41f2246671bb857c76a2"},{"author":{"_account_id":28752,"name":"Satish Patel","email":"satish.txt@gmail.com","username":"spatel"},"change_message_id":"8e58f4c9fa1fb0591c1e44d9d4918d2b0bca9a8e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"535a2afc_dd5a2aea","updated":"2022-03-08 17:15:58.000000000","message":"I had this bug in manila where default policy.json file cause issue. I would like to this patch to get commit :) ","commit_id":"051dcbe5a23f65c5827f41f2246671bb857c76a2"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"0e4ab981bad71ccd3981c05dfdc929d6d3d2ee18","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e943b361_8855dbcc","in_reply_to":"535a2afc_dd5a2aea","updated":"2022-03-08 19:23:11.000000000","message":"Please use \"source\" images. The name is misleading but \"source\" images are well-supported by us and \"binary\" images will be gone in a cycle.","commit_id":"051dcbe5a23f65c5827f41f2246671bb857c76a2"}],"docker/glance/glance-base/Dockerfile.j2":[{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"b1f2b752b2ecb52236c94fa3028903c526baa249","unresolved":true,"context_lines":[{"line_number":23,"context_line":"    {% elif base_package_type \u003d\u003d \u0027deb\u0027 %}"},{"line_number":24,"context_line":"        {% set glance_base_packages \u003d ["},{"line_number":25,"context_line":"            \u0027glance\u0027,"},{"line_number":26,"context_line":"            \u0027glance-common\u0027,"},{"line_number":27,"context_line":"            \u0027python3-boto3\u0027,"},{"line_number":28,"context_line":"            \u0027python3-os-brick\u0027,"},{"line_number":29,"context_line":"            \u0027python3-oslo.vmware\u0027,"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"d85f8666_8ab7c1fb","line":26,"range":{"start_line":26,"start_character":12,"end_line":26,"end_character":28},"updated":"2021-11-23 12:08:55.000000000","message":"this is dependency of glance -\u003e glance-api -\u003e glance-common so does not need to be listed","commit_id":"051dcbe5a23f65c5827f41f2246671bb857c76a2"}],"docker/macros.j2":[{"author":{"_account_id":32398,"name":"Gaël THEROND","display_name":"Fl1nt","email":"gael.therond@bitswalk.com","username":"Fl1nt"},"change_message_id":"c9b0cec0815a8907289c77c33b732a3f2abc2fa0","unresolved":true,"context_lines":[{"line_number":86,"context_line":"{% endmacro %}"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"{% macro default_policy_cleanup(path) -%}"},{"line_number":89,"context_line":"    RUN rm -f {{ path }}/policy.d/* \u0026\u0026 rm -f {{ path }}/policy.yaml \u0026\u0026 rm -f {{ path }}/policy.json"},{"line_number":90,"context_line":"{%- endmacro %}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"71c2b6af_302714da","line":89,"updated":"2021-10-05 11:39:32.000000000","message":"Shouldn\u0027t we rather have a conditional check for DEBIAN and the remove kolla opiniated folder?","commit_id":"cd94d386e950588f07b9da9de526af2e8a22d660"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"eebf3b4e24cc5c856635c0476035dda3b6a80bbd","unresolved":true,"context_lines":[{"line_number":86,"context_line":"{% endmacro %}"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"{% macro default_policy_cleanup(path) -%}"},{"line_number":89,"context_line":"    RUN rm -f {{ path }}/policy.d/* \u0026\u0026 rm -f {{ path }}/policy.yaml \u0026\u0026 rm -f {{ path }}/policy.json"},{"line_number":90,"context_line":"{%- endmacro %}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"40a59108_8721ffac","line":89,"in_reply_to":"71c2b6af_302714da","updated":"2021-10-06 10:38:52.000000000","message":"Well, macro is as simple as it can be, rm bash cmd using -f argument so it will never fail. Moreover, if any distro implement policy.d also (which is default dir name in upstream btw), this macro will work always.\n\nYeah, it can be conditioned, but is it really necessary ? I don\u0027t think so ..","commit_id":"cd94d386e950588f07b9da9de526af2e8a22d660"}],"docker/neutron/neutron-base/Dockerfile.j2":[{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"b1f2b752b2ecb52236c94fa3028903c526baa249","unresolved":true,"context_lines":[{"line_number":36,"context_line":"            \u0027net-tools\u0027,"},{"line_number":37,"context_line":"            \u0027radvd\u0027,"},{"line_number":38,"context_line":"            \u0027neutron-plugin-ml2\u0027,"},{"line_number":39,"context_line":"            \u0027neutron-common\u0027,"},{"line_number":40,"context_line":"            \u0027neutron-server\u0027,"},{"line_number":41,"context_line":"            \u0027openvswitch-switch\u0027,"},{"line_number":42,"context_line":"            \u0027python3-networking-sfc\u0027,"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"4c348b15_f4602ad4","line":39,"range":{"start_line":39,"start_character":6,"end_line":39,"end_character":29},"updated":"2021-11-23 12:08:55.000000000","message":"Another not needed dependency: neutron-server -\u003e neutron-api -\u003e neutron-common","commit_id":"051dcbe5a23f65c5827f41f2246671bb857c76a2"}]}