-
Notifications
You must be signed in to change notification settings - Fork 22
CP-310799 Enhance ACK Test Case Selection Functionality #231
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
base: master
Are you sure you want to change the base?
CP-310799 Enhance ACK Test Case Selection Functionality #231
Conversation
Signed-off-by: Catherine Liu <[email protected]>
Signed-off-by: Catherine Liu <[email protected]>
| def normalize_test_class_name(n): | ||
| while n.startswith("autocertkit."): | ||
| n = n[11:] | ||
| return n |
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.
I don't see why we need this function.
| However, if there is a failure such that the ack-submission package is not created, then we would ask that you submit the following: | ||
|
|
||
| - Server Status Report - this can be obtained either by either running the ‘xen-bugtool –y’ command on the coordinator host, or by using XenCenter (Tools - Get Server Status Report). | ||
| - Server Status Report - this can be obtained either by either running the ‘xen-bugtool -y’ command on the coordinator host, or by using XenCenter (Tools - Get Server Status Report). |
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.
It's not introduced in this PR, but there're two either: obtained either by either
| if "run_classes" not in config: | ||
| return | ||
|
|
||
| from test_generators import enumerate_all_test_classes |
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.
I don't think it's necessary to import here, as test_generators is already imported at top of the file.
This PR enhances the ACK test case selection functionality with the following improvements:
In addition to these functional changes, this PR also updates the publication date to align with the latest release plan.
Regression test (232076) has passed
Following test case also passed with manual test:
"run_classes=network_tests.MulticastTestClass
network_tests.IntraHostSRIOVTestClass2
operations_tests.CrashDumpTestClass"