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

[ChassisBase][ModuleBase] Remove redundant 'get_serial_number()' method #130

Merged
merged 1 commit into from
Oct 14, 2020
Merged

[ChassisBase][ModuleBase] Remove redundant 'get_serial_number()' method #130

merged 1 commit into from
Oct 14, 2020

Conversation

jleveque
Copy link
Contributor

ChassisBase and ModuleBase inherit from the DeviceBase class, which already provides a get_serial() method. Thus, these methods were redundant.

@jleveque jleveque self-assigned this Oct 14, 2020
@jleveque jleveque requested a review from keboliu October 14, 2020 02:20
jleveque added a commit to sonic-net/sonic-mgmt that referenced this pull request Oct 14, 2020
…t get_serial_number() methods (#2343)

No longer test deprecated get_serial_number() methods of Chassis and Module classes

The get_serial_number() methods are redundant and are being removed in sonic-net/sonic-platform-common#130
@jleveque jleveque merged commit ec9082e into sonic-net:master Oct 14, 2020
@jleveque jleveque deleted the add_comment branch October 14, 2020 18:10
mprabhu-nokia pushed a commit to mprabhu-nokia/sonic-platform-common that referenced this pull request Oct 21, 2020
…od (sonic-net#130)

ChassisBase and ModuleBase inherit from the DeviceBase class, which already provides a get_serial() method. Thus, these methods were redundant.
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-platform-common that referenced this pull request Oct 25, 2024
…alizeGlobalConfig (sonic-net#130)

The check for multiAsic before calling initializeGlobalConfig was done in xcvrd earlier. 
Adding now to the other processes in sonic-platform-daemons as well.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants