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

[swss] 'oper_status' attribute can have a value of '2'? #456

Closed
jleveque opened this issue Mar 30, 2017 · 3 comments
Closed

[swss] 'oper_status' attribute can have a value of '2'? #456

jleveque opened this issue Mar 30, 2017 · 3 comments
Assignees
@stcheng
Labels
Bug 🐛

Comments

@jleveque
Copy link
Contributor

The 'oper_status' attribute of a port in our DB has three potential values: 'up', 'down' and '2'.

Can reproduce on a T0 setup. Simply 'ifdown' a port that's part of the VLAN. Reading the database rapidly, you will see the 'oper_status' change from 'up' to 'down' and then quickly to '2', where it will stay.

Likewise, bringing the port back up using 'ifup', you will see the 'oper_status' change from '2' to 'down' and then to 'up'.
@stcheng stcheng self-assigned this Apr 16, 2017
@stcheng
Copy link
Contributor

stcheng commented Apr 16, 2017

This happens because currently both portsyncd and orchagent are updating this field simultaneously. When a port oper_status changes, ASIC will send port_status_change notification as well as kernel will send netlink message. Because in orchagent code, there is no mapping between the enum and the string, 2 is not converted to 'down' and it causes this bug.

In order to fix this issue, only portsyncd's update logic is needed to override the current status of a port in APP_DB.

@stcheng stcheng mentioned this issue Apr 16, 2017
Merged
@stcheng stcheng changed the title 'oper_status' attribute can have a value of '2'? [swss] 'oper_status' attribute can have a value of '2'? Apr 16, 2017
@lguohan
Copy link
Collaborator

lguohan commented Apr 22, 2017

is this resolved?

@stcheng
Copy link
Contributor

stcheng commented Apr 22, 2017

yes. resolved. i'll update the swss submodule.

@stcheng stcheng closed this as completed Apr 22, 2017
Open
Kalimuthu-Velappan pushed a commit to Kalimuthu-Velappan/sonic-buildimage that referenced this issue Sep 12, 2019
@ramachandrareddygaddam @jleveque
Fix for KeyError: 'DEVICE_NEIGHBOR' when executing 'show interfaces n…
84aafb6 
…eighbor expected' command (sonic-net#456)
madhanmellanox pushed a commit to madhanmellanox/sonic-buildimage that referenced this issue Mar 23, 2020
@sihuihan88
[countercheck]: move pfc counter alerting to countercheckorch (sonic-…
8995b30 
…net#456)

* [countercheck]: move pfc counter alerting to countercheckorch

Signed-off-by: Sihui Han <sihan@microsoft.com>

* remove no need header

Signed-off-by: Sihui Han <sihan@microsoft.com>
tahmed-dev added a commit to tahmed-dev/sonic-buildimage that referenced this issue Feb 18, 2021
@tahmed-dev
[sonic-swss-common] Submodule Update
ef74455 
Change in this update:
    b75aab7 [swss-common] Add LINKMGR CFG and MUX LINKMGR state table names (sonic-net#421)
    4a77d1c [ci]: add vstest (sonic-net#459)
    07258a6 [ci]: use build template (sonic-net#457)
    ddcae3e runRedisScript api to process integer returned by script run in the redis (sonic-net#447)
    33d89c7 [systemlag] Schema defs for system lag (sonic-net#448)
    af01f37 spell check fixes (sonic-net#456)
    7afd43d Update to make getNamespaces() API at par with the get_ns_list() swssdk-py API. (sonic-net#455)

signed-off-by: Tamer Ahmed <tamer.ahmed@microsoft.com>
@tahmed-dev tahmed-dev mentioned this issue Feb 18, 2021
Merged
4 tasks
tahmed-dev added a commit that referenced this issue Feb 18, 2021
@tahmed-dev
[sonic-swss-common] Submodule Update (#6812)
2fc37dd 
Change in this update:
    b75aab7 [swss-common] Add LINKMGR CFG and MUX LINKMGR state table names (#421)
    4a77d1c [ci]: add vstest (#459)
    07258a6 [ci]: use build template (#457)
    ddcae3e runRedisScript api to process integer returned by script run in the redis (#447)
    33d89c7 [systemlag] Schema defs for system lag (#448)
    af01f37 spell check fixes (#456)
    7afd43d Update to make getNamespaces() API at par with the get_ns_list() swssdk-py API. (#455)

signed-off-by: Tamer Ahmed <tamer.ahmed@microsoft.com>
daall pushed a commit that referenced this issue Feb 25, 2021
@tahmed-dev @daall
[sonic-swss-common] Submodule Update (#6812)
08bf372 
Change in this update:
    b75aab7 [swss-common] Add LINKMGR CFG and MUX LINKMGR state table names (#421)
    4a77d1c [ci]: add vstest (#459)
    07258a6 [ci]: use build template (#457)
    ddcae3e runRedisScript api to process integer returned by script run in the redis (#447)
    33d89c7 [systemlag] Schema defs for system lag (#448)
    af01f37 spell check fixes (#456)
    7afd43d Update to make getNamespaces() API at par with the get_ns_list() swssdk-py API. (#455)

signed-off-by: Tamer Ahmed <tamer.ahmed@microsoft.com>
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this issue Aug 7, 2021
@tahmed-dev
[sonic-swss-common] Submodule Update (sonic-net#6812)
1cc6850 
Change in this update:
    b75aab7 [swss-common] Add LINKMGR CFG and MUX LINKMGR state table names (sonic-net#421)
    4a77d1c [ci]: add vstest (sonic-net#459)
    07258a6 [ci]: use build template (sonic-net#457)
    ddcae3e runRedisScript api to process integer returned by script run in the redis (sonic-net#447)
    33d89c7 [systemlag] Schema defs for system lag (sonic-net#448)
    af01f37 spell check fixes (sonic-net#456)
    7afd43d Update to make getNamespaces() API at par with the get_ns_list() swssdk-py API. (sonic-net#455)

signed-off-by: Tamer Ahmed <tamer.ahmed@microsoft.com>
mssonicbld added a commit that referenced this issue Apr 18, 2024
@mssonicbld
[submodule] Update submodule sonic-platform-daemons to the latest HEA…
a5281fb 
…D automatically (#18707)

#### Why I did it
src/sonic-platform-daemons
```
* 0f61e15 - (HEAD -> master, origin/master, origin/HEAD) [chassis][linecard] Fix Module LINECARD<> went off-line message for empty slot issue (#462) (8 hours ago) [Marty Y. Lok]
* 25e22cd - Support to get MEDIA_SETTING and OPTICS_SI from both platform folder and HWSKU folder (#456) (21 hours ago) [Kebo Liu]
* 805c76a - [xcvrd] Ignore processing ports that are not front-panel (#455) (21 hours ago) [Vivek]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stcheng stcheng

Labels
Bug 🐛
Projects
None yet
Milestone
No milestone
Development

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