)]}'
{"mariadb/templates/bin/_backup_mariadb.sh.tpl":[{"author":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"change_message_id":"e46e1ebbada0f9068ab5636b363e4a30fc77171e","unresolved":false,"context_lines":[{"line_number":57,"context_line":"mkdir -p $ARCHIVE_DIR"},{"line_number":58,"context_line":"#Dumpinp the database"},{"line_number":59,"context_line":"mysqldump --single-transaction --quick --skip-opt --add-drop-database --add-drop-table --databases \"${DBNAME[@]}\"  \u003e $BACKUPS_DIR/mariadb-all.sql 2\u003edberror.log"},{"line_number":60,"context_line":"if [ $(check_backup \"$?\" \"$BACKUPS_DIR/mariadb-all.sql\") -ne 0 ]"},{"line_number":61,"context_line":"then"},{"line_number":62,"context_line":"  #TODO: This can be convert into mail alert of alert send to a monitoring system"},{"line_number":63,"context_line":"  echo \"Backup failed and need attention.\""}],"source_content_type":"text/x-smarty","patch_set":7,"id":"9fdfeff1_6d66b058","line":60,"range":{"start_line":60,"start_character":21,"end_line":60,"end_character":23},"updated":"2019-03-05 03:11:53.000000000","message":"As we set -xe, so I think $? will must be 0 here. Because if $? is not 0, script exits before this line.","commit_id":"7882c462caaa118ab184ec8b97c322ca6325eccf"}],"mariadb/values.yaml":[{"author":{"_account_id":29268,"name":"Matthew Heler","email":"matthew.heler@hotmail.com","username":"mheler"},"change_message_id":"1c03764b775b118a10532389c1d4dc0cf94c0f43","unresolved":false,"context_lines":[{"line_number":175,"context_line":"jobs:"},{"line_number":176,"context_line":"  # TODO(kn4078): Add a cron job to take a dump of mariadb database"},{"line_number":177,"context_line":"  backup_mariadb:"},{"line_number":178,"context_line":"    cron: \"0 */1 * * *\""},{"line_number":179,"context_line":"    history:"},{"line_number":180,"context_line":"      success: 3"},{"line_number":181,"context_line":"      failed: 1"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9fdfeff1_67a0fa43","line":178,"updated":"2019-02-21 02:59:17.000000000","message":"can we set this to 15 or 30 minutes? this way the gates catch this","commit_id":"c26185d6636c7647938bf50406c08a636dfb3a57"}],"postgresql/values.yaml":[{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"6e28418b1a4fde53f7be2cc15876c930e1324ed9","unresolved":false,"context_lines":[{"line_number":79,"context_line":"          memory: \"1024Mi\""},{"line_number":80,"context_line":"          cpu: \"2000m\""},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"jobs:"},{"line_number":83,"context_line":"  # TODO(kn4078): Add a cron job to take a dump of mariadb database"},{"line_number":84,"context_line":"  backup_postgresql:"},{"line_number":85,"context_line":"    cron: \"0/5 * * * *\""},{"line_number":86,"context_line":"    history:"},{"line_number":87,"context_line":"      success: 3"},{"line_number":88,"context_line":"      failed: 1"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"# using dockerhub postgresql: https://hub.docker.com/r/library/postgres/tags/"},{"line_number":91,"context_line":"images:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"5fc1f717_d33d2c2d","line":88,"range":{"start_line":82,"start_character":0,"end_line":88,"end_character":15},"updated":"2019-03-05 19:03:21.000000000","message":"I\u0027d prefer we handled this under a top level conf: key, similar to other charts that configure cron job fields","commit_id":"9a85bfc696190ad44c4a869cdcbb9e8b4944592d"}],"tools/images/mariadb-backup/Dockerfile":[{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"6e28418b1a4fde53f7be2cc15876c930e1324ed9","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM ubuntu:16.04"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"RUN set -ex ;\\"},{"line_number":4,"context_line":"    apt-get update ;\\"},{"line_number":5,"context_line":"    apt-get upgrade -y ;\\"},{"line_number":6,"context_line":"    apt-get install -y mysql-client ;\\"},{"line_number":7,"context_line":"    apt-get install -y postgresql-client;\\"},{"line_number":8,"context_line":"    apt-get install -y --no-install-recommends ;\\"},{"line_number":9,"context_line":"    rm -rf \\"},{"line_number":10,"context_line":"       /var/log/* \\"},{"line_number":11,"context_line":"       /tmp/*"}],"source_content_type":"text/x-dockerfile","patch_set":9,"id":"5fc1f717_5305bc79","line":11,"range":{"start_line":1,"start_character":0,"end_line":11,"end_character":13},"updated":"2019-03-05 19:03:21.000000000","message":"Why do we need an image for this? Can\u0027t we just use either the mariadb or postgresql image in the respective charts?","commit_id":"9a85bfc696190ad44c4a869cdcbb9e8b4944592d"}]}
