Skip to content

[muxcable][config] add CLI support for mux mode detach#2425

Merged
zjswhhh merged 1 commit intosonic-net:masterfrom
zjswhhh:mux_mode_AA
Oct 7, 2022
Merged

[muxcable][config] add CLI support for mux mode detach#2425
zjswhhh merged 1 commit intosonic-net:masterfrom
zjswhhh:mux_mode_AA

Conversation

@zjswhhh
Copy link
Copy Markdown
Contributor

@zjswhhh zjswhhh commented Oct 5, 2022

What I did

Add support for config mux mode detach. Stemming from sonic-net/sonic-linkmgrd#79

sign-off: Jing Zhang zhangjing@microsoft.com

How I did it

Add detach to the choice list.

How to verify it

  • Existing unit test.
  • Tested on DUT
admin@svcstr-7050-acs-1:~$ sudo config mux mode detach Ethernet12
port        state
----------  -------
Ethernet12  OK

After config change, shutdown peer's link, self side got peer link prober unknown event but didn't toggle peer's mux state as it's in Detached mode.

Oct  5 23:47:03.510501 svcstr-7050-acs-1 WARNING mux#linkmgrd: link_manager/LinkManagerStateMachineActiveActive.cpp:524 handlePeerStateChange: Ethernet12: Received peer link prober event, new state: PeerUnknown

A auto mode interface would toggle:

Oct  5 23:46:21.472932 svcstr-7050-acs-1 WARNING mux#linkmgrd: link_manager/LinkManagerStateMachineActiveActive.cpp:524 handlePeerStateChange: Ethernet8: Received peer link prober event, new state: PeerUnknown
Oct  5 23:46:21.473244 svcstr-7050-acs-1 WARNING mux#linkmgrd: link_manager/LinkManagerStateMachineActiveActive.cpp:909 switchPeerMuxState: Ethernet8: Switching peer MUX state to 'Standby'
Oct  5 23:46:21.552629 svcstr-7050-acs-1 WARNING mux#linkmgrd: link_manager/LinkManagerStateMachineActiveActive.cpp:323 handlePeerMuxStateNotification: Ethernet8: state db mux state: Standby

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

@zjswhhh zjswhhh requested review from vdahiya12 and yxieca October 5, 2022 23:40
@zjswhhh zjswhhh merged commit 4237794 into sonic-net:master Oct 7, 2022
EdenGri pushed a commit to EdenGri/sonic-utilities that referenced this pull request Oct 12, 2022
What I did
Add support for config mux mode detach. Stemming from sonic-net/sonic-linkmgrd#79

sign-off: Jing Zhang zhangjing@microsoft.com

How I did it
Add detach to the choice list.

How to verify it
Existing unit test.
Tested on DUT
mdanish-kh pushed a commit to hamnarauf/sonic-utilities that referenced this pull request Oct 22, 2022
What I did
Add support for config mux mode detach. Stemming from sonic-net/sonic-linkmgrd#79

sign-off: Jing Zhang zhangjing@microsoft.com

How I did it
Add detach to the choice list.

How to verify it
Existing unit test.
Tested on DUT
yxieca pushed a commit that referenced this pull request Oct 25, 2022
What I did
Add support for config mux mode detach. Stemming from sonic-net/sonic-linkmgrd#79

sign-off: Jing Zhang zhangjing@microsoft.com

How I did it
Add detach to the choice list.

How to verify it
Existing unit test.
Tested on DUT
preetham-singh pushed a commit to preetham-singh/sonic-utilities that referenced this pull request Nov 21, 2022
What I did
Add support for config mux mode detach. Stemming from sonic-net/sonic-linkmgrd#79

sign-off: Jing Zhang zhangjing@microsoft.com

How I did it
Add detach to the choice list.

How to verify it
Existing unit test.
Tested on DUT
@zjswhhh zjswhhh deleted the mux_mode_AA branch February 1, 2023 22:38
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