sonic-swss icon indicating copy to clipboard operation
sonic-swss copied to clipboard

[muxorch] Adding case for maintaining current state

Open Ndancejic opened this issue 2 years ago • 12 comments

What I did: Added a case where if current and previous Mux states are the same, post a log entry that acknowleges this change and returns without doing anything.

Why I did it: Valid state changes such as active-active and standby-standby were causing error logs

How I tested it: added a case to check active-active state change and standby-standby state change.

Signed-off-by: Nikola Dancejic [email protected]

Ndancejic avatar May 17 '22 22:05 Ndancejic

@Ndancejic , seems like there are some VS failures due to the change:

test_mux.py::TestMuxTunnel::test_Tunnel PASSED                           [ 50%]
test_mux.py::TestMuxTunnel::test_Peer PASSED                             [ 50%]
test_mux.py::TestMuxTunnel::test_Neighbor FAILED                         [ 51%]
test_mux.py::TestMuxTunnel::test_Fdb FAILED                              [ 51%]
test_mux.py::TestMuxTunnel::test_Route FAILED                            [ 51%]
test_mux.py::TestMuxTunnel::test_acl FAILED                              [ 51%]

prsunny avatar May 18 '22 23:05 prsunny

/azp run

Ndancejic avatar Aug 29 '22 19:08 Ndancejic

Commenter does not have sufficient privileges for PR 2280 in repo sonic-net/sonic-swss

azure-pipelines[bot] avatar Aug 29 '22 19:08 azure-pipelines[bot]

/azpw run

Ndancejic avatar Aug 29 '22 19:08 Ndancejic

/AzurePipelines run

mssonicbld avatar Aug 29 '22 19:08 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Aug 29 '22 19:08 azure-pipelines[bot]

/azpw run

Ndancejic avatar Aug 30 '22 18:08 Ndancejic

/AzurePipelines run

mssonicbld avatar Aug 30 '22 18:08 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Aug 30 '22 18:08 azure-pipelines[bot]

/azpw run

Ndancejic avatar Sep 06 '22 18:09 Ndancejic

/AzurePipelines run

mssonicbld avatar Sep 06 '22 18:09 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Sep 06 '22 18:09 azure-pipelines[bot]

/azpw run

Ndancejic avatar Nov 18 '22 19:11 Ndancejic

/AzurePipelines run

mssonicbld avatar Nov 18 '22 19:11 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Nov 18 '22 19:11 azure-pipelines[bot]