)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"4d9f5f317d6d49eade0881f4f66d486375cd052a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0eea3704_01fa8f87","updated":"2024-02-13 07:12:43.000000000","message":"ping","commit_id":"5087b3300e8cf650eb02cf168dc10e4c79dcb2be"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"3e1ba8a6d5799edaee41ac81be61dc106aedf3cc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4819e731_3563d263","updated":"2024-02-09 14:21:32.000000000","message":"recheck non-related rocky fail","commit_id":"5087b3300e8cf650eb02cf168dc10e4c79dcb2be"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"ef0034125d484b26f71d61a5bed4e24be72bbae1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"afb549ab_53bd82e0","updated":"2024-03-01 12:21:30.000000000","message":"Again Ping Ping trivial, Thanks.","commit_id":"b961f184da4ea71043d0ab7c562dab67adebc7ff"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"3b33c24dec12e1945917c77625c2c3e7d157fd7d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d9fb4f3d_27ef7e83","updated":"2024-02-29 11:37:31.000000000","message":"Any reviewers ? This is really trivial bugfix...","commit_id":"b961f184da4ea71043d0ab7c562dab67adebc7ff"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"2d5fb6725d25dd171cf78f3f6bce14b4f0a6c46e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6278f40c_a7276f66","updated":"2024-02-20 12:36:23.000000000","message":"ping","commit_id":"b961f184da4ea71043d0ab7c562dab67adebc7ff"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"15b7b9e811444b7d0113c3beb0d3f8dfe3fe37e7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"41d95abe_efcf9ae6","updated":"2024-09-16 08:05:33.000000000","message":"please apply quoting consistently, rest of the code LGTM","commit_id":"161f300f0f254da0575acc58f5246d72f1959f2e"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"40337719aae706df465fc0bbcf20adab61ebc857","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"127c62c4_be4e7cfc","updated":"2024-10-02 16:22:08.000000000","message":"It\u0027s bash, so it\u0027s ugly and could be improved upon (there\u0027s stuff to nitpick for sure ;))\nThat being said it should do the job for 99% of the affected use cases. LGTM","commit_id":"18899a7d525bc4b4d2e547e78d4b7aa9ec9645a0"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"f3ab8484a12afeeab62664f40d7f385aa65faa6e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3559dbc7_5a983cf1","updated":"2024-09-16 09:38:09.000000000","message":"nitpick. quotes around constant string in shell unneeded, only necessary when variable used. anyway lgtm.","commit_id":"18899a7d525bc4b4d2e547e78d4b7aa9ec9645a0"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"42dfc93471c19cf4b01ff2c8a793bef52cb75d75","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"07c2bbaf_b916919c","updated":"2024-09-20 23:09:12.000000000","message":"ping","commit_id":"18899a7d525bc4b4d2e547e78d4b7aa9ec9645a0"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"2dea877e05ab2e93d1223f869efd599cfde815c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"5f93b54b_a422e8a9","in_reply_to":"127c62c4_be4e7cfc","updated":"2024-10-02 16:23:01.000000000","message":"thx","commit_id":"18899a7d525bc4b4d2e547e78d4b7aa9ec9645a0"}],"tools/cleanup-host":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"264e0f06e570b424df0cd8a6dc5bab9d66039df4","unresolved":true,"context_lines":[{"line_number":85,"context_line":"       [ \"$dir\" \u003d\u003d \"$FOLDER_PATH/certificates\" ]; then"},{"line_number":86,"context_line":"        echo \"Skipping:\" $dir"},{"line_number":87,"context_line":"    else"},{"line_number":88,"context_line":"        rm -rfv $dir"},{"line_number":89,"context_line":"        for inventory in $kolla_ansible_inventories; do"},{"line_number":90,"context_line":"            if [ \"$dir\" \u003d\u003d \"$inventory\" ]; then"},{"line_number":91,"context_line":"                echo \"Skipping: $dir\""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"145be5b5_46f787a3","line":88,"range":{"start_line":88,"start_character":8,"end_line":88,"end_character":20},"updated":"2024-03-01 14:37:18.000000000","message":"sorry, but doesn\u0027t this line delete the inventory file before you skip it below?\n\ne.g. `kolla_ansible_inventories` is `/etc/kolla/inventory` and `FOLDER_PATH\u003d\"/etc/kolla\"`\n\nthis deletes `/etc/kolla/inventory` before your secondary for loop even starts, or am I missing something?\n\nso this only works if inventories outside /etc/kolla/ are supplied by the user, but that is not what is in the commit message or in the bug report.\n\nMaybe I have a misunderstanding because the code is spread over 3 files and different languages, I never looked at how inventories are handled and this whole cleanup script looks rather messy (this is off topic of course).","commit_id":"b961f184da4ea71043d0ab7c562dab67adebc7ff"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"99f089f348a4ca4dba9a7b3b45a03037197b09fc","unresolved":false,"context_lines":[{"line_number":85,"context_line":"       [ \"$dir\" \u003d\u003d \"$FOLDER_PATH/certificates\" ]; then"},{"line_number":86,"context_line":"        echo \"Skipping:\" $dir"},{"line_number":87,"context_line":"    else"},{"line_number":88,"context_line":"        rm -rfv $dir"},{"line_number":89,"context_line":"        for inventory in $kolla_ansible_inventories; do"},{"line_number":90,"context_line":"            if [ \"$dir\" \u003d\u003d \"$inventory\" ]; then"},{"line_number":91,"context_line":"                echo \"Skipping: $dir\""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"0c301a64_79a544b3","line":88,"range":{"start_line":88,"start_character":8,"end_line":88,"end_character":20},"in_reply_to":"145be5b5_46f787a3","updated":"2024-09-13 19:37:37.000000000","message":"Yep, you were right... my fault 😊. Good catch. Logic is now little bit changed 😊. Now it works.","commit_id":"b961f184da4ea71043d0ab7c562dab67adebc7ff"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"15b7b9e811444b7d0113c3beb0d3f8dfe3fe37e7","unresolved":true,"context_lines":[{"line_number":77,"context_line":""},{"line_number":78,"context_line":"echo \"Getting folders name...\""},{"line_number":79,"context_line":"for dir in $FOLDER_PATH/*; do"},{"line_number":80,"context_line":"    skip\u003dfalse"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    for inventory in $kolla_ansible_inventories; do"},{"line_number":83,"context_line":"        if [ \"$dir\" \u003d\u003d \"$inventory\" ]; then"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"4932759e_db24152f","line":80,"range":{"start_line":80,"start_character":9,"end_line":80,"end_character":14},"updated":"2024-09-16 08:05:33.000000000","message":"```suggestion\n    skip\u003d\"false\"\n```","commit_id":"161f300f0f254da0575acc58f5246d72f1959f2e"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"7659b651d2e47e9aef3c780b2a1288eca62c2d70","unresolved":false,"context_lines":[{"line_number":77,"context_line":""},{"line_number":78,"context_line":"echo \"Getting folders name...\""},{"line_number":79,"context_line":"for dir in $FOLDER_PATH/*; do"},{"line_number":80,"context_line":"    skip\u003dfalse"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    for inventory in $kolla_ansible_inventories; do"},{"line_number":83,"context_line":"        if [ \"$dir\" \u003d\u003d \"$inventory\" ]; then"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"e47ef499_a00760c4","line":80,"range":{"start_line":80,"start_character":9,"end_line":80,"end_character":14},"in_reply_to":"4932759e_db24152f","updated":"2024-09-16 08:16:07.000000000","message":"Done","commit_id":"161f300f0f254da0575acc58f5246d72f1959f2e"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"15b7b9e811444b7d0113c3beb0d3f8dfe3fe37e7","unresolved":true,"context_lines":[{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    for inventory in $kolla_ansible_inventories; do"},{"line_number":83,"context_line":"        if [ \"$dir\" \u003d\u003d \"$inventory\" ]; then"},{"line_number":84,"context_line":"            skip\u003dtrue"},{"line_number":85,"context_line":"            break"},{"line_number":86,"context_line":"        fi"},{"line_number":87,"context_line":"    done"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"e2ebd36e_5568243f","line":84,"range":{"start_line":84,"start_character":17,"end_line":84,"end_character":21},"updated":"2024-09-16 08:05:33.000000000","message":"```suggestion\n            skip\u003d\"true\"\n```","commit_id":"161f300f0f254da0575acc58f5246d72f1959f2e"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"7659b651d2e47e9aef3c780b2a1288eca62c2d70","unresolved":false,"context_lines":[{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    for inventory in $kolla_ansible_inventories; do"},{"line_number":83,"context_line":"        if [ \"$dir\" \u003d\u003d \"$inventory\" ]; then"},{"line_number":84,"context_line":"            skip\u003dtrue"},{"line_number":85,"context_line":"            break"},{"line_number":86,"context_line":"        fi"},{"line_number":87,"context_line":"    done"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"94fc94c6_96ced58f","line":84,"range":{"start_line":84,"start_character":17,"end_line":84,"end_character":21},"in_reply_to":"e2ebd36e_5568243f","updated":"2024-09-16 08:16:07.000000000","message":"Done","commit_id":"161f300f0f254da0575acc58f5246d72f1959f2e"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"15b7b9e811444b7d0113c3beb0d3f8dfe3fe37e7","unresolved":true,"context_lines":[{"line_number":92,"context_line":"       [ \"$dir\" \u003d\u003d \"$FOLDER_PATH/kolla-build.conf\" ] || \\"},{"line_number":93,"context_line":"       [ \"$dir\" \u003d\u003d \"$FOLDER_PATH/config\" ] || \\"},{"line_number":94,"context_line":"       [ \"$dir\" \u003d\u003d \"$FOLDER_PATH/certificates\" ]; then"},{"line_number":95,"context_line":"        echo \"Skipping:\" $dir"},{"line_number":96,"context_line":"        skip\u003dtrue"},{"line_number":97,"context_line":"    fi"},{"line_number":98,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":3,"id":"34f3d2b4_74a82b25","line":95,"range":{"start_line":95,"start_character":26,"end_line":95,"end_character":29},"updated":"2024-09-16 08:05:33.000000000","message":"```suggestion\n        echo \"Skipping:\" \"$dir\"\n```","commit_id":"161f300f0f254da0575acc58f5246d72f1959f2e"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"7659b651d2e47e9aef3c780b2a1288eca62c2d70","unresolved":false,"context_lines":[{"line_number":92,"context_line":"       [ \"$dir\" \u003d\u003d \"$FOLDER_PATH/kolla-build.conf\" ] || \\"},{"line_number":93,"context_line":"       [ \"$dir\" \u003d\u003d \"$FOLDER_PATH/config\" ] || \\"},{"line_number":94,"context_line":"       [ \"$dir\" \u003d\u003d \"$FOLDER_PATH/certificates\" ]; then"},{"line_number":95,"context_line":"        echo \"Skipping:\" $dir"},{"line_number":96,"context_line":"        skip\u003dtrue"},{"line_number":97,"context_line":"    fi"},{"line_number":98,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":3,"id":"e016f154_1e9116d0","line":95,"range":{"start_line":95,"start_character":26,"end_line":95,"end_character":29},"in_reply_to":"34f3d2b4_74a82b25","updated":"2024-09-16 08:16:07.000000000","message":"Done","commit_id":"161f300f0f254da0575acc58f5246d72f1959f2e"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"15b7b9e811444b7d0113c3beb0d3f8dfe3fe37e7","unresolved":true,"context_lines":[{"line_number":93,"context_line":"       [ \"$dir\" \u003d\u003d \"$FOLDER_PATH/config\" ] || \\"},{"line_number":94,"context_line":"       [ \"$dir\" \u003d\u003d \"$FOLDER_PATH/certificates\" ]; then"},{"line_number":95,"context_line":"        echo \"Skipping:\" $dir"},{"line_number":96,"context_line":"        skip\u003dtrue"},{"line_number":97,"context_line":"    fi"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    # If it\u0027s not to be skipped, remove it"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"914be33b_5089152e","line":96,"range":{"start_line":96,"start_character":13,"end_line":96,"end_character":17},"updated":"2024-09-16 08:05:33.000000000","message":"```suggestion\n        skip\u003d\"true\"\n```","commit_id":"161f300f0f254da0575acc58f5246d72f1959f2e"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"7659b651d2e47e9aef3c780b2a1288eca62c2d70","unresolved":false,"context_lines":[{"line_number":93,"context_line":"       [ \"$dir\" \u003d\u003d \"$FOLDER_PATH/config\" ] || \\"},{"line_number":94,"context_line":"       [ \"$dir\" \u003d\u003d \"$FOLDER_PATH/certificates\" ]; then"},{"line_number":95,"context_line":"        echo \"Skipping:\" $dir"},{"line_number":96,"context_line":"        skip\u003dtrue"},{"line_number":97,"context_line":"    fi"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    # If it\u0027s not to be skipped, remove it"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"c8a2476e_6db6c142","line":96,"range":{"start_line":96,"start_character":13,"end_line":96,"end_character":17},"in_reply_to":"914be33b_5089152e","updated":"2024-09-16 08:16:07.000000000","message":"Done","commit_id":"161f300f0f254da0575acc58f5246d72f1959f2e"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"15b7b9e811444b7d0113c3beb0d3f8dfe3fe37e7","unresolved":true,"context_lines":[{"line_number":97,"context_line":"    fi"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    # If it\u0027s not to be skipped, remove it"},{"line_number":100,"context_line":"    if [ \"$skip\" \u003d\u003d false ]; then"},{"line_number":101,"context_line":"        rm -rfv $dir"},{"line_number":102,"context_line":"    fi"},{"line_number":103,"context_line":"done"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"39229790_1eb7753f","line":100,"range":{"start_line":100,"start_character":20,"end_line":100,"end_character":25},"updated":"2024-09-16 08:05:33.000000000","message":"```suggestion\n    if [ \"$skip\" \u003d\u003d \"false\" ]; then\n```","commit_id":"161f300f0f254da0575acc58f5246d72f1959f2e"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"7659b651d2e47e9aef3c780b2a1288eca62c2d70","unresolved":false,"context_lines":[{"line_number":97,"context_line":"    fi"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    # If it\u0027s not to be skipped, remove it"},{"line_number":100,"context_line":"    if [ \"$skip\" \u003d\u003d false ]; then"},{"line_number":101,"context_line":"        rm -rfv $dir"},{"line_number":102,"context_line":"    fi"},{"line_number":103,"context_line":"done"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"6b6d9a03_734ac503","line":100,"range":{"start_line":100,"start_character":20,"end_line":100,"end_character":25},"in_reply_to":"39229790_1eb7753f","updated":"2024-09-16 08:16:07.000000000","message":"Done","commit_id":"161f300f0f254da0575acc58f5246d72f1959f2e"}],"tools/kolla-ansible":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"e86e71651d4ba7ba57aad627d0da9d3b51d81ab1","unresolved":true,"context_lines":[{"line_number":469,"context_line":"        if [[ \"${INCLUDE_DEV}\" \u003d\u003d \"--include-dev\" ]]; then"},{"line_number":470,"context_line":"            EXTRA_OPTS\u003d\"$EXTRA_OPTS -e destroy_include_dev\u003dyes\""},{"line_number":471,"context_line":"        fi"},{"line_number":472,"context_line":"        EXTRA_OPTS\u003d\"$EXTRA_OPTS -e inventories_comma_separated\u003d${INVENTORIES_COMMA_DELIMITED}\""},{"line_number":473,"context_line":"        if [[ \"${DANGER_CONFIRM}\" !\u003d \"--yes-i-really-really-mean-it\" ]]; then"},{"line_number":474,"context_line":"            cat \u003c\u003c EOF"},{"line_number":475,"context_line":"WARNING:"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"bc94ced9_7e410cdb","line":472,"range":{"start_line":472,"start_character":35,"end_line":472,"end_character":62},"updated":"2024-02-15 12:45:44.000000000","message":"comma_separated, comma_delimited, can we decide which one? ;-)","commit_id":"5087b3300e8cf650eb02cf168dc10e4c79dcb2be"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"6e635ee0d9d673d340d97c2635f473096c30edd7","unresolved":false,"context_lines":[{"line_number":469,"context_line":"        if [[ \"${INCLUDE_DEV}\" \u003d\u003d \"--include-dev\" ]]; then"},{"line_number":470,"context_line":"            EXTRA_OPTS\u003d\"$EXTRA_OPTS -e destroy_include_dev\u003dyes\""},{"line_number":471,"context_line":"        fi"},{"line_number":472,"context_line":"        EXTRA_OPTS\u003d\"$EXTRA_OPTS -e inventories_comma_separated\u003d${INVENTORIES_COMMA_DELIMITED}\""},{"line_number":473,"context_line":"        if [[ \"${DANGER_CONFIRM}\" !\u003d \"--yes-i-really-really-mean-it\" ]]; then"},{"line_number":474,"context_line":"            cat \u003c\u003c EOF"},{"line_number":475,"context_line":"WARNING:"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"c4899dae_4b586e03","line":472,"range":{"start_line":472,"start_character":35,"end_line":472,"end_character":62},"in_reply_to":"b24a1ea7_9e3a1f06","updated":"2024-02-19 12:47:29.000000000","message":"Done","commit_id":"5087b3300e8cf650eb02cf168dc10e4c79dcb2be"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"f5f0e0f3c84f82f50bfd0b9a82c07ef675d7f8a8","unresolved":true,"context_lines":[{"line_number":469,"context_line":"        if [[ \"${INCLUDE_DEV}\" \u003d\u003d \"--include-dev\" ]]; then"},{"line_number":470,"context_line":"            EXTRA_OPTS\u003d\"$EXTRA_OPTS -e destroy_include_dev\u003dyes\""},{"line_number":471,"context_line":"        fi"},{"line_number":472,"context_line":"        EXTRA_OPTS\u003d\"$EXTRA_OPTS -e inventories_comma_separated\u003d${INVENTORIES_COMMA_DELIMITED}\""},{"line_number":473,"context_line":"        if [[ \"${DANGER_CONFIRM}\" !\u003d \"--yes-i-really-really-mean-it\" ]]; then"},{"line_number":474,"context_line":"            cat \u003c\u003c EOF"},{"line_number":475,"context_line":"WARNING:"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"b24a1ea7_9e3a1f06","line":472,"range":{"start_line":472,"start_character":35,"end_line":472,"end_character":62},"in_reply_to":"bc94ced9_7e410cdb","updated":"2024-02-15 12:56:57.000000000","message":"In each of us, there are several versions of ourselves sleeping. Sure i will.","commit_id":"5087b3300e8cf650eb02cf168dc10e4c79dcb2be"}]}