Kaitlin Rupert wrote:
Deepti B. Kalakeri wrote:
# HG changeset patch # User Deepti B. Kalakeri <deeptik@linux.vnet.ibm.com> # Date 1246972788 25200 # Node ID 9903b1d91bd966f453392ddbd0378feb50a4f771 # Parent 479f287a17fd08e4e22ac37459393f6f8327315a [TEST] Modified the indication_tester.py to support MigrationIndications.
Modified indication_tester.py to include - sub_ind() --> Subscribe for the indications to be watched. - handle_request() --> Filters the required indications. - poll_for_ind() --> Wait for the required indications to be triggered.
Tested with KVM and current sources on F10. Signed-off-by: Deepti B. Kalakeri <deeptik@linux.vnet.ibm.com>
diff -r 479f287a17fd -r 9903b1d91bd9 suites/libvirt-cim/lib/XenKvmLib/indication_tester.py
I'm torn on whether to include these changes in this file. Really, the indication_tester.py was a standalone tool that wasn't originally included in the cimtest tree. It was pulled in later so that the automated test cases could leverage its functionality.
Anytime a change is made to other copy, the changes need to be pushed to the other so that the two remain in sync. These changes won't ever be used by the indication_tester.py itself, so I'm inclined to suggest moving them into their own module.
Generally, I'm against having lots of different modules - I am in favor of consolidating where appropriate. However, I think this makes sense given that indication_tester is a special case.
I agree with you.. I too came up with the same problem of whether to extend the indication_tester.py to include the new fns(). I have moved the required fns() for Indications to a new file called indications.py, so now indication_tester.py is independent to be moved after the CSI/01_created_indication.py is modified to use the indications.py fns(). I think since we have a different dir which standalone tc, we can move the indication_tester.py to suites/libvirt-cim/misc_cimtests/, and how about migration_tester.py ...thoughts ?? -- Thanks and Regards, Deepti B. Kalakeri IBM Linux Technology Center deeptik@linux.vnet.ibm.com