Skip to content

ignore SCD kernel warning about I2C communication ack errors#21959

Merged
StormLiangMS merged 1 commit intosonic-net:masterfrom
lipxu:20260118_publicMaster_ignoreSCDWarning
Jan 20, 2026
Merged

ignore SCD kernel warning about I2C communication ack errors#21959
StormLiangMS merged 1 commit intosonic-net:masterfrom
lipxu:20260118_publicMaster_ignoreSCDWarning

Conversation

@lipxu
Copy link
Copy Markdown
Contributor

@lipxu lipxu commented Jan 18, 2026

Description of PR

Summary:
Fixes # (issue)
36327584

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
  • 202511

Approach

What is the motivation for this PR?

In the new testbed, the kernel outputs warning messages containing the word “error,” which causes LogAnalyzer to treat them as errors.

How did you do it?

It's a warning message, should not be failed in loganalyzer, ignore these messages.

How did you verify/test it?

https://elastictest.org/scheduler/testplan/696d71df4bbe3bd7ad16cdf3
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436d
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436b
https://elastictest.org/scheduler/testplan/696d71dc15026fd4f746b8f0
https://elastictest.org/scheduler/testplan/696d71db4bbe3bd7ad16cdf1
https://elastictest.org/scheduler/testplan/696d71da5e4aaa3c28ac7499

Any platform specific information?

Arista

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

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@ZhaohuiS
Copy link
Copy Markdown
Contributor

@lipxu why is it in draft? still not sure to ignore it yet?

Copy link
Copy Markdown
Contributor

@byu343 byu343 left a comment

Choose a reason for hiding this comment

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

LGTM

@lipxu lipxu marked this pull request as ready for review January 19, 2026 22:31
@lipxu lipxu requested a review from yxieca as a code owner January 19, 2026 22:31
@lipxu
Copy link
Copy Markdown
Contributor Author

lipxu commented Jan 19, 2026

@lipxu why is it in draft? still not sure to ignore it yet?

Thanks for your review, @ZhaohuiS , I kept the PR in draft status because the verification tests had not finished. The Elastic tests have now completed, and the results look as expected. thanks.

@lipxu lipxu requested a review from StormLiangMS January 19, 2026 22:36
@lipxu
Copy link
Copy Markdown
Contributor Author

lipxu commented Jan 19, 2026

Hi, @StormLiangMS , Could you please help to review the PR and merge it if no more comments, thanks a lot.

Copy link
Copy Markdown
Collaborator

@StormLiangMS StormLiangMS left a comment

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 0d0585d into sonic-net:master Jan 20, 2026
21 checks passed
@StormLiangMS StormLiangMS added the Request for 202511 branch Request to backport a change to 202511 branch label Jan 21, 2026
PriyanshTratiya pushed a commit to PriyanshTratiya/sonic-mgmt that referenced this pull request Jan 21, 2026
…et#21959)

What is the motivation for this PR?
In the new testbed, the kernel outputs warning messages containing the word “error,” which causes LogAnalyzer to treat them as errors.

How did you do it?
It's a warning message, should not be failed in loganalyzer, ignore these messages.

How did you verify/test it?
https://elastictest.org/scheduler/testplan/696d71df4bbe3bd7ad16cdf3
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436d
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436b
https://elastictest.org/scheduler/testplan/696d71dc15026fd4f746b8f0
https://elastictest.org/scheduler/testplan/696d71db4bbe3bd7ad16cdf1
https://elastictest.org/scheduler/testplan/696d71da5e4aaa3c28ac7499

Signed-off-by: Priyansh Tratiya <[email protected]>
saravanan-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Jan 22, 2026
…et#21959)

What is the motivation for this PR?
In the new testbed, the kernel outputs warning messages containing the word “error,” which causes LogAnalyzer to treat them as errors.

How did you do it?
It's a warning message, should not be failed in loganalyzer, ignore these messages.

