[Libvirt-cim] [PATCH] Fix indication_tester's filter deletion issue

Jay Gagnon grendel at linux.vnet.ibm.com
Mon Mar 10 16:49:18 UTC 2008


# HG changeset patch
# User Jay Gagnon <grendel at linux.vnet.ibm.com>
# Date 1205167743 14400
# Node ID 4b34c7474e59f939200cdfeaa7d3414cfc6f7c13
# Parent  d93bb8c00423a88f1c9475a8e6076b1945e1dd01
Fix indication_tester's filter deletion issue

A while ago we discovered that giving the same name to various tests in indication_tester doesn't work, even when the first test's filter, handler, and subscription should have been deleted once it was done running.  It turns out that indication_tester wasn't deleting the filter, due to some bad xml.  This fixes the xml and eliminates the need to use the -n switch unless you want multiple simultaneous tests.

Signed-off-by: Jay Gagnon <grendel at linux.vnet.ibm.com>

diff -r d93bb8c00423 -r 4b34c7474e59 tools/indication_tester.py
--- a/tools/indication_tester.py	Mon Mar 03 06:48:26 2008 -0800
+++ b/tools/indication_tester.py	Mon Mar 10 12:49:03 2008 -0400
@@ -181,7 +181,7 @@ def delete_inst_xml(name, type, sysname)
               <NAMESPACE NAME="PG_InterOp"/> 
             </LOCALNAMESPACEPATH> 
             <IPARAMVALUE NAME="InstanceName"> 
-              <INSTANCENAME CLASSNAME="CIM_Indication%sCIMXML"> 
+              <INSTANCENAME CLASSNAME="CIM_Indication%s"> 
                 <KEYBINDING NAME="SystemCreationClassName"> 
                   <KEYVALUE>CIM_ComputerSystem</KEYVALUE> 
                 </KEYBINDING> 
@@ -189,7 +189,7 @@ def delete_inst_xml(name, type, sysname)
                   <KEYVALUE>%s</KEYVALUE> 
                 </KEYBINDING> 
                 <KEYBINDING NAME="CreationClassName"> 
-                  <KEYVALUE>CIM_Indication%sCIMXML</KEYVALUE> 
+                  <KEYVALUE>CIM_Indication%s</KEYVALUE> 
                 </KEYBINDING> 
                 <KEYBINDING NAME="Name"> 
                   <KEYVALUE>%s%s</KEYVALUE> 




More information about the Libvirt-cim mailing list