Skip to content

add one errmsg to be ignored in duplicate routes test case#21684

Merged
StormLiangMS merged 1 commit intosonic-net:masterfrom
fkong-cisco:master
Feb 13, 2026
Merged

add one errmsg to be ignored in duplicate routes test case#21684
StormLiangMS merged 1 commit intosonic-net:masterfrom
fkong-cisco:master

Conversation

@fkong-cisco
Copy link
Copy Markdown
Contributor

Description of PR

There is a new error message that gets generated when test_duplicate_routes.py is run, i.e. when a duplicate route entry is trying to be downloaded, which is an error condition. This new error message is not recognized by the existed "ignore list" and it should be added to it to avoid flagging a known condition as an unknown error.

Summary:
Simply add the new error message to the ignore list.

Type of change

script 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

Let the test run smoothly for code base with new error message

How did you do it?

just add it to the list

How did you verify/test it?

tested the script

Any platform specific information?

worked on sim

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

n/a

Documentation

n/a

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla bot commented Dec 12, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: fkong-cisco / name: Fan Kong (d6c1d40)

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Fan Kong <fkong@cisco.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Contributor

@wsycqyz wsycqyz left a comment

Choose a reason for hiding this comment

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

i am ok with this PR

@wsycqyz wsycqyz added the Request for 202511 branch Request to backport a change to 202511 branch label Feb 13, 2026
@StormLiangMS StormLiangMS merged commit 62bf97b into sonic-net:master Feb 13, 2026
18 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Feb 13, 2026
…#21684)

Approach
Let the test run smoothly for code base with new error message

How did you do it?
just add it to the list

How did you verify/test it?
tested the script

Any platform specific information?
worked on sim

Supported testb

Signed-off-by: mssonicbld <sonicbld@microsoft.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202511: #22397

mssonicbld pushed a commit that referenced this pull request Feb 13, 2026
Approach
Let the test run smoothly for code base with new error message

How did you do it?
just add it to the list

How did you verify/test it?
tested the script

Any platform specific information?
worked on sim

Supported testb

Signed-off-by: mssonicbld <sonicbld@microsoft.com>
anilal-amd pushed a commit to anilal-amd/anilal-forked-sonic-mgmt that referenced this pull request Feb 19, 2026
…#21684)

Approach
Let the test run smoothly for code base with new error message

How did you do it?
just add it to the list

How did you verify/test it?
tested the script

Any platform specific information?
worked on sim

Supported testb

Signed-off-by: Zhuohui Tan <zhuohui.tan@amd.com>
aronovic pushed a commit to aronovic/sonic-mgmt that referenced this pull request Mar 3, 2026
…#21684)

Approach
Let the test run smoothly for code base with new error message

How did you do it?
just add it to the list

How did you verify/test it?
tested the script

Any platform specific information?
worked on sim

Supported testb

Signed-off-by: Mihut Aronovici <aronovic@cisco.com>
ravaliyel pushed a commit to ravaliyel/sonic-mgmt that referenced this pull request Mar 12, 2026
…#21684)

Approach
Let the test run smoothly for code base with new error message

How did you do it?
just add it to the list

How did you verify/test it?
tested the script

Any platform specific information?
worked on sim

Supported testb

Signed-off-by: Ravali Yeluri (WIPRO LIMITED) <v-ryeluri@microsoft.com>
abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Mar 17, 2026
…#21684)

Approach
Let the test run smoothly for code base with new error message

How did you do it?
just add it to the list

How did you verify/test it?
tested the script

Any platform specific information?
worked on sim

Supported testb

Signed-off-by: Abhishek <abhishek@nexthop.ai>
ravaliyel pushed a commit to ravaliyel/sonic-mgmt that referenced this pull request Mar 27, 2026
…#21684)

Approach
Let the test run smoothly for code base with new error message

How did you do it?
just add it to the list

How did you verify/test it?
tested the script

Any platform specific information?
worked on sim

Supported testb
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.

5 participants