Dan Kenigsberg has submitted this change and it was merged. Change subject: certificates: remove config.read call ......................................................................
certificates: remove config.read call config.read(file) loads a new configuration to 'config' based on the file it reads. This call causes a bug and incosistency of vdsm configuration since it overrides the right configuration. config should be built from files in pre configured paths (drop-in dirs) with priority determined by configuration file names. Change-Id: I3f7c092a81af2dbe44cb60ab86c4bf74570b4514 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1368115 Signed-off-by: Irit Goihman <[email protected]> Reviewed-on: https://gerrit.ovirt.org/62885 Continuous-Integration: Jenkins CI Reviewed-by: Yaniv Bronhaim <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M lib/vdsm/tool/configurators/certificates.py 1 file changed, 0 insertions(+), 6 deletions(-) Approvals: Yaniv Bronhaim: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Irit Goihman: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/62885 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3f7c092a81af2dbe44cb60ab86c4bf74570b4514 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Irit Goihman <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Irit Goihman <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
