Skip to content

LogAnalyzer: Ignore a non-functional error thrown from Linkmgrd#19099

Merged
StormLiangMS merged 1 commit into
sonic-net:masterfrom
dhanasekar-arista:fix_msft_issue_624
Jul 1, 2025
Merged

LogAnalyzer: Ignore a non-functional error thrown from Linkmgrd#19099
StormLiangMS merged 1 commit into
sonic-net:masterfrom
dhanasekar-arista:fix_msft_issue_624

Conversation

@dhanasekar-arista

@dhanasekar-arista dhanasekar-arista commented Jun 19, 2025

Copy link
Copy Markdown
Contributor

Description of PR

Whenever CFG_MUX_CABLE_TABLE_NAME is updated we get the following error,

2025 Jun 15 22:30:03.459694 gd272 ERR mux#linkmgrd: MuxManager.cpp:297 updateProberType: Ethernet140: Unsupported link failure detection type for : software

This is a regression caused by PR : sonic-net/sonic-linkmgrd#288.

This is not causing any functionality breakage. But log analyzer reports this new error and hence the testcase fails.
Since there is no functionality impact, we have added this error message to the list of ignored patterns.

Summary:
Fixes #
https://github.com/aristanetworks/sonic-qual.msft/issues/624
sonic-net/sonic-linkmgrd#301

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Approach

What is the motivation for this PR?

Loganalyzer was not aware of this newly introduced error message and was failing many of the tests.

How did you do it?

Added a regex pattern to ignore these errors.

How did you verify/test it?

By running sonic-mgmt tests.

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

Adding a regex pattern to ignore the following type of error,
  ERR mux#linkmgrd: MuxManager.cpp:297 updateProberType: Ethernet140: Unsupported link failure detection type for : software
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@wsycqyz

wsycqyz commented Jun 23, 2025

Copy link
Copy Markdown
Contributor

Restart the failed PR test job.

@lolyu
lolyu self-requested a review July 1, 2025 00:38

@lolyu lolyu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@StormLiangMS StormLiangMS left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@StormLiangMS
StormLiangMS merged commit 97d9137 into sonic-net:master Jul 1, 2025
19 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jul 1, 2025
…c-net#19099)

What is the motivation for this PR?
Loganalyzer was not aware of this newly introduced error message and was failing many of the tests.

How did you do it?
Added a regex pattern to ignore these errors.

How did you verify/test it?
By running sonic-mgmt tests.
@mssonicbld

Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202505: #19298

@dhanasekar-arista
dhanasekar-arista deleted the fix_msft_issue_624 branch July 1, 2025 11:56
yejianquan pushed a commit that referenced this pull request Jul 2, 2025
…) (#19298)

What is the motivation for this PR?
Loganalyzer was not aware of this newly introduced error message and was failing many of the tests.

How did you do it?
Added a regex pattern to ignore these errors.

How did you verify/test it?
By running sonic-mgmt tests.

Co-authored-by: Dhanasekar Rathinavel <dhanasekar@arista.com>
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
…c-net#19099)

What is the motivation for this PR?
Loganalyzer was not aware of this newly introduced error message and was failing many of the tests.

How did you do it?
Added a regex pattern to ignore these errors.

How did you verify/test it?
By running sonic-mgmt tests.

Signed-off-by: opcoder0 <110003254+opcoder0@users.noreply.github.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
…c-net#19099)

What is the motivation for this PR?
Loganalyzer was not aware of this newly introduced error message and was failing many of the tests.

How did you do it?
Added a regex pattern to ignore these errors.

How did you verify/test it?
By running sonic-mgmt tests.

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
…c-net#19099)

What is the motivation for this PR?
Loganalyzer was not aware of this newly introduced error message and was failing many of the tests.

How did you do it?
Added a regex pattern to ignore these errors.

How did you verify/test it?
By running sonic-mgmt tests.
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
…c-net#19099)

What is the motivation for this PR?
Loganalyzer was not aware of this newly introduced error message and was failing many of the tests.

How did you do it?
Added a regex pattern to ignore these errors.

How did you verify/test it?
By running sonic-mgmt tests.

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
…c-net#19099)

What is the motivation for this PR?
Loganalyzer was not aware of this newly introduced error message and was failing many of the tests.

How did you do it?
Added a regex pattern to ignore these errors.

How did you verify/test it?
By running sonic-mgmt tests.

Signed-off-by: Yael Tzur <ytzur@nvidia.com>
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.

6 participants