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

[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
Labels

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 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
Kalimuthu-Velappan pushed a commit to Kalimuthu-Velappan/sonic-buildimage that referenced this issue Sep 12, 2019
madhanmellanox pushed a commit to madhanmellanox/sonic-buildimage that referenced this issue Mar 23, 2020
…net#456)

* [countercheck]: move pfc counter alerting to countercheckorch

Signed-off-by: Sihui Han <[email protected]>

* remove no need header

Signed-off-by: Sihui Han <[email protected]>
tahmed-dev added a commit to tahmed-dev/sonic-buildimage that referenced this issue Feb 18, 2021
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 <[email protected]>
tahmed-dev added a commit that referenced this issue Feb 18, 2021
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 <[email protected]>
daall pushed a commit that referenced this issue Feb 25, 2021
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 <[email protected]>
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this issue Aug 7, 2021
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 <[email protected]>
mssonicbld added a commit that referenced this issue Apr 18, 2024
…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
Labels
Projects
None yet
Development

No branches or pull requests

3 participants