Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[202411] Temporary skip subnet decap test with issue since feature haven't been added to yang model #16152

Conversation

xwjiang-ms
Copy link
Contributor

Description of PR

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

Approach

What is the motivation for this PR?

Cherry-pick PR of #15947, in sonic-net/sonic-utilities#3102, the feature comparing yang model with configDB was added to sonic-utilities submodule, but feature subnet decap haven't been added to yang model, so the PR test advancing sonic-utilities to buildimage would fail in subnet decap test.

How did you do it?

Created issue sonic-net/sonic-buildimage#21090 and temporary skip subnet decap test until issue resolved

How did you verify/test it?

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@kperumalbfn kperumalbfn merged commit 95182e8 into sonic-net:202411 Dec 19, 2024
4 checks passed
@xwjiang-ms xwjiang-ms deleted the temporary_skip_subnet_decap_test_202411 branch December 19, 2024 01:49
wangxin pushed a commit that referenced this pull request Dec 19, 2024
What is the motivation for this PR?
In PR #16152, we need to skip subnet decap test in PR test if issue not resolved, but conditions should be or, not and

How did you do it?
Set conditions_logical_operator to or

How did you verify/test it?
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