[Libvirt-cim] [PATCH 1 of 3] [TEST] KVM support and refactor to Profile.01

[Kaitlin Rupert]

> +    checklist = [[x.InstanceID, x.RegisteredOrganization, 
> +                  x.RegisteredName, x.RegisteredVersion] for x in proflist]
> +    for exp_prof in explist:
> +        if exp_prof in checklist:
> +            continue
> +        else:
> +            logger.error("Profile %s is not found" % exp_prof[0])
> +            status = FAIL

You could reorganize this so that if the exp_prof isn't in the 
checklist, you log an error and return a failure.  Otherwise, you log 
the info message and return.


> -
> -# The execution will reach here only if all the checks are successful       
> -    Globals.logger.log(int(Globals.logging.PRINT), "Verification of the properties \
> -for the class '%s' was successful", cn)
> +    if status == PASS:
> +        logger.info("Properties check for %s passed" % cn)
>      return status
> 

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