Irit Goihman has uploaded a new change for review.

Change subject: certificates: remove config.read call
......................................................................

certificates: remove config.read call

certificates initiated config read from 'vdsm.conf' file, while config
should be built from a few configuration files. This call overrides
vdsm configuration and should be removed.

Change-Id: I3f7c092a81af2dbe44cb60ab86c4bf74570b4514
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1368115
Signed-off-by: Irit Goihman <igoih...@redhat.com>
---
M lib/vdsm/tool/configurators/certificates.py
1 file changed, 0 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/85/62885/1

diff --git a/lib/vdsm/tool/configurators/certificates.py 
b/lib/vdsm/tool/configurators/certificates.py
index bdb686a..0462b4d 100644
--- a/lib/vdsm/tool/configurators/certificates.py
+++ b/lib/vdsm/tool/configurators/certificates.py
@@ -68,11 +68,5 @@
 
 
 def _certsExist():
-    config.read(
-        os.path.join(
-            SYSCONF_PATH,
-            'vdsm/vdsm.conf'
-        )
-    )
     return not config.getboolean('vars', 'ssl') or\
         os.path.isfile(CERT_FILE)


-- 
To view, visit https://gerrit.ovirt.org/62885
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3f7c092a81af2dbe44cb60ab86c4bf74570b4514
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Irit Goihman <igoih...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Reply via email to