[Libvirt-cim] [PATCH 1 of 2] [TEST] Modified the indication_tester.py to support MigrationIndications

[Kaitlin Rupert]

Deepti B. Kalakeri wrote:
> # HG changeset patch
> # User Deepti B. Kalakeri <deeptik at 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 at 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.

-- 
Kaitlin Rupert
IBM Linux Technology Center
kaitlin at linux.vnet.ibm.com