<br><tt><font size=2>libvirt-cim-bounces@redhat.com wrote on 2009-02-07
01:12:30:<br>
<br>
> > @@ -172,10 +173,17 @@<br>
> > <br>
> >              curr_cim_rev,
changeset = get_provider_version(virt, server)<br>
> >              if 'DiskPool'
in instid and (virt =='Xen' or virt == <br>
> 'XenFV') and \<br>
> > -                curr_cim_rev
>= libvirt_rasd_template_changes:<br>
> > +                curr_cim_rev
>= libvirt_rasd_template_changes and \<br>
> > +                curr_cim_rev
< libvirt_rasd_new_changes:<br>
> >                  #
For Diskpool, we have info 1 for each of Min, Max, <br>
> >                  #
default, Increment and 1 for each of PV and FV <br>
> >                  #
hence 4 * 2 = 8 records<br>
> > +                exp_len
= 8<br>
> > +            elif 'DiskPool' in
instid and (virt =='Xen' or virt <br>
> == 'XenFV') and \<br>
> <br>
> This line is longer than 80 characters.<br>
> <br>
> > +                curr_cim_rev
>= libvirt_rasd_new_changes:<br>
> > +                exp_len
= 16<br>
> > +            elif 'DiskPool' in
instid and virt == 'KVM' and \<br>
> > +                curr_cim_rev
>= libvirt_rasd_new_changes:<br>
> >                  exp_len
= 8<br>
> >              else:<br>
> >                  exp_len
= 4<br>
> > <br>
> <br>
> Also this can be simplified to:<br>
> <br>
> if 'DiskPool' in instid:<br>
>    if (virt =='Xen' or virt == 'XenFV') and \<br>
>       curr_cim_rev >= libvirt_rasd_new_changes:<br>
> <br>
>     (etc)<br>
> </font></tt>
<br><tt><font size=2>  Fixed them on #3 patch.</font></tt>
<br><tt><font size=2><br>
> -- <br>
> Kaitlin Rupert<br>
> IBM Linux Technology Center<br>
> kaitlin@linux.vnet.ibm.com<br>
> <br>
> _______________________________________________<br>
> Libvirt-cim mailing list<br>
> Libvirt-cim@redhat.com<br>
> https://www.redhat.com/mailman/listinfo/libvirt-cim<br>
</font></tt>