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

sbonazzo at redhat.com sbonazzo at redhat.com
Mon Sep 8 07:37:31 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>
---
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
  Yedidyah Bar David: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I92776791af606b6fdb664c9e44ed119a167d15ac
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Artyom Lukianov <alukiano at redhat.com>
Gerrit-Reviewer: Artyom Lukianov <alukiano at redhat.com>
Gerrit-Reviewer: Jiří Moskovčák <jmoskovc at redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list