[BGPCEP-782] Different peer-id on different versions Created: 03/Apr/18  Updated: 14/Jun/18  Resolved: 04/Apr/18

Status: Verified
Project: bgpcep
Component/s: BGP
Affects Version/s: Fluorine
Fix Version/s: Fluorine

Type: Bug Priority: Medium
Reporter: Tomas Markovic Assignee: Claudio David Gasparini
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: File conf_peer.py     File default.cfg     File get_ipv6_topology.sh    

 Description   

Overview peer-id in full rib is different on oxygen and less, where router-id is taken instead of neighbor address/local-address from exabgp. On flurine there is neighbor address as it should be (imo).

Steps to reproduce

  • install:feature odl-restconf odl-bgpcep-bgp odl-bgpcep-bgp-config-example
  • conf_peer.py (configures internal neighbor ::1)
  • env exabgp.tcp.port=1790 exabgp --debug default.cfg
    (starts exabgp with one ipv6 route)

In case of <=oxygen peer-id is set to 1.2.3.4 from router-id from exabgp config, even though there is no neighbor configured at that address.
Fluorine correctly assigns peer-id as ipv6 as is neighbor configuration and local-address in exabgp config.

fluorine output from full-rib

{
          "peer-id": "bgp://::1",
          "peer-role": "ibgp",
          "adj-rib-in": {
            "tables": [
              {

output from oxygen

{
          "peer-id": "bgp://1.2.3.4",
          "peer-role": "ibgp",
          "adj-rib-in": {
            "tables": [
              {

routes are the same in both cases, and are present in ipv6 topology as well.



 Comments   
Comment by Claudio David Gasparini [ 03/Apr/18 ]

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

Comment by Claudio David Gasparini [ 03/Apr/18 ]

Could you test the fix and close the bug.

 

Regards, 

Comment by Tomas Markovic [ 04/Apr/18 ]

does not work on fluorine https://jenkins.opendaylight.org/sandbox/job/tomas-bgpcep-csit-1node-userfeatures-all-fluorine/11/

Comment by Tomas Markovic [ 04/Apr/18 ]

It now takes router-id even on fluorine

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