)]}'
{"sonobuoy/templates/bin/_publish_results.sh.tpl":[{"author":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"change_message_id":"218c1aa4cda558dd73297d81841c6df5993e2ad1","unresolved":false,"context_lines":[{"line_number":34,"context_line":"swift post {{ .Values.conf.swift.container_name }} $prefixed_file_name -H \\\"X-Delete-After:{{ .Values.conf.swift.delete_objects_after_seconds }}\\\""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"# NOTE(aw442m): Delete results after publishing to avoid collisions when using host path"},{"line_number":37,"context_line":"rm -rf \"${RESULTS_DIR}\"/*"}],"source_content_type":"text/x-smarty","patch_set":5,"id":"9fb8cfa7_f7eabe0c","line":37,"updated":"2019-06-28 12:14:14.000000000","message":"I\u0027m a little hesitant with this. Anytime I see a `rm -rf VARIABLE/*` I worry what would happen if VARIABLE was empty.\n\nI think we can change this to:\nrm \"${RESULTS_DIR}\"/\"${file_name}\"","commit_id":"b09be5ce0784213e45286013bbe9b36f5e03d71a"},{"author":{"_account_id":28618,"name":"Drew Walters","email":"drewwalters@microsoft.com","username":"drewwalters96"},"change_message_id":"b25fa733434ab5811507de4da274ebe3be97d484","unresolved":false,"context_lines":[{"line_number":34,"context_line":"swift post {{ .Values.conf.swift.container_name }} $prefixed_file_name -H \\\"X-Delete-After:{{ .Values.conf.swift.delete_objects_after_seconds }}\\\""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"# NOTE(aw442m): Delete results after publishing to avoid collisions when using host path"},{"line_number":37,"context_line":"rm -rf \"${RESULTS_DIR}\"/*"}],"source_content_type":"text/x-smarty","patch_set":5,"id":"9fb8cfa7_85f6ac8e","line":37,"in_reply_to":"9fb8cfa7_f7eabe0c","updated":"2019-06-28 14:48:50.000000000","message":"\u003e I\u0027m a little hesitant with this. Anytime I see a `rm -rf\n \u003e VARIABLE/*` I worry what would happen if VARIABLE was empty.\n \u003e \n \u003e I think we can change this to:\n \u003e rm \"${RESULTS_DIR}\"/\"${file_name}\"\n\nDone.","commit_id":"b09be5ce0784213e45286013bbe9b36f5e03d71a"},{"author":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"change_message_id":"900174752c7a8cfe6b2b2a16d888e88f06b29f5f","unresolved":false,"context_lines":[{"line_number":34,"context_line":"swift post {{ .Values.conf.swift.container_name }} $prefixed_file_name -H \\\"X-Delete-After:{{ .Values.conf.swift.delete_objects_after_seconds }}\\\""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"# NOTE(aw442m): Delete results after publishing to avoid collisions when using host path"},{"line_number":37,"context_line":"if [[ ! -z \"${RESULTS_DIR}\" ]]; then"},{"line_number":38,"context_line":"  rm \"${RESULTS_DIR}\"/\"${file_name}\""},{"line_number":39,"context_line":"fi"}],"source_content_type":"text/x-smarty","patch_set":7,"id":"7faddb67_1fff0a02","line":37,"updated":"2019-07-08 13:45:09.000000000","message":"I\u0027m not opposed to this check, but I\u0027m curious how this situation happened. I\u0027m not understanding how this if check would prevent doing an rm. I thought the file would always exist at this point.","commit_id":"32888e88fa91da4d181a5c380157962758a2002c"}]}