Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[sonic-py-common] Add recirc_prefix definition #9471

Merged
merged 1 commit into from
Dec 13, 2021

Conversation

Staphylo
Copy link
Collaborator

@Staphylo Staphylo commented Dec 8, 2021

Why I did it

This interface type is used for recirculation on chassis.
The definition is required to prevent this interface from being
considered a physical interface in sonic-platform-common and sonic-platform-daemon

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Required for chassis work

Description for the changelog

Add recirc_prefix definition in sonic_py_common

This interface type is used for recirculation on chassis.
The definition is required to prevent this interface from being
considered a physical interface in sonic-platform-common and
sonic-platform-daemon
@judyjoseph judyjoseph merged commit 6cf0663 into sonic-net:master Dec 13, 2021
@judyjoseph judyjoseph added the Request for 202111 Branch For PRs being requested for 202111 branch label Dec 13, 2021
sujinmkang pushed a commit to sonic-net/sonic-platform-daemons that referenced this pull request Dec 17, 2021
Description
Prevent ledd crash when an event on recirc port happens

Motivation and Context
A recirculation port is not a physical interface tied to a xcvr
Calling the led change event on such a port could lead to a ledd crash

How Has This Been Tested?
This change was applied to running switch with its dependency sonic-net/sonic-buildimage#9471
Verified that ledd runs without crashing
judyjoseph pushed a commit that referenced this pull request Dec 27, 2021
This interface type is used for recirculation on chassis.
The definition is required to prevent this interface from being
considered a physical interface in sonic-platform-common and
sonic-platform-daemon
judyjoseph pushed a commit to sonic-net/sonic-platform-daemons that referenced this pull request Jan 23, 2022
Description
Prevent ledd crash when an event on recirc port happens

Motivation and Context
A recirculation port is not a physical interface tied to a xcvr
Calling the led change event on such a port could lead to a ledd crash

How Has This Been Tested?
This change was applied to running switch with its dependency sonic-net/sonic-buildimage#9471
Verified that ledd runs without crashing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Included in 202111 Branch Request for 202111 Branch For PRs being requested for 202111 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants