← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1348479] [NEW] _extend_extra_router_dict does not handle boolean correctly

 

Public bug reported:

Method:

https://github.com/openstack/neutron/blob/master/neutron/db/l3_attrs_db.py#L50

is used to add extension attributes to the router object during the
handling of the API response. When attributes are unspecified, the
router is extended with default values.

In the case of boolean attributes things don't work as they should,
because a default value as True takes over on the right side of the
boolean expression on:

https://github.com/openstack/neutron/blob/master/neutron/db/l3_attrs_db.py#L56

The end user so is led to believe that the server did not honor the
request, when effectively it did.

** Affects: neutron
     Importance: Undecided
     Assignee: Armando Migliaccio (armando-migliaccio)
         Status: New

** Changed in: neutron
     Assignee: (unassigned) => Armando Migliaccio (armando-migliaccio)

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

Title:
  _extend_extra_router_dict does not handle boolean correctly

Status in OpenStack Neutron (virtual network service):
  New

Bug description:
  Method:

  https://github.com/openstack/neutron/blob/master/neutron/db/l3_attrs_db.py#L50

  is used to add extension attributes to the router object during the
  handling of the API response. When attributes are unspecified, the
  router is extended with default values.

  In the case of boolean attributes things don't work as they should,
  because a default value as True takes over on the right side of the
  boolean expression on:

  https://github.com/openstack/neutron/blob/master/neutron/db/l3_attrs_db.py#L56

  The end user so is led to believe that the server did not honor the
  request, when effectively it did.

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


Follow ups

References