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.

Sign up for GitHub

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

Jump to bottom

port is not brought up after init #130

Closed
lguohan opened this issue Nov 10, 2016 · 3 comments
Closed

port is not brought up after init #130

lguohan opened this issue Nov 10, 2016 · 3 comments
Assignees
@stcheng

Comments

@lguohan
Copy link
Contributor

lguohan commented Nov 10, 2016

The port is not brought up after init. In the sai api record, it sets the port admin state to false in the end.

sairedis.2016-11-10.03_35_51.976755.txt
@kcudnik
Copy link
Contributor

kcudnik commented Nov 10, 2016

All ports here are set to false

@lguohan
Copy link
Contributor Author

lguohan commented Nov 10, 2016

yes, i suspect there is bug in sonic-swss that did not setup them to true. @stcheng

@stcheng stcheng self-assigned this Mar 24, 2017
@stcheng
Copy link
Contributor

stcheng commented Mar 24, 2017

sonic-net/sonic-buildimage#381 with this pull request the issue is not seen for now.

@stcheng stcheng closed this as completed Mar 24, 2017
EdenGri pushed a commit to EdenGri/sonic-swss that referenced this issue Feb 28, 2022
[portstat]: Combine ADMIN and OPER fields (sonic-net#130)
9913106 
- Move back to one column of port state to be compatible with old parsers
- The current state field has four choices: U, D, X, and N/A.

  U means the port is admin + oper status UP
  D means the port is admin UP but oper DOWN
  X means the port is admin DOWN and disabled
  N/A for the rest abnormal scenarios

- Refactor the code

Signed-off-by: Shu0T1an ChenG <[email protected]>
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-swss that referenced this issue Mar 1, 2023
@kcudnik
Add primitive cli (sonic-net#130)
96bcb63 
* Add primitive cli

* Addressing comments
lukasstockner pushed a commit to genesiscloud/sonic-swss that referenced this issue Apr 2, 2023
@jleveque
[ChassisBase][ModuleBase] Remove redundant 'get_serial_number()' meth…
ec9082e 
…od (sonic-net#130)

ChassisBase and ModuleBase inherit from the DeviceBase class, which already provides a get_serial() method. Thus, these methods were redundant.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stcheng stcheng

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@lguohan @stcheng @kcudnik