)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35024},"change_message_id":"23c42c73ec709b910c1fc147ab11d3bf49b620e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ad26388a_e547ff86","updated":"2022-08-01 14:06:22.000000000","message":"Dependent change passed lint.","commit_id":"6b0252c8e9d4be974391fd3694125bbb83bec31a"},{"author":{"_account_id":35024},"change_message_id":"fe95cdcc50444cb61be914a05b8d4bdaad0b2a23","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"62935f74_af160ed7","updated":"2022-08-03 18:07:44.000000000","message":"I\u0027ve got another lint change I\u0027m working on but this can stand on its own.","commit_id":"6b0252c8e9d4be974391fd3694125bbb83bec31a"},{"author":{"_account_id":35024},"change_message_id":"8d0585d036cbe7fec652ce3777b60901ce0e3d1f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"272b41ce_e36fe120","updated":"2022-07-22 20:26:15.000000000","message":"Should make sure https://review.opendev.org/c/openstack/openstack-helm/+/850776 test patch works before merging","commit_id":"6b0252c8e9d4be974391fd3694125bbb83bec31a"}],"tools/gate/lint.sh":[{"author":{"_account_id":31728,"name":"Oleksii Shcherba","display_name":"Oleksii Shcherba","email":"v-oshcherba@microsoft.com","username":"mihalych"},"change_message_id":"d77497e039f3ad3e1b0639a9ea316fb14cd5b8d4","unresolved":true,"context_lines":[{"line_number":21,"context_line":"    ln -s ../../../openstack-helm-infra/helm-toolkit $i/charts/helm-toolkit"},{"line_number":22,"context_line":"    ${LINT_DIR}/helm template $i --output-dir ${LINT_DIR} 2\u003e\u00261 \u003e /dev/null"},{"line_number":23,"context_line":"done"},{"line_number":24,"context_line":"rm -rf */charts/helm-toolkit"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"find .yamllint -type f -exec sed -i \u0027s/%%%.*/XXX/g\u0027 {} +"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"fb400044_2c7a186d","side":"PARENT","line":24,"updated":"2022-08-03 19:48:50.000000000","message":"Was it deleted to avoid duplication check?","commit_id":"381d066f010a759ef223cd4b26b9d9ae745a7542"},{"author":{"_account_id":35024},"change_message_id":"64d9fca717f0c3f290edb8b32034afcff7d3de37","unresolved":true,"context_lines":[{"line_number":21,"context_line":"    ln -s ../../../openstack-helm-infra/helm-toolkit $i/charts/helm-toolkit"},{"line_number":22,"context_line":"    ${LINT_DIR}/helm template $i --output-dir ${LINT_DIR} 2\u003e\u00261 \u003e /dev/null"},{"line_number":23,"context_line":"done"},{"line_number":24,"context_line":"rm -rf */charts/helm-toolkit"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"find .yamllint -type f -exec sed -i \u0027s/%%%.*/XXX/g\u0027 {} +"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"dbec0e05_37689441","side":"PARENT","line":24,"in_reply_to":"fb400044_2c7a186d","updated":"2022-08-03 19:58:39.000000000","message":"I don\u0027t know, and I don\u0027t know why we do this rm twice - here and on L10, but this is the one that causes the issue. Both were in the original change that created this file.\n\nAnd in case it wasn\u0027t obvious, this causes a problem in openstack-helm and not here, that\u0027s where the symlink exists and gets removed.","commit_id":"381d066f010a759ef223cd4b26b9d9ae745a7542"},{"author":{"_account_id":31728,"name":"Oleksii Shcherba","display_name":"Oleksii Shcherba","email":"v-oshcherba@microsoft.com","username":"mihalych"},"change_message_id":"d77497e039f3ad3e1b0639a9ea316fb14cd5b8d4","unresolved":true,"context_lines":[{"line_number":18,"context_line":"    # avoid helm-toolkit to symlink on itself"},{"line_number":19,"context_line":"    [ -d \"$i/templates\" -a \"$i\" !\u003d \"helm-toolkit/\" ] || continue"},{"line_number":20,"context_line":"    mkdir -p $i/charts"},{"line_number":21,"context_line":"    ln -s ../../../openstack-helm-infra/helm-toolkit $i/charts/helm-toolkit"},{"line_number":22,"context_line":"    ${LINT_DIR}/helm template $i --output-dir ${LINT_DIR} 2\u003e\u00261 \u003e /dev/null"},{"line_number":23,"context_line":"done"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"1a9d0e2e_6d958622","line":21,"updated":"2022-08-03 19:48:50.000000000","message":"how these symlinks would be deleted without following rm command?","commit_id":"6b0252c8e9d4be974391fd3694125bbb83bec31a"},{"author":{"_account_id":35024},"change_message_id":"f179e770bca4395038a2ab927d70ca01dab30a9e","unresolved":true,"context_lines":[{"line_number":18,"context_line":"    # avoid helm-toolkit to symlink on itself"},{"line_number":19,"context_line":"    [ -d \"$i/templates\" -a \"$i\" !\u003d \"helm-toolkit/\" ] || continue"},{"line_number":20,"context_line":"    mkdir -p $i/charts"},{"line_number":21,"context_line":"    ln -s ../../../openstack-helm-infra/helm-toolkit $i/charts/helm-toolkit"},{"line_number":22,"context_line":"    ${LINT_DIR}/helm template $i --output-dir ${LINT_DIR} 2\u003e\u00261 \u003e /dev/null"},{"line_number":23,"context_line":"done"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"087e77b3_b5cc80fc","line":21,"in_reply_to":"1a9d0e2e_6d958622","updated":"2022-08-03 20:04:37.000000000","message":"Ah, yes I didn\u0027t pick-up on that. I\u0027ll need to think about this one again.","commit_id":"6b0252c8e9d4be974391fd3694125bbb83bec31a"}]}