Skip to content

[voq/inbandif] Support for inband port as regular port - #6477

Merged
abdosi merged 4 commits into
sonic-net:masterfrom
vganesan-nokia:voq-inbandif-port
Apr 1, 2021
Merged

[voq/inbandif] Support for inband port as regular port#6477
abdosi merged 4 commits into
sonic-net:masterfrom
vganesan-nokia:voq-inbandif-port

Conversation

@vganesan-nokia

@vganesan-nokia vganesan-nokia commented Jan 18, 2021

Copy link
Copy Markdown
Contributor

- Why I did it

Inband port can be made available in PORT table. But regular port handlngs are
not applicable for Inband port.

Changes in this PR are to make LLDP to consider Inband port and to avoid regular
port handling on Inband port.

Currently the Inband interface is used for VOQ chassis systems for asic to asic communications.
Ref: VOQ HLD https://github.com/Azure/SONiC/blob/master/doc/voq/voq_hld.md

- How I did it

  • Changes done in supervisor conf for LLDP to include Inband in the interface list
  • Changes done in platform common files to handle Inand port similar to how backplane port is handled

- How to verify it

  • Expose a port with name prefixed with "Inband" to SONiC via port_config.ini
  • Have an entry in the PORT table for this port with name "Inband"
  • When the system is booted, there should not be errors related to Inband interface
  • test_virtual_chassis.py which configures a front panel port as inband interface pass

- Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

This was referenced Jan 18, 2021
Merged
Merged
Merged
@anshuv-mfst
anshuv-mfst requested a review from abdosi January 27, 2021 17:10
@anshuv-mfst

anshuv-mfst commented Jan 27, 2021

Copy link
Copy Markdown

@eswaran (Arista) - could you please review the PR, thanks.

@anshuv-mfst

anshuv-mfst commented Jan 27, 2021

Copy link
Copy Markdown

@eswaran (Arista), @abdosi - could you please review the PR, thanks.

mlorrillere
mlorrillere reviewed Feb 3, 2021
View reviewed changes
Comment thread dockers/docker-lldp/lldpmgrd Outdated

@mlorrillere mlorrillere Feb 3, 2021

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should it be port_name.startswith('Inband') to match only on prefix?

@vganesan-nokia vganesan-nokia Feb 3, 2021

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As per Recycle port HLD PR discussion, it was recommended to use name starting with Ethernet. For inband interface, it will be Ethernet-Inband0. For recirc port it will be Ethernet-Recirc0. I'll change this to port_name.startswith("Ethernet-Inband") or use the inband_prefix() as recommended in next comment.

@vganesan-nokia vganesan-nokia Feb 6, 2021

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

mlorrillere
mlorrillere reviewed Feb 3, 2021
View reviewed changes
Comment thread dockers/docker-lldp/lldpmgrd Outdated

@mlorrillere mlorrillere Feb 3, 2021

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you use inband_prefix() from sonic_py_common/interface.py?

@vganesan-nokia vganesan-nokia Feb 3, 2021

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. I'll fix this.

@vganesan-nokia vganesan-nokia Feb 6, 2021

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

vedganes added 4 commits February 17, 2021 14:24
vedganes
[voqinband]Support for inband port as regular port
b87dba8 
Signed-off-by: vedganes <vedavinayagam.ganesan@nokia.com>

Inband port is avaialable in PORT table. But regular port handlings are
not applicable for Inband port.
Changes are to make lldp to consider Inband port and to avoid regular
port handling on Inband port.
vedganes
[inbandif]Inband if support - code review comments fix -1
68eecee 
Signed-off-by: vedganes <vedavinayagam.ganesan@nokia.com>

Fixed code review comments
vedganes
[inbandif] Avoid lldp on inband interface
d9e446f 
Signed-off-by: vedganes <vedavinayagam.ganesan@nokia.com>

Changes to avoid LLDP config on Inband interface. Since port type inband
interfaces are recycle ports (or may normal ports) used for asic-to-asic
communication, LLDP is not configured on these interfaces.
vedganes
[voq/inband]Code review comments fix - 2
b513a1e 
Signed-off-by: vedganes <vedavinayagam.ganesan@nokia.com>

(1) Based on review comments Recyle port HLD
(sonic-net/SONiC#742) the inband port name prefix is
changed from Inband to "Ethernet-IB" similar to what we have for
Ethernet-BP (Ethernet-Backplane) in multi-asic design. Changes are done
in interface.py to handle this changed name
(2) Code review comments fix for lldpmgrd
@vganesan-nokia

vganesan-nokia commented Feb 17, 2021

Copy link
Copy Markdown
Contributor Author

/Azurepipelines run

@azure-pipelines

azure-pipelines Bot commented Feb 17, 2021

Copy link
Copy Markdown
Commenter does not have sufficient privileges for PR 6477 in repo Azure/sonic-buildimage

@abdosi

abdosi commented Mar 7, 2021

Copy link
Copy Markdown
Contributor

/Azurepipelines run

@azure-pipelines

azure-pipelines Bot commented Mar 7, 2021

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@vganesan-nokia

vganesan-nokia commented Mar 9, 2021

Copy link
Copy Markdown
Contributor Author

@abdosi, thanks for starting the Azurepipeline runs. Would you please complete review and approve so that we can merge? The PRs sonic-net/sonic-platform-common#159, sonic-net/sonic-platform-daemons#145 and sonic-net/sonic-utilities#1363 which you have already approved depend on this PR for build success.

@abdosi
abdosi merged commit 973affc into sonic-net:master Apr 1, 2021
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-buildimage that referenced this pull request May 23, 2021
@vganesan-nokia @raphaelt-nvidia
[voq/inbandif] Support for inband port as regular port (sonic-net#6477)
9bce4fc 
Changes in this PR are to make LLDP to consider Inband port and to avoid regular
port handling on Inband port.
@ysmanman ysmanman mentioned this pull request Jun 23, 2021
Merged
4 tasks
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
@vganesan-nokia Carl Keene
[voq/inbandif] Support for inband port as regular port (sonic-net#6477)
75c1efa 
Changes in this PR are to make LLDP to consider Inband port and to avoid regular
port handling on Inband port.
xdqi pushed a commit to canonical/sonic-buildimage that referenced this pull request Jul 6, 2026
@vganesan-nokia
[voq/inbandif] Support for inband port as regular port (sonic-net#6477)
a417fd0 
Changes in this PR are to make LLDP to consider Inband port and to avoid regular
port handling on Inband port.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@abdosi abdosi abdosi approved these changes
@lguohan lguohan Awaiting requested review from lguohan lguohan is a code owner
+1 more reviewer
@mlorrillere mlorrillere mlorrillere approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@vganesan-nokia @anshuv-mfst @abdosi @mlorrillere