separate the plugin code into backend specific part and generic part

Bug #1567770 reported by YAMAMOTO Takashi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tap-as-a-service
Fix Released
Undecided
Yoichiro Iura

Bug Description

currently the plugin and db mixin have some backend specific logic hardcoded.
it's better to separate them into their own classes, or a service driver.
so that generic part can be used by other implementations.

examples of such logic:
- agent rpc (TaasAgentApi, TaasCallbacks)
- vlan id management (TapIdAssociation)

an example of an alternative backend, which doesn't need those logic:
https://review.openstack.org/#/c/301843/
it's sdn-controller backed (vs agent) and uses a different
encapsulation (non-vlan)

description: updated
summary: - separate the plugin into backend specific part and generic part
+ separate the plugin code into backend specific part and generic part
description: updated
Changed in tap-as-a-service:
assignee: nobody → Yoichiro Iura (iura-yoichiro)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tap-as-a-service (master)

Fix proposed to branch: master
Review: https://review.openstack.org/307708

Changed in tap-as-a-service:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tap-as-a-service (master)

Reviewed: https://review.openstack.org/307708
Committed: https://git.openstack.org/cgit/openstack/tap-as-a-service/commit/?id=5035aa4a57e7641e6bb202f61ded18c5956e705c
Submitter: Jenkins
Branch: master

commit 5035aa4a57e7641e6bb202f61ded18c5956e705c
Author: Yoichiro Iura <email address hidden>
Date: Tue Apr 19 09:33:45 2016 +0000

    Support service framework to separate plugin logic

    This fix supports service framework in order to separate
    the plugin code into backend specific and generic part.

    Change-Id: I66d0f4a9ce5453ec7ca0bf0fe52f24a3a2e3c8ea
    Closes-Bug: #1567770

Changed in tap-as-a-service:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.