Skip to content

[202505][dualtor] Skip test_replace_fec#21359

Merged
StormLiangMS merged 1 commit intosonic-net:202505from
lolyu:202505_test_replace_fec_skip_dualtor_m2
Dec 29, 2025
Merged

[202505][dualtor] Skip test_replace_fec#21359
StormLiangMS merged 1 commit intosonic-net:202505from
lolyu:202505_test_replace_fec_skip_dualtor_m2

Conversation

@lolyu
Copy link
Copy Markdown
Collaborator

@lolyu lolyu commented Nov 19, 2025

Description of PR

Summary:
Fixes # (issue)

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?

Cherry-pick #21358 into 202505.

Signed-off-by: Longxiang Lyu [email protected]

How did you do it?

How did you verify/test it?

Any platform specific information?

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

Documentation

Skip `test_replace_fec` on dualtor as the test is trying to change
downlink's fec from `none` to `rs`, this is not a valid/supported
scenario, the testcase needs a further improvement.

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

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@StormLiangMS StormLiangMS reopened this Dec 3, 2025
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@StormLiangMS StormLiangMS merged commit 5e69d2a into sonic-net:202505 Dec 29, 2025
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants