← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1404782] [NEW] ml2: superfluous %s in LOG.debug() format

 

Public bug reported:

ml2.db.get_dynamic_segment() includes this line:

   LOG.debug("No dynamic segment %s found for "
	     "Network:%(network_id)s, "
	     "Physical network:%(physnet)s, "
	     "segmentation_id:%(segmentation_id)s",
	     {'network_id': network_id,
	      'physnet': physical_network,
	      'segmentation_id': segmentation_id})

Note the superfluous %s in the format string.  At run-time, %s prints
the args hash again and doesn't cause an error, but this is clearly
unintended.

** Affects: neutron
     Importance: Undecided
     Assignee: Angus Lees (gus)
         Status: In Progress

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

Title:
  ml2: superfluous %s in LOG.debug() format

Status in OpenStack Neutron (virtual network service):
  In Progress

Bug description:
  ml2.db.get_dynamic_segment() includes this line:

     LOG.debug("No dynamic segment %s found for "
  	     "Network:%(network_id)s, "
  	     "Physical network:%(physnet)s, "
  	     "segmentation_id:%(segmentation_id)s",
  	     {'network_id': network_id,
  	      'physnet': physical_network,
  	      'segmentation_id': segmentation_id})

  Note the superfluous %s in the format string.  At run-time, %s prints
  the args hash again and doesn't cause an error, but this is clearly
  unintended.

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


Follow ups

References