Yedidyah Bar David has posted comments on this change.

Change subject: pki: Allow changing the manager hostname
......................................................................


Patch Set 17: (4 inline comments)

....................................................
File packaging/bin/ovirt-engine-rename-manager-host
Line 34: aia_in_ca=no
Line 35: 
Line 36: BCKSFX=".$(date +%Y-%m-%d-%H-%M-%S).bckrenovirt"
Line 37: 
Line 38: TMP=`mktemp -d /tmp/ovirt-engine-rename-manager-host.XXXXXX`
Done
Line 39: changedfiles=$TMP/changedfiles
Line 40: 
Line 41: cleanup(){
Line 42:        #rm -rf "${TMP}"


Line 38: TMP=`mktemp -d /tmp/ovirt-engine-rename-manager-host.XXXXXX`
Line 39: changedfiles=$TMP/changedfiles
Line 40: 
Line 41: cleanup(){
Line 42:        #rm -rf "${TMP}"
Doesn't use temp files anymore, removed this func
Line 43:        :
Line 44: }
Line 45: 
Line 46: trap cleanup EXIT


Line 110:               die "Failed updating ${apachecert}"
Line 111: 
Line 112:       echo
Line 113:       echo "Restating apache httpd service:"
Line 114:       service httpd restart
I do not think so. No harm will be done if a normal user will run it.
Line 115:       echo "done."
Line 116:       echo
Line 117: }
Line 118: 


Line 181:               elif echo "${line}" | grep -q '^file.[0-9]*.md5' && \
Line 182:                               grep -q "^${filename}\$" 
"${changedfiles}" && \
Line 183:                               ! grep -q "^${filename}\$" 
"${finishedfiles}"; then
Line 184:                       echo "${filename}" >> "${finishedfiles}"
Line 185:                       md5=$(md5sum "${filename}" | awk '{print $1}')
Done
Line 186:                       groupfile="${GROUPSDIR}/${group}"
Line 187:                       if ! [ -f "${groupfile}" ]; then
Line 188:                               # Write group header
Line 189:                               cat << __EOF__ > "${groupfile}"


-- 
To view, visit http://gerrit.ovirt.org/16404
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifb03d638be1e030865d3ebabaa163ca9be9e70ce
Gerrit-PatchSet: 17
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <[email protected]>
Gerrit-Reviewer: Alex Lourie <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Kiril Nesenko <[email protected]>
Gerrit-Reviewer: Moran Goldboim <[email protected]>
Gerrit-Reviewer: Ofer Schreiber <[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

Reply via email to