How did you verify/test it?
https://elastictest.org/scheduler/testplan/696d71df4bbe3bd7ad16cdf3
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436d
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436b
https://elastictest.org/scheduler/testplan/696d71dc15026fd4f746b8f0
https://elastictest.org/scheduler/testplan/696d71db4bbe3bd7ad16cdf1
https://elastictest.org/scheduler/testplan/696d71da5e4aaa3c28ac7499

Signed-off-by: Saravanan Sellappa <[email protected]>
justin-oliver pushed a commit to justin-oliver/sonic-mgmt that referenced this pull request Jan 26, 2026
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
…et#21959)

What is the motivation for this PR?
In the new testbed, the kernel outputs warning messages containing the word “error,” which causes LogAnalyzer to treat them as errors.

How did you do it?
It's a warning message, should not be failed in loganalyzer, ignore these messages.

How did you verify/test it?
https://elastictest.org/scheduler/testplan/696d71df4bbe3bd7ad16cdf3
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436d
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436b
https://elastictest.org/scheduler/testplan/696d71dc15026fd4f746b8f0
https://elastictest.org/scheduler/testplan/696d71db4bbe3bd7ad16cdf1
https://elastictest.org/scheduler/testplan/696d71da5e4aaa3c28ac7499

Signed-off-by: Yael Tzur <[email protected]>
abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Feb 6, 2026
Anirudh-nokia pushed a commit to Anirudh-nokia/sonic-mgmt-fork that referenced this pull request Feb 6, 2026
…et#21959)

What is the motivation for this PR?
In the new testbed, the kernel outputs warning messages containing the word “error,” which causes LogAnalyzer to treat them as errors.

How did you do it?
It's a warning message, should not be failed in loganalyzer, ignore these messages.

How did you verify/test it?
https://elastictest.org/scheduler/testplan/696d71df4bbe3bd7ad16cdf3
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436d
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436b
https://elastictest.org/scheduler/testplan/696d71dc15026fd4f746b8f0
https://elastictest.org/scheduler/testplan/696d71db4bbe3bd7ad16cdf1
https://elastictest.org/scheduler/testplan/696d71da5e4aaa3c28ac7499

Signed-off-by: ayya <[email protected]>
@lipxu
Copy link
Copy Markdown
Contributor Author

lipxu commented Feb 9, 2026

@vmittal-msft Could you please approve the request for 202511, thank you very much.

nnelluri-cisco pushed a commit to nnelluri-cisco/sonic-mgmt that referenced this pull request Feb 12, 2026
…et#21959)

What is the motivation for this PR?
In the new testbed, the kernel outputs warning messages containing the word “error,” which causes LogAnalyzer to treat them as errors.

How did you do it?
It's a warning message, should not be failed in loganalyzer, ignore these messages.

How did you verify/test it?
https://elastictest.org/scheduler/testplan/696d71df4bbe3bd7ad16cdf3
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436d
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436b
https://elastictest.org/scheduler/testplan/696d71dc15026fd4f746b8f0
https://elastictest.org/scheduler/testplan/696d71db4bbe3bd7ad16cdf1
https://elastictest.org/scheduler/testplan/696d71da5e4aaa3c28ac7499

Signed-off-by: nnelluri-cisco <[email protected]>
rraghav-cisco pushed a commit to rraghav-cisco/sonic-mgmt that referenced this pull request Feb 13, 2026
…et#21959)

What is the motivation for this PR?
In the new testbed, the kernel outputs warning messages containing the word “error,” which causes LogAnalyzer to treat them as errors.

How did you do it?
It's a warning message, should not be failed in loganalyzer, ignore these messages.

How did you verify/test it?
https://elastictest.org/scheduler/testplan/696d71df4bbe3bd7ad16cdf3
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436d
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436b
https://elastictest.org/scheduler/testplan/696d71dc15026fd4f746b8f0
https://elastictest.org/scheduler/testplan/696d71db4bbe3bd7ad16cdf1
https://elastictest.org/scheduler/testplan/696d71da5e4aaa3c28ac7499

