[BGPCEP-284] Routes to be withdrawn ignored when UPDATE contains both WITHDRAWN ROUTES and NLRI fields. Created: 05/Oct/15  Updated: 03/Mar/19  Resolved: 20/Oct/15

Status: Resolved
Project: bgpcep
Component/s: BGP
Affects Version/s: Bugzilla Migration
Fix Version/s: Bugzilla Migration

Type: Bug
Reporter: Radovan Sajben Assignee: Claudio David Gasparini
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: All
Platform: All

Attachments: Text File Unexpected error while handling negotiation message Keepalive.txt     Text File karaf.log.txt    
External issue ID: 4409

 Description   

First UPDATE advertised a feasible routes (8.0.1.0/28, 8.0.1.16/28) in NLRI variable. Second UPDATE announced next two routes in NLRI and the 8.0.1.0/28 as to be withdrawn from service in WITHDRAWN ROUTES variable. The withdrawal is not processed but it shall be (RFC 4271, chapter 4.3, page 20). For details see the attached karaf.log, please.

robot/report/log after the 8.0.1.0/28 withdrawal - single UPDATE:
Count prefixes in {"topology":[{"topology-id":"example-ipv4-topology","topology-types":{},"node":[{"node-id":"192.0.2.1","l3-unicast-igp-topology:igp-node-attributes":{"prefix":[

{"prefix":"8.0.1.48/28"}

,

{"prefix":"8.0.1.16/28"}

,

{"prefix":"8.0.1.0/28"}

,

{"prefix":"8.0.1.32/28"}

]}}],"server-provided":true}]}.

Reproducible for the distribution-karaf-0.3.1-Lithium-SR1.zip as well for the latest distribution-karaf-0.3.2-20151003.125909-114.zip.

Note: the same scenario using dedicated UPDATES for NLRI and WITHDRAWN ROUTES variables works fine:
robot/report/log after the 8.0.1.0/28 withdrawal - dedicated UPDATEs:
Count prefixes in {"topology":[{"topology-id":"example-ipv4-topology","topology-types":{},"node":[{"node-id":"192.0.2.1","l3-unicast-igp-topology:igp-node-attributes":{"prefix":[

{"prefix":"8.0.1.48/28"}

,

{"prefix":"8.0.1.16/28"}

,

{"prefix":"8.0.1.32/28"}

]}}],"server-provided":true}]}.

 Comments   
Comment by Radovan Sajben [ 05/Oct/15 ]

Attachment karaf.log.txt has been added with description: log from controller

Comment by Claudio David Gasparini [ 06/Oct/15 ]

https://git.opendaylight.org/gerrit/#/c/27951/

Comment by Claudio David Gasparini [ 08/Oct/15 ]

https://git.opendaylight.org/gerrit/#/c/28052/ Lithium

Comment by Radovan Sajben [ 20/Oct/15 ]

stable/lithium
==============

Verified on distribution-karaf-0.3.3-20151020.060748-43.zip using bgpcep functional test suite bug_4409 (https://git.opendaylight.org/gerrit/#/c/27191/50)

==============================================================================
bgpcep-userfeatures.txt.Bgpuser.BGPCEP-284 :: Test to cover the bgpcep bug_44...
==============================================================================
Set Karaf Log Levels :: Set Karaf log level | PASS |
------------------------------------------------------------------------------
Check_For_Empty_Topology_Before_Talking :: Sanity check example-ip... | PASS |
------------------------------------------------------------------------------
Reconfigure_ODL_To_Accept_Connection :: Configure BGP peer module ... | PASS |
------------------------------------------------------------------------------
Start_Talking_BGP_speaker :: Start Python speaker. Update messages... | PASS |
------------------------------------------------------------------------------
Wait_For_Talking_Topology :: Wait until example-ipv4-topology beco... | PASS |
------------------------------------------------------------------------------
Check_Talking_Topology_Count :: Count the routes in example-ipv4-t... | PASS |
------------------------------------------------------------------------------
Kill_Talking_BGP_Speaker :: Abort the Python speaker. Also, attemp... | PASS |
------------------------------------------------------------------------------
Check_For_Empty_Topology_After_Talking :: See example-ipv4-topolog... | PASS |
------------------------------------------------------------------------------
Start_Talking_BGP_speaker_2 :: Start Python speaker. Update messag... | PASS |
------------------------------------------------------------------------------
Wait_For_Talking_Topology_2 :: Wait until example-ipv4-topology be... | PASS |
------------------------------------------------------------------------------
Check_Talking_Topology_Count_2 :: Count the routes in example-ipv4... | PASS |
------------------------------------------------------------------------------
Kill_Talking_BGP_Speaker_2 :: Abort the Python speaker. Also, atte... | PASS |
------------------------------------------------------------------------------
Check_For_Empty_Topology_After_Talking_2 :: Post-condition: Check ... | PASS |
------------------------------------------------------------------------------
Delete_Bgp_Peer_Configuration :: Revert the BGP configuration to t... | PASS |
------------------------------------------------------------------------------
bgpcep-userfeatures.txt.Bgpuser.BGPCEP-284 :: Test to cover the bgpc... | PASS |
5 critical tests, 5 passed, 0 failed
14 tests total, 14 passed, 0 failed

Comment by Radovan Sajben [ 20/Oct/15 ]

Attachment Unexpected error while handling negotiation message Keepalive.txt has been added with description: Unexpected error while handling negotiation message Keepalive

Comment by Radovan Sajben [ 20/Oct/15 ]

beryllium
=========

BLOCKED verification (see the attached part of karaf log Unexpected error while handling negotiation message Keepalive) on distribution-karaf-0.4.0-20151020.061531-1721.zip using bgpcep functional test suite bug_4409 (https://git.opendaylight.org/gerrit/#/c/27191/50)

Generated at Wed Feb 07 19:12:34 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.