Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.2]: Add additional exception to prevent socket.gaierror error

sbonazzo at redhat.com sbonazzo at redhat.com
Mon Sep 8 07:37:49 UTC 2014


Sandro Bonazzola has submitted this change and it was merged.

Change subject: Add additional exception to prevent socket.gaierror error
......................................................................


Add additional exception to prevent socket.gaierror error

In time of second host, you need to pass FQDN of first host to
receive answer file, but FQDN not pass any validation and can be
incorrect, it will lead to socket.gaierror in socket.getaddrinfo
method.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1136015

Change-Id: I92776791af606b6fdb664c9e44ed119a167d15ac
Signed-off-by: Artyom Lukianov <alukiano at redhat.com>
Signed-off-by: Sandro Bonazzola <sbonazzo at redhat.com>
(cherry picked from commit 38995557e4623003f9dbba5f5ff07063fffe948f)
---
M src/plugins/ovirt-hosted-engine-setup/core/remote_answerfile.py
1 file changed, 10 insertions(+), 6 deletions(-)

Approvals:
  Sandro Bonazzola: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I92776791af606b6fdb664c9e44ed119a167d15ac
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: ovirt-hosted-engine-setup-1.2
Gerrit-Owner: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Artyom Lukianov <alukiano at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list