-
Notifications
You must be signed in to change notification settings - Fork 708
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
Fix syslog test failure about fixture tbinfo #13821
Conversation
The pre-commit check detected issues in the files touched by this pull request. Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
The pre-commit check detected issues in the files touched by this pull request. Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
This reverts commit 1a14165.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Added the request label for 202405
@yejianquan would you be able to help approve/merge this? thanks. |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@wumiaont PR conflicts with 202405 branch |
Hi @wumiaont , please raise a separate PR resolving the conflict to 202405 branch |
OK |
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]
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]
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]
Cherry-pick PR to 202405: #14547 |
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]
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]
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]
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
Back port request
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