)]}'
{"helm-toolkit/templates/manifests/_job-ks-user.yaml.tpl":[{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"be44ddfdeefb6910eb0b7154feca015cc080c853","unresolved":false,"context_lines":[{"line_number":39,"context_line":"spec:"},{"line_number":40,"context_line":"  backoffLimit: {{ $backoffLimit }}"},{"line_number":41,"context_line":"  {{- if $activeDeadlineSeconds }}"},{"line_number":42,"context_line":"  activeDeadlineSeconds: {{ $activeDeadlineSeconds }}"},{"line_number":43,"context_line":"  {{- end }}"},{"line_number":44,"context_line":"  template:"},{"line_number":45,"context_line":"    metadata:"}],"source_content_type":"text/x-smarty","patch_set":54,"id":"7faddb67_9c63efdc","line":42,"range":{"start_line":42,"start_character":0,"end_line":42,"end_character":53},"updated":"2019-07-09 19:56:35.000000000","message":"what if activeDeadlineSeconds is not provided? what is the default?","commit_id":"d0da97ec4fdd2213d91ec72d659febea404fbdd5"},{"author":{"_account_id":28875,"name":"Koffi Nogbe","email":"kn4078@att.com","username":"kn4078"},"change_message_id":"8ab72e18892187f0aa41d03f30fb67347ebc20c8","unresolved":false,"context_lines":[{"line_number":39,"context_line":"spec:"},{"line_number":40,"context_line":"  backoffLimit: {{ $backoffLimit }}"},{"line_number":41,"context_line":"  {{- if $activeDeadlineSeconds }}"},{"line_number":42,"context_line":"  activeDeadlineSeconds: {{ $activeDeadlineSeconds }}"},{"line_number":43,"context_line":"  {{- end }}"},{"line_number":44,"context_line":"  template:"},{"line_number":45,"context_line":"    metadata:"}],"source_content_type":"text/x-smarty","patch_set":54,"id":"7faddb67_2c9dd358","line":42,"range":{"start_line":42,"start_character":0,"end_line":42,"end_character":53},"in_reply_to":"7faddb67_9c63efdc","updated":"2019-07-10 16:43:21.000000000","message":"Done","commit_id":"d0da97ec4fdd2213d91ec72d659febea404fbdd5"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"053c4002071fdbb391e536368081337863b3c38f","unresolved":false,"context_lines":[{"line_number":27,"context_line":"{{- $serviceUser :\u003d index . \"serviceUser\" | default $serviceName -}}"},{"line_number":28,"context_line":"{{- $serviceUserPretty :\u003d $serviceUser | replace \"_\" \"-\" -}}"},{"line_number":29,"context_line":"{{- $backoffLimit :\u003d index . \"backoffLimit\" | default 6 -}}"},{"line_number":30,"context_line":"{{- $activeDeadlineSeconds :\u003d index . \"activeDeadlineSeconds\" | default 600 -}}"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"{{- $serviceAccountName :\u003d printf \"%s-%s\" $serviceUserPretty \"ks-user\" }}"},{"line_number":33,"context_line":"{{ tuple $envAll \"ks_user\" $serviceAccountName | include \"helm-toolkit.snippets.kubernetes_pod_rbac_serviceaccount\" }}"}],"source_content_type":"text/x-smarty","patch_set":58,"id":"7faddb67_6fce0c2e","line":30,"range":{"start_line":30,"start_character":0,"end_line":30,"end_character":79},"updated":"2019-07-19 02:14:47.000000000","message":"I think the default should be 0, which means no deadline at all. In fact, I would suggest not adding the activeDeadlineSeconds if it\u0027s not needed. This could affect all pre-existing keystone user jobs as well.","commit_id":"9ba0499a44ccf2551695dda066e83d0dd45290fd"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"90c199a94a19e10a21b1dfc5797a3f56a01bab8c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":65,"id":"7faddb67_26d73d52","updated":"2019-07-26 13:58:57.000000000","message":"Per Sophie and Cliff\u0027s comments, can we rebase or depends-on: the change here https://review.opendev.org/#/c/672738/3 - it looks like both mariadb \u0026 postgresql backup need them - it would be nice if both patchset uses the same code vs having helm-toolkit changes separately.","commit_id":"1e5e8b89619f1ce7a5b04efcc62b8b291675a996"},{"author":{"_account_id":28875,"name":"Koffi Nogbe","email":"kn4078@att.com","username":"kn4078"},"change_message_id":"6f47f1487e8b11ccb4ee056a8891729f3803b506","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":65,"id":"7faddb67_d98c0ed9","in_reply_to":"7faddb67_26d73d52","updated":"2019-07-26 15:40:02.000000000","message":"Yes. I am working on splitting the PS and depend-on that htk change.","commit_id":"1e5e8b89619f1ce7a5b04efcc62b8b291675a996"}],"mariadb/templates/bin/_backup_mariadb.sh.tpl":[{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"053c4002071fdbb391e536368081337863b3c38f","unresolved":false,"context_lines":[{"line_number":80,"context_line":"  ARCHIVE_RET\u003d$?"},{"line_number":81,"context_line":"  if [ $ARCHIVE_RET -eq 0 ] \u0026\u0026 [ $REMOTE_ARCHIVE ]"},{"line_number":82,"context_line":"  then"},{"line_number":83,"context_line":"    cp -rf $ARCHIVE_DIR/$ARCHIVE_FILE $STAGING_DIR/"},{"line_number":84,"context_line":"    rm -rf $STAGING_DIR/backup.lock"},{"line_number":85,"context_line":"  fi"},{"line_number":86,"context_line":"  popd 1\u003e/dev/null"}],"source_content_type":"text/x-smarty","patch_set":59,"id":"7faddb67_2297974a","line":83,"range":{"start_line":83,"start_character":0,"end_line":83,"end_character":51},"updated":"2019-07-19 02:14:47.000000000","message":"If remote backup is disabled, this is going to leave a copy of the tarball in the staging directory, which effectively doubles the amount of space occupied on the local PVC.","commit_id":"902ab3f052b63f6a18246d5110a648ab68d0ae76"},{"author":{"_account_id":28875,"name":"Koffi Nogbe","email":"kn4078@att.com","username":"kn4078"},"change_message_id":"ac44d1fb3a11ac033c735be9d89855ce91326c9d","unresolved":false,"context_lines":[{"line_number":80,"context_line":"  ARCHIVE_RET\u003d$?"},{"line_number":81,"context_line":"  if [ $ARCHIVE_RET -eq 0 ] \u0026\u0026 [ $REMOTE_ARCHIVE ]"},{"line_number":82,"context_line":"  then"},{"line_number":83,"context_line":"    cp -rf $ARCHIVE_DIR/$ARCHIVE_FILE $STAGING_DIR/"},{"line_number":84,"context_line":"    rm -rf $STAGING_DIR/backup.lock"},{"line_number":85,"context_line":"  fi"},{"line_number":86,"context_line":"  popd 1\u003e/dev/null"}],"source_content_type":"text/x-smarty","patch_set":59,"id":"7faddb67_f9be11ac","line":83,"range":{"start_line":83,"start_character":0,"end_line":83,"end_character":51},"in_reply_to":"7faddb67_2297974a","updated":"2019-07-19 14:30:44.000000000","message":"Nope. This copy will only be made if remote backup is enabled.\nif [ $ARCHIVE_RET -eq 0 ] \u0026\u0026 [ $REMOTE_ARCHIVE ]","commit_id":"902ab3f052b63f6a18246d5110a648ab68d0ae76"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"6d79379df1b970a7afac8bddba37b5b6ea37ac6b","unresolved":false,"context_lines":[{"line_number":80,"context_line":"  ARCHIVE_RET\u003d$?"},{"line_number":81,"context_line":"  if [ $ARCHIVE_RET -eq 0 ] \u0026\u0026 [ $REMOTE_ARCHIVE ]"},{"line_number":82,"context_line":"  then"},{"line_number":83,"context_line":"    cp -rf $ARCHIVE_DIR/$ARCHIVE_FILE $STAGING_DIR/"},{"line_number":84,"context_line":"    rm -rf $STAGING_DIR/backup.lock"},{"line_number":85,"context_line":"  fi"},{"line_number":86,"context_line":"  popd 1\u003e/dev/null"}],"source_content_type":"text/x-smarty","patch_set":59,"id":"7faddb67_64103401","line":83,"range":{"start_line":83,"start_character":0,"end_line":83,"end_character":51},"in_reply_to":"7faddb67_f9be11ac","updated":"2019-07-19 15:04:26.000000000","message":"Got it.  I missed the second condition on the if.  Disregard.","commit_id":"902ab3f052b63f6a18246d5110a648ab68d0ae76"}],"mariadb/templates/bin/_remote_retrieve_mariadb.sh.tpl":[{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"be44ddfdeefb6910eb0b7154feca015cc080c853","unresolved":false,"context_lines":[{"line_number":73,"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\""},{"line_number":74,"context_line":"  echo \"options: \""},{"line_number":75,"context_line":"  echo \"list_archives\""},{"line_number":76,"context_line":"  echo \"download archive_file\""},{"line_number":77,"context_line":"}"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"mkdir -p $REMOTE_ARCHIVE_STAGING_DIR"}],"source_content_type":"text/x-smarty","patch_set":54,"id":"7faddb67_1cb85f74","line":76,"range":{"start_line":76,"start_character":16,"end_line":76,"end_character":17},"updated":"2019-07-09 19:56:35.000000000","message":"Need description for the \"all\" option","commit_id":"d0da97ec4fdd2213d91ec72d659febea404fbdd5"},{"author":{"_account_id":28875,"name":"Koffi Nogbe","email":"kn4078@att.com","username":"kn4078"},"change_message_id":"8ab72e18892187f0aa41d03f30fb67347ebc20c8","unresolved":false,"context_lines":[{"line_number":73,"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\""},{"line_number":74,"context_line":"  echo \"options: \""},{"line_number":75,"context_line":"  echo \"list_archives\""},{"line_number":76,"context_line":"  echo \"download archive_file\""},{"line_number":77,"context_line":"}"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"mkdir -p $REMOTE_ARCHIVE_STAGING_DIR"}],"source_content_type":"text/x-smarty","patch_set":54,"id":"7faddb67_8cade789","line":76,"range":{"start_line":76,"start_character":16,"end_line":76,"end_character":17},"in_reply_to":"7faddb67_1cb85f74","updated":"2019-07-10 16:43:21.000000000","message":"Done","commit_id":"d0da97ec4fdd2213d91ec72d659febea404fbdd5"}],"mariadb/templates/bin/_restore_mariadb.sh.tpl":[{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"4f22a08474ac5de2d0346f51afd1fe5abf7666be","unresolved":false,"context_lines":[{"line_number":185,"context_line":"  echo \"$0 options\""},{"line_number":186,"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\""},{"line_number":187,"context_line":"  echo \"options: \""},{"line_number":188,"context_line":"  echo \"list_archives\""},{"line_number":189,"context_line":"  echo \"list_archives remote\""},{"line_number":190,"context_line":"  echo \"list_databases archive_filename\""},{"line_number":191,"context_line":"  echo \"list_databases remote archive_filename\""},{"line_number":192,"context_line":"  echo \"restore archive_filename [DB_NAME or ALL/all]\""},{"line_number":193,"context_line":"  echo \"restore remote archive_filename [DB_NAME or ALL/all]\""},{"line_number":194,"context_line":"}"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"is_Option() {"}],"source_content_type":"text/x-smarty","patch_set":3,"id":"bfb3d3c7_af6cfdcd","line":193,"range":{"start_line":188,"start_character":0,"end_line":193,"end_character":61},"updated":"2019-05-29 14:19:02.000000000","message":"In my opinion, the list and restore commands should default to remote retrieval instead of local, because the user story specifies that the backups should be restored from the remote ceph RGW. We have chosen to give the user the additional capability to restore from a local archive as well, but it\u0027s not an official deliverable. Just my 2cents though...","commit_id":"20ab6f662f924e71e89cf00f016f7709324abb7d"},{"author":{"_account_id":28875,"name":"Koffi Nogbe","email":"kn4078@att.com","username":"kn4078"},"change_message_id":"8bee6ef3700eb86114b1f2417ac9970d589b63df","unresolved":false,"context_lines":[{"line_number":185,"context_line":"  echo \"$0 options\""},{"line_number":186,"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\""},{"line_number":187,"context_line":"  echo \"options: \""},{"line_number":188,"context_line":"  echo \"list_archives\""},{"line_number":189,"context_line":"  echo \"list_archives remote\""},{"line_number":190,"context_line":"  echo \"list_databases archive_filename\""},{"line_number":191,"context_line":"  echo \"list_databases remote archive_filename\""},{"line_number":192,"context_line":"  echo \"restore archive_filename [DB_NAME or ALL/all]\""},{"line_number":193,"context_line":"  echo \"restore remote archive_filename [DB_NAME or ALL/all]\""},{"line_number":194,"context_line":"}"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"is_Option() {"}],"source_content_type":"text/x-smarty","patch_set":3,"id":"bfb3d3c7_ef8195c4","line":193,"range":{"start_line":188,"start_character":0,"end_line":193,"end_character":61},"in_reply_to":"bfb3d3c7_af6cfdcd","updated":"2019-05-29 14:22:32.000000000","message":"You might not have that ability all the time but you will obviously have the local ability all the time. So default should be what is consistent and option to what is given sometime. That is my though process.","commit_id":"20ab6f662f924e71e89cf00f016f7709324abb7d"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"be44ddfdeefb6910eb0b7154feca015cc080c853","unresolved":false,"context_lines":[{"line_number":236,"context_line":"      ARCHIVE_DIR\u003d$REMOTE_ARCHIVE_STAGING_DIR"},{"line_number":237,"context_line":"      list_archives"},{"line_number":238,"context_line":"    else"},{"line_number":239,"context_line":"      usage"},{"line_number":240,"context_line":"      exit 0"},{"line_number":241,"context_line":"    fi"},{"line_number":242,"context_line":"  else"}],"source_content_type":"text/x-smarty","patch_set":54,"id":"7faddb67_ffd7754e","line":239,"range":{"start_line":239,"start_character":0,"end_line":239,"end_character":11},"updated":"2019-07-09 19:56:35.000000000","message":"what if someone wants to list the archives in the local PVC?","commit_id":"d0da97ec4fdd2213d91ec72d659febea404fbdd5"},{"author":{"_account_id":28875,"name":"Koffi Nogbe","email":"kn4078@att.com","username":"kn4078"},"change_message_id":"8ab72e18892187f0aa41d03f30fb67347ebc20c8","unresolved":false,"context_lines":[{"line_number":236,"context_line":"      ARCHIVE_DIR\u003d$REMOTE_ARCHIVE_STAGING_DIR"},{"line_number":237,"context_line":"      list_archives"},{"line_number":238,"context_line":"    else"},{"line_number":239,"context_line":"      usage"},{"line_number":240,"context_line":"      exit 0"},{"line_number":241,"context_line":"    fi"},{"line_number":242,"context_line":"  else"}],"source_content_type":"text/x-smarty","patch_set":54,"id":"7faddb67_0cb337ec","line":239,"range":{"start_line":239,"start_character":0,"end_line":239,"end_character":11},"in_reply_to":"7faddb67_ffd7754e","updated":"2019-07-10 16:43:21.000000000","message":"Done","commit_id":"d0da97ec4fdd2213d91ec72d659febea404fbdd5"}],"tools/deployment/backup-restore/mariadb-backup-test.sh":[{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"d47573fed7cb0f8a876b4e4df0c600895f7a5c8f","unresolved":false,"context_lines":[{"line_number":13,"context_line":"  kubectl patch cronjobs mariadb-backup -p \u0027{\"spec\" : {\"suspend\" : false }}\u0027 -n $MARIADB_POD_NAMESPACE"},{"line_number":14,"context_line":"  #Tearing down the environment"},{"line_number":15,"context_line":"  #Drop backuptest database"},{"line_number":16,"context_line":"  kubectl exec -it -n $MARIADB_POD_NAMESPACE mariadb-backup-manual -c mariadb-backup-manual -- mysql \\"},{"line_number":17,"context_line":"  --defaults-file\u003d/etc/mysql/admin_user.cnf --connect-timeout 10 -e \"drop database backuptest;\""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"  kubectl delete configmap mariadb-test-db-schema -n $MARIADB_POD_NAMESPACE"}],"source_content_type":"text/x-sh","patch_set":57,"id":"7faddb67_0d0c6739","line":16,"range":{"start_line":16,"start_character":14,"end_line":16,"end_character":18},"updated":"2019-07-12 20:32:24.000000000","message":"do we want an interactive session with kubectl?","commit_id":"6081ab8953fbf6983d3b20b300a59cd4760967b1"},{"author":{"_account_id":28875,"name":"Koffi Nogbe","email":"kn4078@att.com","username":"kn4078"},"change_message_id":"387639283528be2de2965e092360f7e43d2d5878","unresolved":false,"context_lines":[{"line_number":13,"context_line":"  kubectl patch cronjobs mariadb-backup -p \u0027{\"spec\" : {\"suspend\" : false }}\u0027 -n $MARIADB_POD_NAMESPACE"},{"line_number":14,"context_line":"  #Tearing down the environment"},{"line_number":15,"context_line":"  #Drop backuptest database"},{"line_number":16,"context_line":"  kubectl exec -it -n $MARIADB_POD_NAMESPACE mariadb-backup-manual -c mariadb-backup-manual -- mysql \\"},{"line_number":17,"context_line":"  --defaults-file\u003d/etc/mysql/admin_user.cnf --connect-timeout 10 -e \"drop database backuptest;\""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"  kubectl delete configmap mariadb-test-db-schema -n $MARIADB_POD_NAMESPACE"}],"source_content_type":"text/x-sh","patch_set":57,"id":"7faddb67_35e4f3df","line":16,"range":{"start_line":16,"start_character":14,"end_line":16,"end_character":18},"in_reply_to":"7faddb67_0d0c6739","updated":"2019-07-15 20:21:13.000000000","message":"You are right. We don\u0027t need a session just run the commands.\n\nFixing it now.","commit_id":"6081ab8953fbf6983d3b20b300a59cd4760967b1"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"d47573fed7cb0f8a876b4e4df0c600895f7a5c8f","unresolved":false,"context_lines":[{"line_number":223,"context_line":"echo \"Done waiting. Backup pod is up.\""},{"line_number":224,"context_line":""},{"line_number":225,"context_line":"echo \"Creating the test database.\""},{"line_number":226,"context_line":"kubectl exec -it -n $MARIADB_POD_NAMESPACE mariadb-backup-manual -c mariadb-backup-manual -- /tmp/backup_test_schema.sh"},{"line_number":227,"context_line":"sleep 5"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"echo \"Show databases.\""}],"source_content_type":"text/x-sh","patch_set":57,"id":"7faddb67_6d1abb6f","line":226,"range":{"start_line":226,"start_character":11,"end_line":226,"end_character":17},"updated":"2019-07-12 20:32:24.000000000","message":"ditto","commit_id":"6081ab8953fbf6983d3b20b300a59cd4760967b1"},{"author":{"_account_id":28875,"name":"Koffi Nogbe","email":"kn4078@att.com","username":"kn4078"},"change_message_id":"387639283528be2de2965e092360f7e43d2d5878","unresolved":false,"context_lines":[{"line_number":223,"context_line":"echo \"Done waiting. Backup pod is up.\""},{"line_number":224,"context_line":""},{"line_number":225,"context_line":"echo \"Creating the test database.\""},{"line_number":226,"context_line":"kubectl exec -it -n $MARIADB_POD_NAMESPACE mariadb-backup-manual -c mariadb-backup-manual -- /tmp/backup_test_schema.sh"},{"line_number":227,"context_line":"sleep 5"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"echo \"Show databases.\""}],"source_content_type":"text/x-sh","patch_set":57,"id":"7faddb67_55e96fb4","line":226,"range":{"start_line":226,"start_character":11,"end_line":226,"end_character":17},"in_reply_to":"7faddb67_6d1abb6f","updated":"2019-07-15 20:21:13.000000000","message":"Done","commit_id":"6081ab8953fbf6983d3b20b300a59cd4760967b1"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"d47573fed7cb0f8a876b4e4df0c600895f7a5c8f","unresolved":false,"context_lines":[{"line_number":227,"context_line":"sleep 5"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"echo \"Show databases.\""},{"line_number":230,"context_line":"kubectl exec -it -n $MARIADB_POD_NAMESPACE mariadb-backup-manual -c mariadb-backup-manual -- mysql \\"},{"line_number":231,"context_line":"--defaults-file\u003d/etc/mysql/admin_user.cnf --connect-timeout 10 -e \"show databases;\""},{"line_number":232,"context_line":""},{"line_number":233,"context_line":"echo \"Backing up the databases.\""}],"source_content_type":"text/x-sh","patch_set":57,"id":"7faddb67_8d17f745","line":230,"range":{"start_line":230,"start_character":13,"end_line":230,"end_character":14},"updated":"2019-07-12 20:32:24.000000000","message":"ditto","commit_id":"6081ab8953fbf6983d3b20b300a59cd4760967b1"},{"author":{"_account_id":28875,"name":"Koffi Nogbe","email":"kn4078@att.com","username":"kn4078"},"change_message_id":"387639283528be2de2965e092360f7e43d2d5878","unresolved":false,"context_lines":[{"line_number":227,"context_line":"sleep 5"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"echo \"Show databases.\""},{"line_number":230,"context_line":"kubectl exec -it -n $MARIADB_POD_NAMESPACE mariadb-backup-manual -c mariadb-backup-manual -- mysql \\"},{"line_number":231,"context_line":"--defaults-file\u003d/etc/mysql/admin_user.cnf --connect-timeout 10 -e \"show databases;\""},{"line_number":232,"context_line":""},{"line_number":233,"context_line":"echo \"Backing up the databases.\""}],"source_content_type":"text/x-sh","patch_set":57,"id":"7faddb67_15cf7756","line":230,"range":{"start_line":230,"start_character":13,"end_line":230,"end_character":14},"in_reply_to":"7faddb67_8d17f745","updated":"2019-07-15 20:21:13.000000000","message":"Done","commit_id":"6081ab8953fbf6983d3b20b300a59cd4760967b1"}],"tools/deployment/backup-restore/wait-for-pods.sh":[{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"d47573fed7cb0f8a876b4e4df0c600895f7a5c8f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":57,"id":"7faddb67_aa0f3935","updated":"2019-07-12 20:32:24.000000000","message":"Isn\u0027t this the same file as tools/deployment/common/wait-for-pods.sh? If so, why not symlink to that instead of creating a new file.","commit_id":"6081ab8953fbf6983d3b20b300a59cd4760967b1"},{"author":{"_account_id":28875,"name":"Koffi Nogbe","email":"kn4078@att.com","username":"kn4078"},"change_message_id":"387639283528be2de2965e092360f7e43d2d5878","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":57,"id":"7faddb67_5502cf70","in_reply_to":"7faddb67_aa0f3935","updated":"2019-07-15 20:21:13.000000000","message":"It is not the same file. Or same file name different content. So cannot be linked.\n\nThanks","commit_id":"6081ab8953fbf6983d3b20b300a59cd4760967b1"}],"zuul.d/jobs.yaml":[{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"ced49973b37bff3cfdeff3d7ab762cef1d1cfa48","unresolved":false,"context_lines":[{"line_number":406,"context_line":"        - ./tools/deployment/podsecuritypolicy/007-podsecuritypolicy.sh"},{"line_number":407,"context_line":""},{"line_number":408,"context_line":"- job:"},{"line_number":409,"context_line":"    name: openstack-helm-infra-database-backup-restore"},{"line_number":410,"context_line":"    parent: openstack-helm-infra-functional"},{"line_number":411,"context_line":"    timeout: 7200"},{"line_number":412,"context_line":"    pre-run: playbooks/osh-infra-upgrade-host.yaml"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"bfb3d3c7_742018f8","line":409,"range":{"start_line":409,"start_character":10,"end_line":409,"end_character":54},"updated":"2019-05-29 13:49:56.000000000","message":"You need to add this job to the zuul project file in order for it to run at check time.","commit_id":"20ab6f662f924e71e89cf00f016f7709324abb7d"},{"author":{"_account_id":28875,"name":"Koffi Nogbe","email":"kn4078@att.com","username":"kn4078"},"change_message_id":"f307cb221f9b1c1e578487a2ad31daa2f67b8098","unresolved":false,"context_lines":[{"line_number":406,"context_line":"        - ./tools/deployment/podsecuritypolicy/007-podsecuritypolicy.sh"},{"line_number":407,"context_line":""},{"line_number":408,"context_line":"- job:"},{"line_number":409,"context_line":"    name: openstack-helm-infra-database-backup-restore"},{"line_number":410,"context_line":"    parent: openstack-helm-infra-functional"},{"line_number":411,"context_line":"    timeout: 7200"},{"line_number":412,"context_line":"    pre-run: playbooks/osh-infra-upgrade-host.yaml"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"bfb3d3c7_54cdd42e","line":409,"range":{"start_line":409,"start_character":10,"end_line":409,"end_character":54},"in_reply_to":"bfb3d3c7_742018f8","updated":"2019-05-29 13:51:42.000000000","message":"Done","commit_id":"20ab6f662f924e71e89cf00f016f7709324abb7d"},{"author":{"_account_id":18236,"name":"Sophie Huang","email":"sophieh01@yahoo.com","username":"SophieHuang"},"change_message_id":"84a078f32998a2b9a7f935cfea683e9c870fe327","unresolved":false,"context_lines":[{"line_number":415,"context_line":"        - ./tools/deployment/podsecuritypolicy/007-podsecuritypolicy.sh"},{"line_number":416,"context_line":""},{"line_number":417,"context_line":"- job:"},{"line_number":418,"context_line":"    name: openstack-helm-infra-database-backup-restore"},{"line_number":419,"context_line":"    parent: openstack-helm-infra-functional"},{"line_number":420,"context_line":"    timeout: 7200"},{"line_number":421,"context_line":"    pre-run: playbooks/osh-infra-upgrade-host.yaml"}],"source_content_type":"text/x-yaml","patch_set":62,"id":"7faddb67_88d0c555","line":418,"range":{"start_line":418,"start_character":10,"end_line":418,"end_character":54},"updated":"2019-07-22 22:27:36.000000000","message":"This job has the same name as the one in Id17a8fcb63f5614ea038c58acdc256fb4e05f434, but with different contents. We need to coordinate before merge.","commit_id":"22e96e018ba14b5ffc19db0586d67afed7516069"},{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"ddcbfa168c2384198c5d5ed6b6d9463349b73743","unresolved":false,"context_lines":[{"line_number":415,"context_line":"        - ./tools/deployment/podsecuritypolicy/007-podsecuritypolicy.sh"},{"line_number":416,"context_line":""},{"line_number":417,"context_line":"- job:"},{"line_number":418,"context_line":"    name: openstack-helm-infra-database-backup-restore"},{"line_number":419,"context_line":"    parent: openstack-helm-infra-functional"},{"line_number":420,"context_line":"    timeout: 7200"},{"line_number":421,"context_line":"    pre-run: playbooks/osh-infra-upgrade-host.yaml"}],"source_content_type":"text/x-yaml","patch_set":62,"id":"7faddb67_2eaaf9d1","line":418,"range":{"start_line":418,"start_character":10,"end_line":418,"end_character":54},"in_reply_to":"7faddb67_88d0c555","updated":"2019-07-23 01:52:37.000000000","message":"Very good point. When I originally created the job for Postgresql, I envisioned both MariaDB and Postgresql using the same job for testing backup/restore. I think we can have this job load all that is needed for both components, then run both of the test scripts.","commit_id":"22e96e018ba14b5ffc19db0586d67afed7516069"}]}