yahoo-eng-team team mailing list archive

[OpenStack Infra <1784950@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.openstack.org/588096
Committed: https://git.openstack.org/cgit/openstack/networking-powervm/commit/?id=59fbf0a3b6f14d54a0369408a8bc6bb2ad9df874
Submitter: Zuul
Branch:    master

commit 59fbf0a3b6f14d54a0369408a8bc6bb2ad9df874
Author: Matthew Edmonds <edmondsw@xxxxxxxxxx>
Date:   Wed Aug 1 18:14:44 2018 -0400

    Pass host on get_device_details RPC
    
    Neutron added an optional host argument to the get_device_details RPC
    method a long time ago [1] but a recent change [2] has made that no
    longer really optional, at least for our usage. This adds host to our
    calls so that VIF plug will not timeout with an error in the neutron
    logs stating "Device %s has no active binding in host None".
    
    [1] f7064f2b6c6ba1d0ab5f9872b2d5ad7969a64e7b
    [2] 01bdb47199468805b714ce4c00c7492951267585
    
    Closes-Bug: #1784950
    Change-Id: Ida4102427405e17e42ddc463df4a63c2ba7dcb82


** Changed in: networking-powervm
       Status: In Progress => 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/1784950

Title:
  get_device_details RPC fails if host not specified

Status in networking-powervm:
  Fix Released
Status in neutron:
  New

Bug description:
  An optional (defaults to None) host argument was added to the
  get_device_details RPC method a long time ago [1] but a recent change
  [2] to the master branch has made that no longer really optional, at
  least for the pvm_sea agent from openstack/networking-powervm, since
  not passing it will cause VIF plugging to timeout with an error in the
  neutron logs stating "Device %s has no active binding in host None".

  This can easily be fixed in openstack/networking-powervm by passing
  the host argument, but I expect that neutron also needs to bump the
  version for neutron.plugins.ml2.rpc.RpcCallbacks to reflect that host
  is no longer optional by removing the "=None" default (since it
  doesn't work anymore).

  [1] f7064f2b6c6ba1d0ab5f9872b2d5ad7969a64e7b
  [2] 01bdb47199468805b714ce4c00c7492951267585

To manage notifications about this bug go to:
https://bugs.launchpad.net/networking-powervm/+bug/1784950/+subscriptions