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

Added loganalyzer for pytest #1048

Merged
merged 10 commits into from
Aug 28, 2019

Conversation

yvolynets-mlnx
Copy link
Contributor

Description of PR

Summary: Added loganalyzer module. Created loganalyzer autouse fixture.
Fixes # (issue)

Type of change

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

Approach

How did you do it?

Added new folder: tests/loganalyzer/

How did you verify/test it?

Verified on platform tests on the local setup.

Any platform specific information?

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

Documentation

README.md is available in tests/loganalyzer/README.md

tests/ansible_fixtures.py Outdated Show resolved Hide resolved
tests/conftest.py Outdated Show resolved Hide resolved
tests/conftest.py Outdated Show resolved Hide resolved
tests/conftest.py Outdated Show resolved Hide resolved
tests/conftest.py Outdated Show resolved Hide resolved
tests/lib/helpers.py Outdated Show resolved Hide resolved
tests/loganalyzer/README.md Outdated Show resolved Hide resolved
tests/loganalyzer/loganalyzer.py Outdated Show resolved Hide resolved
tests/loganalyzer/loganalyzer.py Outdated Show resolved Hide resolved
tests/lib/helpers.py Outdated Show resolved Hide resolved
@yvolynets-mlnx yvolynets-mlnx marked this pull request as ready for review August 7, 2019 16:28
Signed-off-by: Yuriy Volynets <yuriyv@mellanox.com>
Signed-off-by: Yuriy Volynets <yuriyv@mellanox.com>
@liat-grozovik
Copy link
Collaborator

@lguohan please provide your feedback, so we can merge.
this is an infra change which is already used by ACL test which depends on it.

@neethajohn neethajohn merged commit 095b466 into sonic-net:master Aug 28, 2019
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.

4 participants