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

[action] [PR:13821] Fix syslog test failure about fixture tbinfo #14547

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

mssonicbld
Copy link
Collaborator

Description of PR

Syslog test failed for complaining about tbinfo usage as it's a fixture. Solution is to add tbinfo as fixture in the orch_logrotate_setup. Then referencing tbinfo will be the yield of tbinfo which is a dict

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

What is the motivation for this PR?

Fix the test failure for syslog test_logrotate.

How did you do it?

How did you verify/test it?

Tested on OC test bed and see test pass now.

Any platform specific information?

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

Documentation

Description of PR
Syslog test failed for complaining about tbinfo usage as it's a fixture. Solution is to add tbinfo as fixture in the orch_logrotate_setup. Then referencing tbinfo will be the yield of tbinfo which is a dict

What is the motivation for this PR?
Fix the test failure for syslog test_logrotate.

How did you do it?
How did you verify/test it?
Tested on OC test bed and see test pass now.

co-authorized by: [email protected]
@mssonicbld
Copy link
Collaborator Author

Original PR: #13821

@mssonicbld mssonicbld merged commit 4546c1b into sonic-net:202405 Sep 12, 2024
13 checks passed
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