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

[Mux] Add support for mux metrics to State DB #1757

Merged
merged 3 commits into from
Jun 4, 2021

Conversation

prsunny
Copy link
Collaborator

@prsunny prsunny commented May 25, 2021

What I did

Log mux state change events to State DB (required for bookkeeping)

127.0.0.1:6379[6]> HGETALL "MUX_METRICS_TABLE|Ethernet4"
1) "orch_switch_standby_start"
2) "2021-May-25 14:43:57.85633"
3) "orch_switch_standby_end"
4) "2021-May-25 14:43:57.93600"

Why I did it
Part of the overall mux metric changes by linkmgrd, xcvrd (PR)

How I verified it

Execute mux state change and verify the State DB tables.

Details if related

@lgtm-com
Copy link

lgtm-com bot commented May 25, 2021

This pull request introduces 1 alert when merging 9d80c9e into d1cd0fd - view on LGTM.com

new alerts:

  • 1 for Use of potentially dangerous function

@prsunny
Copy link
Collaborator Author

prsunny commented May 27, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny
Copy link
Collaborator Author

prsunny commented May 27, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny prsunny merged commit 5847af3 into sonic-net:master Jun 4, 2021
@prsunny prsunny deleted the state_db branch June 4, 2021 17:00
qiluo-msft pushed a commit that referenced this pull request Jun 11, 2021
* Update Mux metric table for orchagent switchover
* Add VS test to check metric fields
liat-grozovik pushed a commit to sonic-net/sonic-buildimage that referenced this pull request Jun 29, 2021
Advance submodule head for sonic-swss on 202012

bb383be2 [Dynamic Buffer Calc][Mellanox] Bug fixes and enhancements for the lua plugins for buffer pool calculation and headroom checking (sonic-net/sonic-swss#1781)
f949dfe9 [Dynamic Buffer Calc] Avoid creating lossy PG for admin down ports during initialization (sonic-net/sonic-swss#1776)
def0a914 Fix config prompt question issue (sonic-net/sonic-swss#1799)
21f97506 [ci]: Merge azure pipelines from master to 202012 branch (sonic-net/sonic-swss#1764)
a83a2a42 [vstest]: add dvs_route fixture
849bdf9c [Mux] Add support for mux metrics to State DB (sonic-net/sonic-swss#1757)
386de717 [qosorch] Dot1p map list initialization fix (sonic-net/sonic-swss#1746)
f99abdca [sub intf] Port object reference count update (sonic-net/sonic-swss#1712)
4a00042d [vstest/nhg]: use dvs_route fixture to make test_nhg more robust

Signed-off-by: Stephen Sun <stephens@nvidia.com>
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-swss that referenced this pull request Oct 5, 2021
* Update Mux metric table for orchagent switchover
* Add VS test to check metric fields
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.

4 participants