← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1306559] Re: Fix python26 compatibility for RFCSysLogHandler

 

For Ceilometer, this was merged in
https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=36638dde9d88431b7e61cce3835a01b80694f404
and released in juno-2

** Changed in: ceilometer
       Status: Triaged => Fix Released

** Changed in: ceilometer
     Assignee: Steve Lewis (steve-lewis) => (unassigned)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1306559

Title:
  Fix python26 compatibility for RFCSysLogHandler

Status in OpenStack Telemetry (Ceilometer):
  Fix Released
Status in Cinder:
  Confirmed
Status in OpenStack Image Registry and Delivery Service (Glance):
  Confirmed
Status in Orchestration API (Heat):
  Fix Released
Status in OpenStack Identity (Keystone):
  Confirmed
Status in Murano:
  Fix Committed
Status in OpenStack Neutron (virtual network service):
  Confirmed
Status in OpenStack Compute (Nova):
  Confirmed
Status in Oslo - a Library of Common OpenStack Code:
  Fix Released
Status in OpenStack Data Processing (Sahara, ex. Savanna):
  Fix Released

Bug description:
  Currently used pattern https://review.openstack.org/#/c/63094/15/openstack/common/log.py (lines 471-479)  will fail for Python 2.6.x.
  In order to fix the broken Python 2.6.x compatibility, old style explicit superclass method calls should be used instead.

  Here is an example of how to check this for Python v2.7 and v2.6: 
  import logging.handlers
  print type(logging.handlers.SysLogHandler)
  print type(logging.Handler)

  Results would be:
  Python 2.7: <type 'type'>, so super() may be used for RFCSysLogHandler(logging.handlers.SysLogHandler)
  Python 2.6:<type 'classobj'>, so super() may *NOT* be used for RFCSysLogHandler(logging.handlers.SysLogHandler)

To manage notifications about this bug go to:
https://bugs.launchpad.net/ceilometer/+bug/1306559/+subscriptions


References