yahoo-eng-team team mailing list archive

[Angus Lees <1404785@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

cisco.db.n1kv_db_v2._validate_segment_range_uniqueness() includes these
lines:

   msg = (_("NetworkProfile name %s already exists"),
          net_p["name"])
   LOG.error(msg)
   raise n_exc.InvalidInput(error_message=msg)

As written, msg is a tuple, and the various logging lines below print
the tuple members without properly expanding the format string as
intended.

The format in msg should have been expanded using % - as was
presumably the intention.  There are few other examples of this
elsewhere in this file.

** 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/1404785

Title:
  Cisco: logging incorrectly called with (fmt, arg) tuple

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

Bug description:
  cisco.db.n1kv_db_v2._validate_segment_range_uniqueness() includes these
  lines:

     msg = (_("NetworkProfile name %s already exists"),
            net_p["name"])
     LOG.error(msg)
     raise n_exc.InvalidInput(error_message=msg)

  As written, msg is a tuple, and the various logging lines below print
  the tuple members without properly expanding the format string as
  intended.

  The format in msg should have been expanded using % - as was
  presumably the intention.  There are few other examples of this
  elsewhere in this file.

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