Open Stack

Jay S. Bryant said on Wed, Jul 09, 2014 at 10:28:00PM -0500:
> > single quote: LOG.info("The ID '%s' is not invalid." % (resource.id))
> +1  This isn't one I have been policing a lot in my reviews, but think
> this is a good practice.

+1 from me.

> > LOG.info("This path is invalid: %s" % (obj.path))
> +1 This is one I have been enforcing in Cinder reviews.

+1


Alexis
-- 
Nova Engineer, HP Cloud.  AKA lealexis, lxsli.