Open Stack

On 4/28/2014 3:03 PM, Denis Makogon wrote:
> Good day, Boden.
>
> I think you should file the blueprint for it and put it into BP meeting
> agenda.
>
> Best regards,
> Denis Makogon
>
> On Mon, Apr 28, 2014 at 9:50 PM, boden <boden at linux.vnet.ibm.com
> <mailto:boden at linux.vnet.ibm.com>> wrote:
>
>     Guys,
>     I have a few small features / enhancements I'd like to suggest. I'm
>     willing to contribute the code / unit tests myself, but am looking
>     for a consensus from the group before I invest the time.
>
>     There are a few enhancements on my list -- I will send details each
>     in a separate email to keep the communication clearer.
>
>
>
>     I'd like to propose the ability to support a pluggable trove
>     conductor manager. Currently the trove conductor manager is
>     hard-coded [1][2] and thus is always
>     'trove.conductor.manager.__Manager'. I'd like to see this conductor
>     manager class be pluggable like nova does [3].
>
>     I'm thinking the same approach nova took:
>     (a) Add a conductor section to the trove-conductor.conf with a
>     property to specify the conductor manager class... e.g.
>     [conductor]
>     manager = package.of.conductor.manager.__Manager
>
>     (b) Default trove's CONF value for conductor.manager to the existing
>     manager trove.conductor.manager.__Manager which ensures backwards
>     compat.
>
>     (c) In trove.cmd.conductor.py <http://trove.cmd.conductor.py> create
>     the RpcService using the conf value. e.g.
>     server = rpc_service.RpcService(__manager=CONF.conductor.__manager,
>     topic=topic)
>
>
>     The above will allow consumers to extend / plug into the conductor
>     manager without upstream changes.
>
>
>     Any disagreement / comments on this enhancement? Again -- I can
>     contribute the code, unless someone is bored and wants to run with
>     it short-term.
>
>     Thanks
>
>
>     [1]
>     https://github.com/openstack/__trove/blob/master/trove/cmd/__conductor.py#L40
>     <https://github.com/openstack/trove/blob/master/trove/cmd/conductor.py#L40>
>     [2]
>     https://github.com/openstack/__trove/blob/master/trove/cmd/__conductor.py#L42
>     <https://github.com/openstack/trove/blob/master/trove/cmd/conductor.py#L42>
>     [3]
>     https://github.com/openstack/__nova/blob/master/nova/cmd/__conductor.py#L43
>     <https://github.com/openstack/nova/blob/master/nova/cmd/conductor.py#L43>
>
>
>     _________________________________________________
>     OpenStack-dev mailing list
>     OpenStack-dev at lists.openstack.__org
>     <mailto:OpenStack-dev at lists.openstack.org>
>     http://lists.openstack.org/__cgi-bin/mailman/listinfo/__openstack-dev <http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev>
>
>
>
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>

Denis thanks...

Done, and done --
https://blueprints.launchpad.net/trove/+spec/pluggable-conductor-manager
https://wiki.openstack.org/wiki/Meetings/TroveBPMeeting#Weekly_Trove_Blueprint_Meeting

I missed today's BP meeting, but I will plan to attend the next one.