Alon Bar-Lev has posted comments on this change.

Change subject: vdsm_config: move download certificate
......................................................................


Patch Set 19:

(1 comment)

http://gerrit.ovirt.org/#/c/26718/19/vdsm_reg/vdsm-reg-setup.in
File vdsm_reg/vdsm-reg-setup.in:

Line 216:         if fOK and not os.path.exists(self.engineWebCACert):
Line 217:             fd_tmp, file_tmp = tempfile.mkstemp()
Line 218:             os.close(fd_tmp)
Line 219: 
Line 220:             registration_via_https = True
> getRhevMCert() get the ssl chain, if we can't get the ssl chain we should c
no... you need to use https unsecured, meaning: without validating certificate 
when downloading the certificate.
Line 221:             try:
Line 222:                 if not deployUtil.getRhevmCert(
Line 223:                     IP=self.vdcName,
Line 224:                     port=self.vdcPORT,


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia2c025eedd2be92b9418ddbe01efc02c913af2a7
Gerrit-PatchSet: 19
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsl...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsl...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to