Change in vdsm[master]: Make getRoutes() return the default gateway of each bridge i...

danken at redhat.com danken at redhat.com
Tue Jan 24 16:40:18 UTC 2012


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Make getRoutes() return the default gateway of each bridge interface
......................................................................


Make getRoutes() return the default gateway of each bridge interface

Assuming `route -n` returns the followings:
Kernel IP routing table
Destination   Gateway     Genmask       Flags Metric Ref Use Iface
192.168.1.0   0.0.0.0     255.255.255.0 U     0      0   0   engine
0.0.0.0       192.168.1.1 0.0.0.0       UG    0      0   0   engine
192.168.10.0  192.168.1.2 255.255.255.0 UG    0      0   0   engine

Then the last shown gateway of engine '192.168.1.2' would be returned
by this functions since it latest occurs in /proc/net/route.
This commit makes this function always return the default gateway of
each bridge interface as ovirt-engine expects.

Change-Id: I6bceb3be2495bc80931c709e2132ff4542e6b763
---
M vdsm/netinfo.py
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Douglas Schilling Landgraf: Verified; Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6bceb3be2495bc80931c709e2132ff4542e6b763
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Hunt Xu <mhuntxu at gmail.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Hunt Xu <mhuntxu at gmail.com>


More information about the vdsm-patches mailing list