Skip to content

Commit

Permalink
ovs-save: Handle different 'ip addr show' output.
Browse files Browse the repository at this point in the history
On RHEL 7.4 (with iproute-3.10.0-87), a DHCP provided
ipv4 address has the "dynamic" keyword set. For e.g
"ip addr show breth0 | grep inet" shows:

    inet 10.116.248.91/20 brd 10.116.255.255 scope global dynamic breth0
    inet6 fe80::250:56ff:fea8:fdf0/64 scope link

The keyword "dynamic" (according to 'man ip-address') is only
used for ipv6, but in this case this is not true. Our current
code will skip the ipv4 address restoration because of this.

With this commit, we special case "dynamic" keyword to be valid
in case of ipv4.

VMware-BZ: #1982196
Signed-off-by: Gurucharan Shetty <guru@ovn.org>
Acked-by: Yi-Hung Wei <yihung.wei@gmail.com>
  • Loading branch information
shettyg committed Oct 20, 2017
1 parent a95c710 commit 394d4c2
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions utilities/ovs-lib.in
Expand Up @@ -329,6 +329,14 @@ move_ip_address () {
while test $# != 0; do
case $1 in
dynamic)
# XXX: According to 'man ip-address', "dynamic" is only
# used for ipv6 addresses. But, atleast on RHEL 7.4
# (iproute-3.10.0-87), it is being used for ipv4
# addresses assigned with dhcp.
if [ "$family" = "inet" ]; then
shift
continue
fi
# Omit kernel-maintained route.
continue 2
;;
Expand Down

0 comments on commit 394d4c2

Please sign in to comment.