Sandro Bonazzola has uploaded a new change for review.

Change subject: vds_info: fixed gateway config handling
......................................................................

vds_info: fixed gateway config handling

Previously the gateway configuration was dropped due
to a mistake in parsing the vds capabilities.
This patch fixes that parsing.

Change-Id: I57cabd5e317bd53cc597cf17fb8e2f9c4139712c
Bug-Url: https://bugzilla.redhat.com/1147254
Signed-off-by: Sandro Bonazzola <sbona...@redhat.com>
---
M src/ovirt_hosted_engine_setup/vds_info.py
1 file changed, 2 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-hosted-engine-setup 
refs/changes/04/33604/1

diff --git a/src/ovirt_hosted_engine_setup/vds_info.py 
b/src/ovirt_hosted_engine_setup/vds_info.py
index 011dcb9..ea496ac 100644
--- a/src/ovirt_hosted_engine_setup/vds_info.py
+++ b/src/ovirt_hosted_engine_setup/vds_info.py
@@ -64,14 +64,13 @@
 
     if 'BOOTPROTO' in port_info['cfg']:
         attrs['bootproto'] = port_info['cfg']['BOOTPROTO']
+    if 'GATEWAY' in port_info['cfg']:
+        attrs['gateway'] = port_info['cfg']['GATEWAY']
     if attrs.get('bootproto') == 'dhcp':
         attrs['blockingdhcp'] = True
     else:
         attrs['ipaddr'] = port_info['addr']
         attrs['netmask'] = port_info['netmask']
-        gateway = port_info.get('gateway')
-        if gateway is not None:
-            attrs['gateway'] = gateway
     return attrs
 
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I57cabd5e317bd53cc597cf17fb8e2f9c4139712c
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbona...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to