Public bug reported:

_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

** Affects: neutron
     Importance: Undecided
     Assignee: Pavel Bondar (pasha117)
         Status: In Progress

** Changed in: neutron
     Assignee: (unassigned) => Pavel Bondar (pasha117)

-- 
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:
  In Progress

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

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to     : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp

Reply via email to