yahoo-eng-team team mailing list archive

[Pavel Bondar <pbondar@xxxxxxxxxxxx>]

** Changed in: neutron
       Status: Fix Committed => Fix Released

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

Title:
  Updating ips for port incorrectly pass port dict in pluggable IPAM

Status in neutron:
  Fix Released

Bug description:
  _ipam_allocate_ips expects to receive port dict as a third argument,
  but 'changes' are passed instead [1].

  This issue do not lead to any misbehaviour for reference IPAM driver,
  because port dict is not used in default AddressRequestFactory[2].

  Port dict in IPAM interface is expected to be used in custom
  AddressRequestFactory, that allows to create specific address requests
  based on port dict info. Found this bug during implementing custom
  AddressRequestFactory for Infoblox IPAM Driver.

  Affects:
  This issue affects only third-party vendors who implements own IPAM driver with custom AddressRequestFactory and uses port dict in it.

  [1] https://github.com/openstack/neutron/blob/master/neutron/db/ipam_pluggable_backend.py#L287
  [2] https://github.com/openstack/neutron/blob/master/neutron/ipam/requests.py#L258

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