)]}'
{"helm-toolkit/templates/scripts/db-backup-restore/_backup_main.sh.tpl":[{"author":{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"},"change_message_id":"16965dbb96d9394e85b12aa54f577e1b449b7dfc","unresolved":false,"context_lines":[{"line_number":327,"context_line":"  rm -f $ERR_LOG_FILE"},{"line_number":328,"context_line":""},{"line_number":329,"context_line":"  #Only delete the old archive after a successful archive"},{"line_number":330,"context_line":"  export LOCAL_DAYS_TO_KEEP\u003d$(echo $LOCAL_DAYS_TO_KEEP | sed \u0027s/\"//g\u0027)"},{"line_number":331,"context_line":"  if [[ \"$LOCAL_DAYS_TO_KEEP\" -gt 0 ]]; then"},{"line_number":332,"context_line":"    remove_old_local_archives"},{"line_number":333,"context_line":"  fi"}],"source_content_type":"text/x-smarty","patch_set":10,"id":"ff570b3c_5a9dd855","line":330,"range":{"start_line":330,"start_character":28,"end_line":330,"end_character":70},"updated":"2020-06-10 17:55:20.000000000","message":"won\u0027t you prefer something like this more if we can\u0027t avoid this from having in input\nexport LOCAL_DAYS_TO_KEEP\u003d${LOCAL_DAYS_TO_KEEP//\\\"}\n\ndon\u0027t think we need to support shell compatibility","commit_id":"b374fe1d0511590ee50a2ccb676fb1ab97b5f3e5"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"f4f22e07a8fce7bfdebe24426940434083bea839","unresolved":false,"context_lines":[{"line_number":327,"context_line":"  rm -f $ERR_LOG_FILE"},{"line_number":328,"context_line":""},{"line_number":329,"context_line":"  #Only delete the old archive after a successful archive"},{"line_number":330,"context_line":"  export LOCAL_DAYS_TO_KEEP\u003d$(echo $LOCAL_DAYS_TO_KEEP | sed \u0027s/\"//g\u0027)"},{"line_number":331,"context_line":"  if [[ \"$LOCAL_DAYS_TO_KEEP\" -gt 0 ]]; then"},{"line_number":332,"context_line":"    remove_old_local_archives"},{"line_number":333,"context_line":"  fi"}],"source_content_type":"text/x-smarty","patch_set":10,"id":"ff570b3c_f3820700","line":330,"range":{"start_line":330,"start_character":28,"end_line":330,"end_character":70},"in_reply_to":"ff570b3c_5a9dd855","updated":"2020-06-11 13:58:25.000000000","message":"Thank you Andrii for the feedback!  I\u0027ll keep looking for a better way, but this script is called from both the cron job and from the CLI, and they pass in the variable differently. One way with the quotes because it is being read from secret, and the other without the quotes because it comes directly from the values.yaml. So it wasn\u0027t done this way for shell compatibility, rather, it was done this way to accommodate the two different ways the variable can be provided to this script.","commit_id":"b374fe1d0511590ee50a2ccb676fb1ab97b5f3e5"},{"author":{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"},"change_message_id":"40539660fb844b3e181d72f5af90d9eac5beecae","unresolved":false,"context_lines":[{"line_number":327,"context_line":"  rm -f $ERR_LOG_FILE"},{"line_number":328,"context_line":""},{"line_number":329,"context_line":"  #Only delete the old archive after a successful archive"},{"line_number":330,"context_line":"  export LOCAL_DAYS_TO_KEEP\u003d$(echo $LOCAL_DAYS_TO_KEEP | sed \u0027s/\"//g\u0027)"},{"line_number":331,"context_line":"  if [[ \"$LOCAL_DAYS_TO_KEEP\" -gt 0 ]]; then"},{"line_number":332,"context_line":"    remove_old_local_archives"},{"line_number":333,"context_line":"  fi"}],"source_content_type":"text/x-smarty","patch_set":10,"id":"ff570b3c_538c93d6","line":330,"range":{"start_line":330,"start_character":28,"end_line":330,"end_character":70},"in_reply_to":"ff570b3c_f3820700","updated":"2020-06-11 14:15:18.000000000","message":"Sorry i meant something else when mentioned shell compatibility. Your current solution is compatible with both bash and sh. The one i suggested (${VAR//\\\"}) - is bash only. As we don\u0027t care about sh compatibility we can use it as more neater one, though I surely don\u0027t insist.","commit_id":"b374fe1d0511590ee50a2ccb676fb1ab97b5f3e5"},{"author":{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"},"change_message_id":"16965dbb96d9394e85b12aa54f577e1b449b7dfc","unresolved":false,"context_lines":[{"line_number":332,"context_line":"    remove_old_local_archives"},{"line_number":333,"context_line":"  fi"},{"line_number":334,"context_line":""},{"line_number":335,"context_line":"  REMOTE_BACKUP\u003d$(echo $REMOTE_BACKUP_ENABLED | sed \u0027s/\"//g\u0027)"},{"line_number":336,"context_line":"  if $REMOTE_BACKUP; then"},{"line_number":337,"context_line":"    store_backup_remotely $ARCHIVE_DIR $TARBALL_FILE"},{"line_number":338,"context_line":"    if [[ $? -ne 0 ]]; then"}],"source_content_type":"text/x-smarty","patch_set":10,"id":"ff570b3c_bac35438","line":335,"range":{"start_line":335,"start_character":2,"end_line":335,"end_character":61},"updated":"2020-06-10 17:55:20.000000000","message":"ditto","commit_id":"b374fe1d0511590ee50a2ccb676fb1ab97b5f3e5"},{"author":{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"},"change_message_id":"16965dbb96d9394e85b12aa54f577e1b449b7dfc","unresolved":false,"context_lines":[{"line_number":340,"context_line":"    fi"},{"line_number":341,"context_line":""},{"line_number":342,"context_line":"    #Only delete the old archive after a successful archive"},{"line_number":343,"context_line":"    export REMOTE_DAYS_TO_KEEP\u003d$(echo $REMOTE_DAYS_TO_KEEP | sed \u0027s/\"//g\u0027)"},{"line_number":344,"context_line":"    if [[ \"$REMOTE_DAYS_TO_KEEP\" -gt 0 ]]; then"},{"line_number":345,"context_line":"      remove_old_remote_archives"},{"line_number":346,"context_line":"    fi"}],"source_content_type":"text/x-smarty","patch_set":10,"id":"ff570b3c_fadaec1a","line":343,"range":{"start_line":343,"start_character":4,"end_line":343,"end_character":74},"updated":"2020-06-10 17:55:20.000000000","message":"ditto","commit_id":"b374fe1d0511590ee50a2ccb676fb1ab97b5f3e5"},{"author":{"_account_id":24816,"name":"Anthony Bellino","email":"ab2434@att.com","username":"anthony.bellino"},"change_message_id":"04cd2ba0ddf2620bd96ff52e70fb01ded85a2301","unresolved":false,"context_lines":[{"line_number":318,"context_line":"  # Get the size of the file"},{"line_number":319,"context_line":"  ARCHIVE_SIZE\u003d$(ls -l $ARCHIVE_DIR/$TARBALL_FILE | awk \u0027{print $5}\u0027)"},{"line_number":320,"context_line":""},{"line_number":321,"context_line":"  log INFO \"${DB_NAME}_backup\" \"Tarball $TARBALL_FILE created successfully.\""},{"line_number":322,"context_line":""},{"line_number":323,"context_line":"  cd $ARCHIVE_DIR"},{"line_number":324,"context_line":""}],"source_content_type":"text/x-smarty","patch_set":21,"id":"bf51134e_11f8e546","line":321,"range":{"start_line":321,"start_character":2,"end_line":321,"end_character":76},"updated":"2020-06-24 14:38:46.000000000","message":"according to log() L85 this takes 4 params?","commit_id":"7ff92a3effbfa7bcfb3c5fdff5091fdd22f4fd73"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"004026676c8c2ef928071f2bc57dcc7d83f6c7e0","unresolved":false,"context_lines":[{"line_number":318,"context_line":"  # Get the size of the file"},{"line_number":319,"context_line":"  ARCHIVE_SIZE\u003d$(ls -l $ARCHIVE_DIR/$TARBALL_FILE | awk \u0027{print $5}\u0027)"},{"line_number":320,"context_line":""},{"line_number":321,"context_line":"  log INFO \"${DB_NAME}_backup\" \"Tarball $TARBALL_FILE created successfully.\""},{"line_number":322,"context_line":""},{"line_number":323,"context_line":"  cd $ARCHIVE_DIR"},{"line_number":324,"context_line":""}],"source_content_type":"text/x-smarty","patch_set":21,"id":"bf51134e_d665bcb6","line":321,"range":{"start_line":321,"start_character":2,"end_line":321,"end_character":76},"in_reply_to":"bf51134e_11f8e546","updated":"2020-06-24 19:15:36.000000000","message":"The 4th parameter is optional, in case you want to dump it into a file.","commit_id":"7ff92a3effbfa7bcfb3c5fdff5091fdd22f4fd73"},{"author":{"_account_id":24816,"name":"Anthony Bellino","email":"ab2434@att.com","username":"anthony.bellino"},"change_message_id":"04cd2ba0ddf2620bd96ff52e70fb01ded85a2301","unresolved":false,"context_lines":[{"line_number":338,"context_line":"    if [[ $? -ne 0 ]]; then"},{"line_number":339,"context_line":"      # This error should print first, then print the summary as the last"},{"line_number":340,"context_line":"      # thing that the user sees in the output."},{"line_number":341,"context_line":"      log ERROR \"${DB_NAME}_backup\" \"Backup could not be sent to remote RGW.\""},{"line_number":342,"context_line":"      set +x"},{"line_number":343,"context_line":"      echo \"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\""},{"line_number":344,"context_line":"      echo \"Local backup successful, but could not send to remote RGW.\""}],"source_content_type":"text/x-smarty","patch_set":21,"id":"bf51134e_118d85d2","line":341,"range":{"start_line":341,"start_character":6,"end_line":341,"end_character":77},"updated":"2020-06-24 14:38:46.000000000","message":"ditto","commit_id":"7ff92a3effbfa7bcfb3c5fdff5091fdd22f4fd73"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"004026676c8c2ef928071f2bc57dcc7d83f6c7e0","unresolved":false,"context_lines":[{"line_number":338,"context_line":"    if [[ $? -ne 0 ]]; then"},{"line_number":339,"context_line":"      # This error should print first, then print the summary as the last"},{"line_number":340,"context_line":"      # thing that the user sees in the output."},{"line_number":341,"context_line":"      log ERROR \"${DB_NAME}_backup\" \"Backup could not be sent to remote RGW.\""},{"line_number":342,"context_line":"      set +x"},{"line_number":343,"context_line":"      echo \"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\""},{"line_number":344,"context_line":"      echo \"Local backup successful, but could not send to remote RGW.\""}],"source_content_type":"text/x-smarty","patch_set":21,"id":"bf51134e_365998fd","line":341,"range":{"start_line":341,"start_character":6,"end_line":341,"end_character":77},"in_reply_to":"bf51134e_118d85d2","updated":"2020-06-24 19:15:36.000000000","message":"The 4th parameter is optional, in case you want to dump it into a file.","commit_id":"7ff92a3effbfa7bcfb3c5fdff5091fdd22f4fd73"}],"postgresql/templates/bin/_backup_postgresql.sh.tpl":[{"author":{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"},"change_message_id":"16965dbb96d9394e85b12aa54f577e1b449b7dfc","unresolved":false,"context_lines":[{"line_number":41,"context_line":"  TMP_DIR\u003d$1"},{"line_number":42,"context_line":"  LOG_FILE\u003d$2"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"  PG_DUMPALL_OPTIONS\u003d$(echo $POSTGRESQL_BACKUP_PG_DUMPALL_OPTIONS | sed \u0027s/\"//g\u0027)"},{"line_number":45,"context_line":"  PG_DUMPALL\u003d\"pg_dumpall \\"},{"line_number":46,"context_line":"                $PG_DUMPALL_OPTIONS \\"},{"line_number":47,"context_line":"                -U $POSTGRESQL_ADMIN_USER \\"}],"source_content_type":"text/x-smarty","patch_set":10,"id":"ff570b3c_3a446474","line":44,"range":{"start_line":44,"start_character":21,"end_line":44,"end_character":81},"updated":"2020-06-10 17:55:20.000000000","message":"ditto","commit_id":"b374fe1d0511590ee50a2ccb676fb1ab97b5f3e5"}],"postgresql/templates/bin/_restore_postgresql.sh.tpl":[{"author":{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"},"change_message_id":"16965dbb96d9394e85b12aa54f577e1b449b7dfc","unresolved":false,"context_lines":[{"line_number":127,"context_line":"    extract_single_db_dump $TMP_DIR/$SQL_FILE $SINGLE_DB_NAME $TMP_DIR"},{"line_number":128,"context_line":"    if [[ -f $TMP_DIR/$SINGLE_DB_NAME.sql \u0026\u0026 -s $TMP_DIR/$SINGLE_DB_NAME.sql ]]; then"},{"line_number":129,"context_line":"      # First drop the database"},{"line_number":130,"context_line":"      $PSQL -tc \"DROP DATABASE $SINGLE_DB_NAME;\""},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"      # Postgresql does not have the concept of creating database if condition."},{"line_number":133,"context_line":"      # This next command creates the database in case it does not exist."}],"source_content_type":"text/x-smarty","patch_set":10,"id":"ff570b3c_ba1fd4b5","line":130,"range":{"start_line":130,"start_character":32,"end_line":130,"end_character":46},"updated":"2020-06-10 17:55:20.000000000","message":"i understand it\u0027s everywhere around, but could you please consider using curved brackets","commit_id":"b374fe1d0511590ee50a2ccb676fb1ab97b5f3e5"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"a3ca471d2840818b9b8294397ff04a156ea6cd2e","unresolved":false,"context_lines":[{"line_number":127,"context_line":"    extract_single_db_dump $TMP_DIR/$SQL_FILE $SINGLE_DB_NAME $TMP_DIR"},{"line_number":128,"context_line":"    if [[ -f $TMP_DIR/$SINGLE_DB_NAME.sql \u0026\u0026 -s $TMP_DIR/$SINGLE_DB_NAME.sql ]]; then"},{"line_number":129,"context_line":"      # First drop the database"},{"line_number":130,"context_line":"      $PSQL -tc \"DROP DATABASE $SINGLE_DB_NAME;\""},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"      # Postgresql does not have the concept of creating database if condition."},{"line_number":133,"context_line":"      # This next command creates the database in case it does not exist."}],"source_content_type":"text/x-smarty","patch_set":10,"id":"ff570b3c_b3b6cfdb","line":130,"range":{"start_line":130,"start_character":32,"end_line":130,"end_character":46},"in_reply_to":"ff570b3c_ba1fd4b5","updated":"2020-06-11 14:00:13.000000000","message":"It\u0027s on my list to do.  if not in this commit, it will be in the next.","commit_id":"b374fe1d0511590ee50a2ccb676fb1ab97b5f3e5"},{"author":{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"},"change_message_id":"b71073465e805ea987f4d0ae21f661a80c6f5518","unresolved":false,"context_lines":[{"line_number":144,"context_line":"      else"},{"line_number":145,"context_line":"        # Dump out the log file for debugging"},{"line_number":146,"context_line":"        cat $LOG_FILE"},{"line_number":147,"context_line":"        echo \"\""},{"line_number":148,"context_line":"        echo \"Database restore Failed.\""},{"line_number":149,"context_line":"        return 1"},{"line_number":150,"context_line":"      fi"},{"line_number":151,"context_line":"    else"}],"source_content_type":"text/x-smarty","patch_set":21,"id":"bf51134e_190f7937","line":148,"range":{"start_line":147,"start_character":0,"end_line":148,"end_character":39},"updated":"2020-06-24 00:52:42.000000000","message":"echo -e \"\\nDatabase restore Failed.\"","commit_id":"7ff92a3effbfa7bcfb3c5fdff5091fdd22f4fd73"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"004026676c8c2ef928071f2bc57dcc7d83f6c7e0","unresolved":false,"context_lines":[{"line_number":144,"context_line":"      else"},{"line_number":145,"context_line":"        # Dump out the log file for debugging"},{"line_number":146,"context_line":"        cat $LOG_FILE"},{"line_number":147,"context_line":"        echo \"\""},{"line_number":148,"context_line":"        echo \"Database restore Failed.\""},{"line_number":149,"context_line":"        return 1"},{"line_number":150,"context_line":"      fi"},{"line_number":151,"context_line":"    else"}],"source_content_type":"text/x-smarty","patch_set":21,"id":"bf51134e_964ea435","line":148,"range":{"start_line":147,"start_character":0,"end_line":148,"end_character":39},"in_reply_to":"bf51134e_190f7937","updated":"2020-06-24 19:15:36.000000000","message":"Will fix.","commit_id":"7ff92a3effbfa7bcfb3c5fdff5091fdd22f4fd73"},{"author":{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"},"change_message_id":"b71073465e805ea987f4d0ae21f661a80c6f5518","unresolved":false,"context_lines":[{"line_number":171,"context_line":"    else"},{"line_number":172,"context_line":"      # Dump out the log file for debugging"},{"line_number":173,"context_line":"      cat $LOG_FILE"},{"line_number":174,"context_line":"      echo \"\""},{"line_number":175,"context_line":"      echo \"Database Restore failed.\""},{"line_number":176,"context_line":"      return 1"},{"line_number":177,"context_line":"    fi"},{"line_number":178,"context_line":"  else"}],"source_content_type":"text/x-smarty","patch_set":21,"id":"bf51134e_5c1a7f6e","line":175,"range":{"start_line":174,"start_character":0,"end_line":175,"end_character":37},"updated":"2020-06-24 00:52:42.000000000","message":"ditto","commit_id":"7ff92a3effbfa7bcfb3c5fdff5091fdd22f4fd73"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"004026676c8c2ef928071f2bc57dcc7d83f6c7e0","unresolved":false,"context_lines":[{"line_number":171,"context_line":"    else"},{"line_number":172,"context_line":"      # Dump out the log file for debugging"},{"line_number":173,"context_line":"      cat $LOG_FILE"},{"line_number":174,"context_line":"      echo \"\""},{"line_number":175,"context_line":"      echo \"Database Restore failed.\""},{"line_number":176,"context_line":"      return 1"},{"line_number":177,"context_line":"    fi"},{"line_number":178,"context_line":"  else"}],"source_content_type":"text/x-smarty","patch_set":21,"id":"bf51134e_f66280ad","line":175,"range":{"start_line":174,"start_character":0,"end_line":175,"end_character":37},"in_reply_to":"bf51134e_5c1a7f6e","updated":"2020-06-24 19:15:36.000000000","message":"Will fix.","commit_id":"7ff92a3effbfa7bcfb3c5fdff5091fdd22f4fd73"}]}