Skip to content

Skip test_fwutil due to not found fixture#6490

Merged
ZhaohuiS merged 1 commit intosonic-net:masterfrom
ZhaohuiS:fix/skip_test_fwutil
Oct 10, 2022
Merged

Skip test_fwutil due to not found fixture#6490
ZhaohuiS merged 1 commit intosonic-net:masterfrom
ZhaohuiS:fix/skip_test_fwutil

Conversation

@ZhaohuiS
Copy link
Copy Markdown
Contributor

@ZhaohuiS ZhaohuiS commented Oct 9, 2022

Signed-off-by: Zhaohui Sun zhaohuisun@microsoft.com

Description of PR

Summary:
Fixes # (issue)

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205

Approach

What is the motivation for this PR?

test_fwutil was introduced into nightly test in #6231, but all failed due to not found fixture.
File an issue #6489 and skip it for now.

How did you do it?

Add skip section in platform test mark condition yml file.

How did you verify/test it?

platform_tests/fwutil/test_fwutil.py::test_fwutil_update_bad_config
SKIPPED                                                                                                          [ 10%]
platform_tests/fwutil/test_fwutil.py::test_fwutil_install_file[str-dx010-acs-4] SKIPPED                                                                                 [ 20%]
platform_tests/fwutil/test_fwutil.py::test_fwutil_install_url[str-dx010-acs-4] SKIPPED                                                                                  [ 30%]
platform_tests/fwutil/test_fwutil.py::test_fwutil_update_current[str-dx010-acs-4] SKIPPED                                                                               [ 40%]
platform_tests/fwutil/test_fwutil.py::test_fwutil_update_next[str-dx010-acs-4] SKIPPED                                                                                  [ 50%]
platform_tests/fwutil/test_fwutil.py::test_fwutil_auto[str-dx010-acs-4-none] SKIPPED                                                                                    [ 60%]
platform_tests/fwutil/test_fwutil.py::test_fwutil_auto[str-dx010-acs-4-cold] SKIPPED                                                                                    [ 70%]
platform_tests/fwutil/test_fwutil.py::test_fwutil_install_bad_name SKIPPED                                                                                              [ 80%]
platform_tests/fwutil/test_fwutil.py::test_fwutil_install_bad_path SKIPPED                                                                                              [ 90%]
platform_tests/fwutil/test_fwutil.py::test_fwutil_update_bad_config SKIPPED

Any platform specific information?

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

Documentation

Signed-off-by: Zhaohui Sun <zhaohuisun@microsoft.com>
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

@ZhaohuiS
Copy link
Copy Markdown
Contributor Author

ZhaohuiS commented Oct 9, 2022

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@ZhaohuiS
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@ZhaohuiS ZhaohuiS merged commit 9701ac3 into sonic-net:master Oct 10, 2022
wangxin pushed a commit that referenced this pull request Oct 11, 2022
What is the motivation for this PR?
test_fwutil was introduced into nightly test in #6231, but all failed due to not found fixture.
File an issue #6489 and skip it for now.

How did you do it?
Add skip section in platform test mark condition yml file.

Signed-off-by: Zhaohui Sun <zhaohuisun@microsoft.com>
wangxin pushed a commit that referenced this pull request Oct 11, 2022
What is the motivation for this PR?
test_fwutil was introduced into nightly test in #6231, but all failed due to not found fixture.
File an issue #6489 and skip it for now.

How did you do it?
Add skip section in platform test mark condition yml file.

Signed-off-by: Zhaohui Sun <zhaohuisun@microsoft.com>
Azarack pushed a commit to Azarack/sonic-mgmt that referenced this pull request Oct 17, 2022
What is the motivation for this PR?
test_fwutil was introduced into nightly test in sonic-net#6231, but all failed due to not found fixture.
File an issue sonic-net#6489 and skip it for now.

How did you do it?
Add skip section in platform test mark condition yml file.

Signed-off-by: Zhaohui Sun <zhaohuisun@microsoft.com>
allen-xf pushed a commit to allen-xf/sonic-mgmt that referenced this pull request Oct 28, 2022
What is the motivation for this PR?
test_fwutil was introduced into nightly test in sonic-net#6231, but all failed due to not found fixture.
File an issue sonic-net#6489 and skip it for now.

How did you do it?
Add skip section in platform test mark condition yml file.

Signed-off-by: Zhaohui Sun <zhaohuisun@microsoft.com>
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.

3 participants