On Tue, 7 Feb 2017 13:40:36 +0900
Iwase Yusuke <[email protected]> wrote:

> Hi Albert,
> 
> 
> diff --git a/ryu/services/protocols/bgp/peer.py 
> b/ryu/services/protocols/bgp/peer.py
> index 8bf96d6..d380e30 100644
> --- a/ryu/services/protocols/bgp/peer.py
> +++ b/ryu/services/protocols/bgp/peer.py
> @@ -1622,7 +1622,7 @@ class Peer(Source, Sink, NeighborConfListener, 
> Activity):
>  
>          aspath = umsg_pattrs.get(BGP_ATTR_TYPE_AS_PATH)
>          # Check if AS_PATH has loops.
> -        if aspath.has_local_as(self.local_as):
> +        if aspath.has_local_as(self.local_as, 
> max_count=self._common_conf.allow_local_as_in_count):
>              LOG.error('Update message AS_PATH has loops. Ignoring this'
>                        ' UPDATE. %s', update_msg)
>              return
> 
> The above seems to exceed the max line characters (>80).
> I haven't investigated why pep8 does not complain it, though...
> 
> 
> Other changes looks good to me!

seems that travis-ci pep8 doesn't complain about the above. So I've
pushed this patch.

Thanks guys!

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Ryu-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ryu-devel

Reply via email to