)]}'
{"functions.sh":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"0fc536859e6ea90382e102c829d9e02c7050d8d1","unresolved":false,"context_lines":[{"line_number":904,"context_line":"    # Copy Ironic nodes console logs if they exist"},{"line_number":905,"context_line":"    if [ -d $BASE/new/ironic-bm-logs ] ; then"},{"line_number":906,"context_line":"        sudo mkdir -p $BASE/logs/ironic-bm-logs"},{"line_number":907,"context_line":"        sudo cp $BASE/new/ironic-bm-logs/*.log $BASE/logs/ironic-bm-logs/"},{"line_number":908,"context_line":""},{"line_number":909,"context_line":"        # Copy the deployment logs if present"},{"line_number":910,"context_line":"        if [ -d $BASE/new/ironic-bm-logs/deploy_logs ]; then"}],"source_content_type":"text/x-sh","patch_set":1,"id":"fa7ab95a_898a6b62","line":907,"updated":"2016-08-30 19:45:46.000000000","message":"is there really that much not-log stuff in there that we don\u0027t want copied?\n\ni.e could you just copy the whole directory and then not need the special case below","commit_id":"1376687ca9d60cf641290f7a977fc4c0abac3593"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"4efb880b9952d022050c949d8a9f07fda0e16c33","unresolved":false,"context_lines":[{"line_number":904,"context_line":"    # Copy Ironic nodes console logs if they exist"},{"line_number":905,"context_line":"    if [ -d $BASE/new/ironic-bm-logs ] ; then"},{"line_number":906,"context_line":"        sudo mkdir -p $BASE/logs/ironic-bm-logs"},{"line_number":907,"context_line":"        sudo cp $BASE/new/ironic-bm-logs/*.log $BASE/logs/ironic-bm-logs/"},{"line_number":908,"context_line":""},{"line_number":909,"context_line":"        # Copy the deployment logs if present"},{"line_number":910,"context_line":"        if [ -d $BASE/new/ironic-bm-logs/deploy_logs ]; then"}],"source_content_type":"text/x-sh","patch_set":1,"id":"fa7ab95a_c47d1cb3","line":907,"in_reply_to":"fa7ab95a_898a6b62","updated":"2016-08-31 09:11:25.000000000","message":"Good point, there\u0027s no \"non-log\" stuff in that folder at all. With the new if on L910 we are basically copying all the content of this directory, so +1 for your suggestion to just copy the whole directory and don\u0027t use wildcards.","commit_id":"1376687ca9d60cf641290f7a977fc4c0abac3593"}]}