Skip to content

[202511] Remove test_radv_ipv6_ra.py xfail for v6 topos#22832

Merged
StormLiangMS merged 1 commit intosonic-net:202511from
markx-arista:202511-remove-radv-ipv6-xfail-for-v6-topo
Mar 12, 2026
Merged

[202511] Remove test_radv_ipv6_ra.py xfail for v6 topos#22832
StormLiangMS merged 1 commit intosonic-net:202511from
markx-arista:202511-remove-radv-ipv6-xfail-for-v6-topo

Conversation

@markx-arista
Copy link
Copy Markdown
Contributor

Description of PR

Remove xfail because the test can pass on v6 topos after garp fix:
#21155
Manual cherry-pick of: #21449

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

  • 202412
  • 202505

Approach

What is the motivation for this PR?

The test case can pass with the fix from #21155

How did you do it?

Remove xfail in tests_mark_conditions.yaml

How did you verify/test it?

The test passed on v6 topo after removing xfail

Any platform specific information?

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

v6 topos

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@github-actions github-actions bot requested a review from xwjiang-ms March 10, 2026 04:07
@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@markx-arista markx-arista force-pushed the 202511-remove-radv-ipv6-xfail-for-v6-topo branch from c4c4e7f to f1493c4 Compare March 10, 2026 16:40
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

@markx-arista markx-arista force-pushed the 202511-remove-radv-ipv6-xfail-for-v6-topo branch from f1493c4 to 33bccf1 Compare March 11, 2026 20:56
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@markx-arista
Copy link
Copy Markdown
Contributor Author

Failing static analysis due to duplicate keys in: #22857

Remove xfail because the test can pass on v6 topos after garp fix:
sonic-net#21155

Signed-off-by: markxiao <[email protected]>
@markx-arista markx-arista force-pushed the 202511-remove-radv-ipv6-xfail-for-v6-topo branch from 33bccf1 to 2c2351b Compare March 11, 2026 22:05
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@markx-arista
Copy link
Copy Markdown
Contributor Author

Failing static analysis due to duplicate keys in: #22857

Fixed in #22887, rebased

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 37694f7 into sonic-net:202511 Mar 12, 2026
16 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