Kiril Nesenko has posted comments on this change. Change subject: core: fixing restore.sh log file location ......................................................................
Patch Set 1: (2 comments) http://gerrit.ovirt.org/#/c/25033/1/packaging/dbscripts/dbcustomfunctions.sh File packaging/dbscripts/dbcustomfunctions.sh: Line 14: DATABASE="engine" Line 15: USERNAME="engine" Line 16: VERBOSE=false Line 17: LOGDIR="/var/log/ovirt-engine" Line 18: if [ -d ${LOGDIR} ]; then please use quotes for shell variables Line 19: LOGFILE="${LOGDIR}/$ME.log" Line 20: else Line 21: LOGFILE="$ME.log" Line 22: fi Line 17: LOGDIR="/var/log/ovirt-engine" Line 18: if [ -d ${LOGDIR} ]; then Line 19: LOGFILE="${LOGDIR}/$ME.log" Line 20: else Line 21: LOGFILE="$ME.log" please be consistent with. Use $var or ${var} Line 22: fi Line 23: DBOBJECT_OWNER="engine" Line 24: NOMD5="false" Line 25: MD5DIR="$(pwd)" -- To view, visit http://gerrit.ovirt.org/25033 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3cd0742896e2fb5cfa79c75147ced47b94c37155 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Eli Mesika <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Itamar Heim <[email protected]> Gerrit-Reviewer: Kiril Nesenko <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Ravi Nori <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
