Skip to content

[action] [PR:12941] Include advanced-reboot ptf exception in pytest assertion#12972

Merged
mssonicbld merged 1 commit intosonic-net:202311from
mssonicbld:cherry/202311/12941
May 24, 2024
Merged

[action] [PR:12941] Include advanced-reboot ptf exception in pytest assertion#12972
mssonicbld merged 1 commit intosonic-net:202311from
mssonicbld:cherry/202311/12941

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Description of PR

This allows the ptf exception to be reported in the Summary field of the V2TestCases table in Kusto.

ADO: 28078972

Summary:
Fixes # (issue)

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205
  • 202305
  • 202311

Approach

What is the motivation for this PR?

Surface more information in our test reporting infrastructure to aid in debugging

How did you do it?

Added more information to a pytest assertion.

How did you verify/test it?

Artificially added an exception to the try-except block and saw that it was printed out in the pytest assertions.

Any platform specific information?

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

Documentation

…12941)

What is the motivation for this PR?
Surface more information in our test reporting infrastructure to aid in debugging

How did you do it?
Added more information to a pytest assertion.

How did you verify/test it?
Artificially added an exception to the try-except block and saw that it was printed out in the pytest assertions.
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #12941

@mssonicbld mssonicbld merged commit 164761d into sonic-net:202311 May 24, 2024
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.

2 participants