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.
_______________________________________________
OpenStack-dev mailing list
[email protected]
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
