Searchlight.service OPTS aren't included in sample conf file.

[Travis Tripp]

Running the config generator as described in the dev env setup [0]
instructions does not include all the options for the service.
This adds those options.

[0] http://docs.openstack.org/developer/searchlight/dev-environment.html

None of the options here: https://github.com/openstack/searchlight/blob/master/searchlight/service.py#L30-L45

are in the generated oslo sample config file.

This is because they aren't included here:

https://github.com/openstack/searchlight/blob/master/searchlight/opts.py

This should add the following: ('service', service.OPTS),

Please note there is another bug that actually is somewhat related: https://bugs.launchpad.net/searchlight/+bug/1493975 The configuration between API service and Listener service is intermingled.

[Travis Tripp (travis-tripp)]

https://review.openstack.org/#/c/254354/

[Liyingjun (liyingjun)]

https://review.openstack.org/#/c/253973/

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/253973
Committed: https://git.openstack.org/cgit/openstack/searchlight/commit/?id=d35dc3676a64d106f3eeacdf628449b869482785
Submitter: Jenkins
Branch: master

commit d35dc3676a64d106f3eeacdf628449b869482785
Author: liyingjun <email address hidden>
Date: Sun Nov 29 23:54:21 2015 +0800

    Set listener workers according to config

    Currently, the get_workers method always return 1, so config
    listener_workers will never work as expected. This patch adds
    the ability to specify number of workers for listener service.
    The default is equal to the number of CPUs available.
    Also, register searchlight.service.OPTS to group 'listener' in
    searchlight.listener.
    Remove not used options 'host' and 'http_timeout'.

    Change-Id: Ifc39d5eaab1a7b0239631aa1b2b6e0af3a07f0c9
    Closes-bug: #1522737
    Closes-bug: #1523639

[OpenStack Infra (hudson-openstack)]

Change abandoned by Travis Tripp (<email address hidden>) on branch: master
Review: https://review.openstack.org/254354
Reason: This patch is superseded by these:

https://review.openstack.org/#/c/253973/7
https://review.openstack.org/#/c/253981/7