Hi, Ishida-san

Thanks for your response.
I’ve checked your patch in my test environment.
But it looks some problem is not resolved.

(1) need for deletion as bellow

diff --git a/ryu/services/protocols/bgp/bmp.py 
b/ryu/services/protocols/bgp/bmp.py
index 4fbbecb..d4b4b4a 100644
--- a/ryu/services/protocols/bgp/bmp.py
+++ b/ryu/services/protocols/bgp/bmp.py
@@ -202,7 +199,6 @@ class BMPClient(Activity):
             self._send(msg)

             for path in peer._adj_rib_in.itervalues():
-                update_msg = peer._construct_update(path)
                 msg = self._construct_route_monitoring(peer, path)
                 self._send(msg)


(2) It looks "Route Monitoring mesages" of ADJ-RIB-in is incorrect

 ( please check the attach )


Attachment: Report_Technical_Issue_bmp.pdf
Description: Adobe PDF document

> 2015/01/09 14:47、ISHIDA Wataru <[email protected]> のメール:
> 
> Hi, Tsuboi-san
> 
> I've reproduced the problem and sent a patch to fix this.
> Could you check it?
> 
> Thanks
> 
> On 2015/01/05 21:00, Toshiki Tsuboi wrote:
>> Hi,Team
>> 
>> I’ve found the issue in bmp.py.
>>  In spite of receiving a update message from eBGP router, Ryu BMP client 
>> can’t send "Route Monitoring messages” of ADJ-RIBs-in
>> to BMP server,
>> It looks BMP client always send "Route Monitoring messages” of ADJ-RIBs-out.
>> Therefore, BMP Server can’t understand the next-hop as peering eBGP router, 
>> in case of next-hop-self mode in BGPSpeaker.
>> 
>> Could you please fix this issue in bmp.py.
>> 
>> Best regards.
>> Toshiki Tsuboi
>> 
>> 
>> 
>> 
>> ------------------------------------------------------------------------------
>> Dive into the World of Parallel Programming! The Go Parallel Website,
>> sponsored by Intel and developed in partnership with Slashdot Media, is your
>> hub for all things parallel software development, from weekly thought
>> leadership blogs to news, videos, case studies, tutorials and more. Take a
>> look and join the conversation now. http://goparallel.sourceforge.net
>> _______________________________________________
>> Ryu-devel mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/ryu-devel
>> 
> 

------------------------------------------------------------------------------
Dive into the World of Parallel Programming! The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net
_______________________________________________
Ryu-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ryu-devel

Reply via email to