Kaitlin,

But when cimtest's vxml.py calls DefineSystem -

###########
service.DefineSystem(SystemSettings=sys_settings,
ResourceSettings=res_settings,
ReferenceConfiguration=ref_conf)
#############


it passes "res_settings" and these were set in vsms.py as -

************
class CIM_GraphicsResourceAllocationSettingData(CIMClassMOF):
def __init__(self, name, res_sub_type="vnc", ip=None, ipv6_flag=None,
lport='-1', keymap="en-us", vnc_passwd=None):
self.InstanceID = '%s/graphics' %name
self.ResourceType = RASD_TYPE_GRAPHICS

if res_sub_type != None:
self.ResourceSubType = res_sub_type

if ip != None and lport != None:
self.Address = '%s:%s' % (ip, lport)

if ipv6_flag != None:
self.is_ipv6_only = ipv6_flag
*************

So the name should change in VSMS.c too.

Sharad Mishra
System x Enablement
Linux Technology Center
IBM

libvirt-cim-bounces@redhat.com wrote on 12/03/2009 04:48:31 PM:

> Kaitlin Rupert <kaitlin@linux.vnet.ibm.com>
> Sent by: libvirt-cim-bounces@redhat.com
>
> 12/03/2009 04:48 PM
>
> Please respond to
> List for discussion and development of libvirt CIM <libvirt-cim@redhat.com>
>
> To
>
> List for discussion and development of libvirt CIM <libvirt-cim@redhat.com>
>
> cc
>
> Subject
>
> Re: [Libvirt-cim] [PATCH 0 of 2] Camel case to lower case variable
> name change.
>
> I don't think these changes are necessary.  IsIPv6Only fits the DMTF's
> naming convention for attributes.  I would say to leave the attribute
> name as is.
>
> Sharad Mishra wrote:
> > Signed-off-by: Sharad Mishra <snmishra@us.ibm.com>
> >
> > _______________________________________________
> > Libvirt-cim mailing list
> > Libvirt-cim@redhat.com
> > https://www.redhat.com/mailman/listinfo/libvirt-cim
>
>
> --
> Kaitlin Rupert
> IBM Linux Technology Center
> kaitlin@linux.vnet.ibm.com
>
> _______________________________________________
> Libvirt-cim mailing list
> Libvirt-cim@redhat.com
> https://www.redhat.com/mailman/listinfo/libvirt-cim