[Libvirt-cim] [PATCH] [TEST] #2 Fix SettingsDefineCapabilities/01_forward.py with appropriate RASD objects for different platforms

[Guo Lian Yun]

libvirt-cim-bounces at redhat.com wrote on 2009-02-07 01:12:30:

> > @@ -172,10 +173,17 @@
> > 
> >              curr_cim_rev, changeset = get_provider_version(virt, 
server)
> >              if 'DiskPool' in instid and (virt =='Xen' or virt == 
> 'XenFV') and \
> > -                curr_cim_rev >= libvirt_rasd_template_changes:
> > +                curr_cim_rev >= libvirt_rasd_template_changes and \
> > +                curr_cim_rev < libvirt_rasd_new_changes:
> >                  # For Diskpool, we have info 1 for each of Min, Max, 
> >                  # default, Increment and 1 for each of PV and FV 
> >                  # hence 4 * 2 = 8 records
> > +                exp_len = 8
> > +            elif 'DiskPool' in instid and (virt =='Xen' or virt 
> == 'XenFV') and \
> 
> This line is longer than 80 characters.
> 
> > +                curr_cim_rev >= libvirt_rasd_new_changes:
> > +                exp_len = 16
> > +            elif 'DiskPool' in instid and virt == 'KVM' and \
> > +                curr_cim_rev >= libvirt_rasd_new_changes:
> >                  exp_len = 8
> >              else:
> >                  exp_len = 4
> > 
> 
> Also this can be simplified to:
> 
> if 'DiskPool' in instid:
>    if (virt =='Xen' or virt == 'XenFV') and \
>       curr_cim_rev >= libvirt_rasd_new_changes:
> 
>     (etc)
> 
  Fixed them on #3 patch.

> -- 
> Kaitlin Rupert
> IBM Linux Technology Center
> kaitlin at linux.vnet.ibm.com
> 
> _______________________________________________
> Libvirt-cim mailing list
> Libvirt-cim at redhat.com
> https://www.redhat.com/mailman/listinfo/libvirt-cim
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvirt-cim/attachments/20090210/78cb0fce/attachment.htm>