Change in vdsm[master]: BZ#741856 delete rhevm cert before conf_update

Yotam Oron yoron at redhat.com
Tue Oct 4 14:12:11 UTC 2011


Yotam Oron has posted comments on this change.

Change subject: BZ#741856 delete rhevm cert before conf_update
......................................................................


Patch Set 1: (2 inline comments)

....................................................
File vdsm_reg/deployUtil.py.in
Line 1327:     return data
Line 1328: 
Line 1329: def nodeCleanup():
Line 1330:     if isOvirt():
Line 1331:         CACERT, dontcare = certPaths('')
Done
Line 1332:         if os.path.exists(CACERT):
Line 1333:             ovirtfunctions.ovirt_safe_delete_config(CACERT)
Line 1334: 
Line 1335: def getRhevmCert(IP, port):


....................................................
File vdsm_reg/vdsm-config
Line 82: 			sed --copy -i "s/\(^vdc_host_name=\)\(..*$\)/\1${vdc_managment_server}/" \
Line 83: 				/etc/vdsm-reg/vdsm-reg.conf
Line 84: 
Line 85: 			res=`python "$DEPLOY_UTIL" --node-cleanup`
Line 86:             ret_val=$?
ERRRRR !!! I HATE THOSE FREAKIN' TABS !! :-)

Done.
Line 87:             echo "$res" >> $LOG 2>&1
Line 88:             if [ ! $ret_val -eq 0 ];then
Line 89:                 echo "Node cleanup failed" >> $LOG 2>&1
Line 90:             fi


--
To view, visit http://gerrit.usersys.redhat.com/987
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I8bc1dc1fb7fa2ffc0bcb1f6389b34a9ee82b4197
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yotam Oron <yoron at redhat.com>
Gerrit-Reviewer: Ayal Baron
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Yotam Oron <yoron at redhat.com>


More information about the vdsm-patches mailing list