Skip to content

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

Merged
kperumalbfn merged 1 commit intosonic-net:202411from
xwjiang-ms:temporary_skip_subnet_decap_test_202411
Dec 19, 2024
Merged

[202411] Temporary skip subnet decap test with issue since feature haven't been added to yang model#16152
kperumalbfn merged 1 commit intosonic-net:202411from
xwjiang-ms:temporary_skip_subnet_decap_test_202411

Conversation

@xwjiang-ms
Copy link
Copy Markdown
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
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

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
@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