Signed-off-by: Raghavendran Ramanathan <[email protected]>
anilal-amd pushed a commit to anilal-amd/anilal-forked-sonic-mgmt that referenced this pull request Feb 19, 2026
…et#21959)

What is the motivation for this PR?
In the new testbed, the kernel outputs warning messages containing the word “error,” which causes LogAnalyzer to treat them as errors.

How did you do it?
It's a warning message, should not be failed in loganalyzer, ignore these messages.

How did you verify/test it?
https://elastictest.org/scheduler/testplan/696d71df4bbe3bd7ad16cdf3
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436d
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436b
https://elastictest.org/scheduler/testplan/696d71dc15026fd4f746b8f0
https://elastictest.org/scheduler/testplan/696d71db4bbe3bd7ad16cdf1
https://elastictest.org/scheduler/testplan/696d71da5e4aaa3c28ac7499

Signed-off-by: Zhuohui Tan <[email protected]>
@mssonicbld
Copy link
Copy Markdown
Collaborator

@lipxu PR conflicts with 202511 branch

lipxu added a commit to lipxu/sonic-mgmt-lipxu-fork that referenced this pull request Feb 22, 2026
lipxu added a commit to lipxu/sonic-mgmt-lipxu-fork that referenced this pull request Feb 22, 2026
…et#21959)

What is the motivation for this PR?
In the new testbed, the kernel outputs warning messages containing the word “error,” which causes LogAnalyzer to treat them as errors.

How did you do it?
It's a warning message, should not be failed in loganalyzer, ignore these messages.

How did you verify/test it?
https://elastictest.org/scheduler/testplan/696d71df4bbe3bd7ad16cdf3
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436d
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436b
https://elastictest.org/scheduler/testplan/696d71dc15026fd4f746b8f0
https://elastictest.org/scheduler/testplan/696d71db4bbe3bd7ad16cdf1
https://elastictest.org/scheduler/testplan/696d71da5e4aaa3c28ac7499

Signed-off-by: xuliping <[email protected]>
@lipxu
Copy link
Copy Markdown
Contributor Author

lipxu commented Feb 22, 2026

@lipxu PR conflicts with 202511 branch

Create a PR to merge it manually, #22532

vmittal-msft pushed a commit that referenced this pull request Feb 23, 2026
…#22532)

What is the motivation for this PR?
In the new testbed, the kernel outputs warning messages containing the word “error,” which causes LogAnalyzer to treat them as errors.

How did you do it?
It's a warning message, should not be failed in loganalyzer, ignore these messages.

How did you verify/test it?
https://elastictest.org/scheduler/testplan/696d71df4bbe3bd7ad16cdf3
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436d
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436b
https://elastictest.org/scheduler/testplan/696d71dc15026fd4f746b8f0
https://elastictest.org/scheduler/testplan/696d71db4bbe3bd7ad16cdf1
https://elastictest.org/scheduler/testplan/696d71da5e4aaa3c28ac7499

Signed-off-by: xuliping <[email protected]>
@lipxu
Copy link
Copy Markdown
Contributor Author

lipxu commented Mar 2, 2026

@lipxu PR conflicts with 202511 branch

Create a PR to merge it manually, #22532

#22532 PR merged

abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Mar 17, 2026
…et#21959)

What is the motivation for this PR?
In the new testbed, the kernel outputs warning messages containing the word “error,” which causes LogAnalyzer to treat them as errors.

How did you do it?
It's a warning message, should not be failed in loganalyzer, ignore these messages.

How did you verify/test it?
https://elastictest.org/scheduler/testplan/696d71df4bbe3bd7ad16cdf3
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436d
https://elastictest.org/scheduler/testplan/696d71dd4b8aa910b618436b
https://elastictest.org/scheduler/testplan/696d71dc15026fd4f746b8f0
https://elastictest.org/scheduler/testplan/696d71db4bbe3bd7ad16cdf1
https://elastictest.org/scheduler/testplan/696d71da5e4aaa3c28ac7499

Signed-off-by: Abhishek <[email protected]>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
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