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 <mi...@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 <gkot...@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@lists.openstack.org >> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev >> > > > > -- > Rackspace Australia > > _______________________________________________ > OpenStack-dev mailing list > OpenStack-dev@lists.openstack.org > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev _______________________________________________ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
