-
Notifications
You must be signed in to change notification settings - Fork 1.5k
[device/celestica]: Add xcvrd event support for Haliburton #6517
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
Conversation
This pull request introduces 6 alerts and fixes 5 when merging 9873730 into cad2025 - view on LGTM.com new alerts:
fixed alerts:
|
This pull request fixes 5 alerts when merging 3ecdfce into 6957e37 - view on LGTM.com fixed alerts:
|
This pull request introduces 1 alert and fixes 6 when merging 83cc619 into 98298f7 - view on LGTM.com new alerts:
fixed alerts:
|
This pull request introduces 1 alert and fixes 6 when merging c1671be into 8a024f0 - view on LGTM.com new alerts:
fixed alerts:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix LTGM warning: get_serial()
method is defined twice in chassis.py
This pull request fixes 6 alerts when merging 2c67713 into f77157f - view on LGTM.com fixed alerts:
|
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
This pull request fixes 6 alerts when merging c4626a8 into bb03e5b - view on LGTM.com fixed alerts:
|
#### Why I did it - The xcvrd service requires an event detection function, unplug or plug in the transceiver. #### How I did it - Add sysfs interrupt to notify userspace app of external interrupt - Implement get_change_event() in chassis api. - Also begin installing Python 3 sonic-platform package for Celestica platforms
…#6517) #### Why I did it - The xcvrd service requires an event detection function, unplug or plug in the transceiver. #### How I did it - Add sysfs interrupt to notify userspace app of external interrupt - Implement get_change_event() in chassis api. - Also begin installing Python 3 sonic-platform package for Celestica platforms
- Why I did it
- How I did it
- How to verify it
show interfaces status
show interfaces transceiver eeprom
Signed-off-by: Wirut Getbamrung [[email protected]]