Open Stack

warn() and warning() are synonyms (literally the same method, aliased). We had to add a similar alias in the oslo ContextAdapter to support code using both forms in existing code. If the documented form is warning(), then I agree we should stick with that, although I don’t think the churn caused by making that change is a good idea at this point in the schedule (Michael’s heuristic below makes sense).

Doug

On Aug 17, 2014, at 5:57 PM, Michael Still <mikal at stillhq.com> wrote:

> My recollection is that this was a request from the oslo team, but it
> was so long ago that I don't recall the details.
> 
> I think the change is low value, so should only be done when someone
> is changing the logging in a file already (the log hinting for
> example).
> 
> Michael
> 
> On Sun, Aug 17, 2014 at 4:26 PM, Gary Kotton <gkotton at vmware.com> wrote:
>> Hi,
>> Over the last few weeks I have seen a number of patches where LOG.warn is
>> replacing LOG.warning. I think that if we do something it should be the
>> opposite as warning is the documented one in python 2 and 3
>> https://docs.python.org/3/howto/logging.html.
>> Any thoughts?
>> Thanks
>> Gary
>> 
>> _______________________________________________
>> OpenStack-dev mailing list
>> OpenStack-dev at lists.openstack.org
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>> 
> 
> 
> 
> -- 
> Rackspace Australia
> 
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev