Change in vdsm[master]: validate listen_tls config value in libvirtd.conf

danken at redhat.com danken at redhat.com
Tue Jul 21 11:16:37 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: validate listen_tls config value in libvirtd.conf
......................................................................


validate listen_tls config value in libvirtd.conf

Currently we don't validate this value which leads to broken
behaviour in migration over ssl.

Change-Id: Iedad148fd0f99fe02988a87a8bdfc0a9cc64d282
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1148024
Signed-off-by: Yaniv Bronhaim <ybronhei at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/43437
Continuous-Integration: Jenkins CI
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M lib/vdsm/tool/configurators/libvirt.py
1 file changed, 10 insertions(+), 6 deletions(-)

Approvals:
  Yaniv Bronhaim: Verified
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iedad148fd0f99fe02988a87a8bdfc0a9cc64d282
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list