yahoo-eng-team team mailing list archive

[ZhaoFengshou <zhao.fengshou@xxxxxxxxx>]

Public bug reported:

release: Havana
file: neutron/openstack/common/rpc/dispather.py
Code:
def dispatch(self, ctxt, version, method, namespace, **kwargs):
    ......
    for proxyobj in self.callbacks:
        ......
        if is_compatible:
            kwargs = self._deserialize_args(ctxt, kwargs)
            result = getattr(proxyobj, method)(ctxt, **kwargs)
            return self.serializer.serialize_entity(ctxt, result)
        ......

Method dispatch will return after call the specified method of the FIRST
proxyobj in self.callbacks. Currently, this code is harmless, because
there is only one ProxyObject in self.callbacks, but the logic is not
right.

** Affects: neutron
     Importance: Undecided
         Status: New


** Tags: rpc

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

Title:
  RPC Dispather implementation error

Status in OpenStack Neutron (virtual network service):
  New

Bug description:
  release: Havana
  file: neutron/openstack/common/rpc/dispather.py
  Code:
  def dispatch(self, ctxt, version, method, namespace, **kwargs):
      ......
      for proxyobj in self.callbacks:
          ......
          if is_compatible:
              kwargs = self._deserialize_args(ctxt, kwargs)
              result = getattr(proxyobj, method)(ctxt, **kwargs)
              return self.serializer.serialize_entity(ctxt, result)
          ......

  Method dispatch will return after call the specified method of the
  FIRST proxyobj in self.callbacks. Currently, this code is harmless,
  because there is only one ProxyObject in self.callbacks, but the logic
  is not right.

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