Ofer Schreiber has posted comments on this change.
Change subject: packaging: Add backup and restore utility
......................................................................
Patch Set 19: (4 inline comments)
....................................................
File packaging/bin/engine-backup.sh
Line 23: BACKUP_FOLDERS="/etc/ovirt-engine
Line 24: /etc/pki/ovirt-engine
Line 25: /etc/ovirt-engine-setup.conf.d
Line 26: /var/lib/ovirt-engine"
Line 27: VERSION_FILE="/usr/share/ovirt-engine/conf/version"
I'm not sure I understood your suggestion.
do you want me to read the VERSION_FILE variable from prolog?
or should I add PACKAGE_VERSION to prolog instead of using the version file?
Line 28: MYPGPASS=""
Line 29: TEMP_FOLDER=""
Line 30: FILE=""
Line 31: DB_BACKUP_FILE_NAME="engine_backup.db"
Line 79: --file=*)
Line 80: FILE="${v}"
Line 81: ;;
Line 82: --backup=*)
Line 83: BACKUP_FILE="${v}"
since during restore we use --file as the data to restore, and --backup for
backing up directories we override
Line 84: ;;
Line 85: --log=*)
Line 86: LOG="${v}"
Line 87: ;;
Line 136: createmd5() {
Line 137: local tardir="$1"
Line 138: local md5file="$2"
Line 139: find "${tardir}" -type f -printf "%P\n" | while read file; do
Line 140: ( cd "${tardir}" && md5sum "${file}" >> "${md5file}" )
|| die "Cannot create checksum for '${file}'"
Done
Line 141: done || die "Find execution failed"
Line 142: }
Line 143:
Line 144: verifymd5() {
Line 248: __EOF__
Line 249: }
Line 250:
Line 251: output() {
Line 252: echo "$1"
add a write to the log is easy.
how do you suggest implementing the die method you suggested? create a "die"
like function (and ignore the one at prolog?)
Line 253: }
Line 254:
Line 255: ## Main
Line 256:
--
To view, visit http://gerrit.ovirt.org/15276
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I7f6c386a0f48ccd520978193639120999e00cf2a
Gerrit-PatchSet: 19
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ofer Schreiber <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Kiril Nesenko <[email protected]>
Gerrit-Reviewer: Moran Goldboim <[email protected]>
Gerrit-Reviewer: Ofer Schreiber <[email protected]>
Gerrit-Reviewer: Ohad Basan <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Yedidyah Bar David <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
