Alon Bar-Lev has uploaded a new change for review.
Change subject: vdsm: bridge: do not allow multiple same ips for engine
......................................................................
vdsm: bridge: do not allow multiple same ips for engine
Change-Id: I3fd95e61d1b1a405650f164d547004151d9f5f68
Signed-off-by: Alon Bar-Lev <alo...@redhat.com>
---
M src/plugins/ovirt-host-deploy/vdsm/bridge.py
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/ovirt-host-deploy
refs/changes/09/14209/1
diff --git a/src/plugins/ovirt-host-deploy/vdsm/bridge.py
b/src/plugins/ovirt-host-deploy/vdsm/bridge.py
index 67f3dec..747ee0b 100644
--- a/src/plugins/ovirt-host-deploy/vdsm/bridge.py
+++ b/src/plugins/ovirt-host-deploy/vdsm/bridge.py
@@ -588,13 +588,13 @@
self.logger.debug('Management interface already exists')
else:
try:
- addresses = [
+ addresses = set([
address[0] for __, __, __, __, address in
socket.getaddrinfo(
host,
None
)
- ]
+ ])
self.logger.debug('Engine %s addresses: %s', host, addresses)
except:
self.logger.debug(
--
To view, visit http://gerrit.ovirt.org/14209
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3fd95e61d1b1a405650f164d547004151d9f5f68
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-host-deploy
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
