[OPNFLWPLUG-308] Also report (TCP) port number (for the OpenFlow connection) for switches Created: 14/Oct/14  Updated: 27/Sep/21  Resolved: 26/Jun/17

Status: Resolved
Project: OpenFlowPlugin
Component/s: General
Affects Version/s: None
Fix Version/s: None

Type: Improvement
Reporter: Colin Dixon Assignee: Jozef Bacigal
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: All
Platform: All

Issue Links:
Blocks
is blocked by OPNFLWPLUG-307 switch IP address not populated Verified

 Description   

Right now, the YANG model for switches has an IP address for the switch:
https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=blob;f=opendaylight/md-sal/model/model-flow-service/src/main/yang/flow-node-inventory.yang;h=fc0eb4cf7220c7428a9977f9eb0232e60750f045;hb=HEAD#l153

However, the IP address is not guaranteed to be unique—especially with mininet. We may want to change the model to also add a port in addition to the IP address so that we have a unique L3 way to address the switch.

 Comments   
Comment by Bhavesh Kumar [ 19/Jan/15 ]

As per my investigation so far, I believe there will be changes in the openflowplugin as well, as opendaylight communicates with the external switches with the help of OF protocol.

Comment by Thanh Ha (zxiiro) [ 21/Jan/15 ]

Gerrit patches are:

controller: https://git.opendaylight.org/gerrit/14318/
openflowplugin: https://git.opendaylight.org/gerrit/14317/

Comment by Michal Rehak [ 10/Nov/15 ]

openflowplugin (model from controller): https://git.opendaylight.org/gerrit/#/c/29484/

Comment by Michal Rehak [ 15/Mar/16 ]

Still the https://git.opendaylight.org/gerrit/14317/ is not merged due to conflicts. I vote for adding this to master.

Comment by Shuva Jyoti Kar [ 15/May/16 ]

re-raised a separate patch after resolving conflicts:

https://git.opendaylight.org/gerrit/38900

Comment by Miroslav Macko [ 03/Oct/16 ]

Hello Shuva,

What is the status for this please?

Thanks,
Miro

Comment by Tomas Slusny [ 22/Jun/17 ]

This was only merged for Helium design, but is is missing from Lithium design, so here is patch that solves this: https://git.opendaylight.org/gerrit/#/c/59383/

Comment by Tomas Slusny [ 26/Jun/17 ]

Patch was merged, closing

Generated at Wed Feb 07 20:32:07 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.