commit 2781fce238e79690bf707a3df4823aec422687bd
Author: Akihiro Motoki <email address hidden>
Date: Sat Aug 23 18:16:18 2014 +0900
Make SecurityGroupsRpcCallback a separate callback class
RPC has a version of itself. In Neutron a plugin implements
several RPC interface, so a single RPC version doesn't work.
In Mixin callback class approach, RPC versioning depends on
each plugin implementation and it makes harder to maintain
RPC version appropriately. This patch series replaces mixin
RPC callback of server side with a separate class.
This commit handles server-side callback of security group
RPC interface.
* The server-side callback of Security group RPC is moved to
api/rpc/handler and db/securitygroups_rpc_base now only
contains a mixin class to add agent-based security group
implementation with db operations.
* get_port_from_device method in server-side callback class
is moved to a mixin class of plugin implementation
(SecurityGroupServerRpcMixin) because it involves DB lookup
and is tightly coupled with plugin implementation rather
than RPC interface definition.
Most unit tests for SGServerRpcCallBackTestCase were skipped
in the base class before, but now they are no longer skipped.
The following items will be planned in later patches
to avoid drastic changes in a single patch.
* Merge security group RPC API and agent callback classes in
agent/securitygroups_rpc into api/rpc/handlers/securitygroup_rpc
* Remove completely duplicated db access code in get_port_from_device
and get_port_and_sgs
Reviewed: https:/ /review. openstack. org/116312 /git.openstack. org/cgit/ openstack/ neutron/ commit/ ?id=2781fce238e 79690bf707a3df4 823aec422687bd
Committed: https:/
Submitter: Jenkins
Branch: master
commit 2781fce238e7969 0bf707a3df4823a ec422687bd
Author: Akihiro Motoki <email address hidden>
Date: Sat Aug 23 18:16:18 2014 +0900
Make SecurityGroupsR pcCallback a separate callback class
RPC has a version of itself. In Neutron a plugin implements
several RPC interface, so a single RPC version doesn't work.
In Mixin callback class approach, RPC versioning depends on
each plugin implementation and it makes harder to maintain
RPC version appropriately. This patch series replaces mixin
RPC callback of server side with a separate class.
This commit handles server-side callback of security group rpc/handler and db/securitygrou ps_rpc_ base now only ation with db operations. from_device method in server-side callback class SecurityGroupSe rverRpcMixin) because it involves DB lookup
RPC interface.
* The server-side callback of Security group RPC is moved to
api/
contains a mixin class to add agent-based security group
implement
* get_port_
is moved to a mixin class of plugin implementation
(
and is tightly coupled with plugin implementation rather
than RPC interface definition.
Most unit tests for SGServerRpcCall BackTestCase were skipped
in the base class before, but now they are no longer skipped.
The following items will be planned in later patches securitygroups_ rpc into api/rpc/ handlers/ securitygroup_ rpc from_device
to avoid drastic changes in a single patch.
* Merge security group RPC API and agent callback classes in
agent/
* Remove completely duplicated db access code in get_port_
and get_port_and_sgs
Partial-Bug: #1359416 49a95667c1b53dd 09675002892
Change-Id: Ia6535217d2e3b8