Brocade plugin options have no help text

[Tom Fifield]

In the Brocade Quantum Plugin (quantum/plugins/brocade/QuantumPlugin.py), options are registered without help text:

 64 SWITCH_OPTS = [cfg.StrOpt('address', default=''),
 65 cfg.StrOpt('username', default=''),
 66 cfg.StrOpt('password', default='', secret=True),
 67 cfg.StrOpt('ostype', default='NOS')
 68 ]
 69
 70 PHYSICAL_INTERFACE_OPTS = [cfg.StrOpt('physical_interface', default='eth0')
 71 ]
 72
 73 cfg.CONF.register_opts(SWITCH_OPTS, "SWITCH")
 74 cfg.CONF.register_opts(PHYSICAL_INTERFACE_OPTS, "PHYSICAL_INTERFACE")
 75 cfg.CONF.register_opts(scheduler.AGENTS_SCHEDULER_OPTS)

I'm not familiar with the expectations of quantum development, but I'd suspect that having help text for every option would be good practice ...

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/44953
Committed: http://github.com/openstack/neutron/commit/e541ca285d5a0f484c886fb28514b84580c3dc02
Submitter: Jenkins
Branch: master

commit e541ca285d5a0f484c886fb28514b84580c3dc02
Author: Justin Hammond <email address hidden>
Date: Tue Sep 3 15:57:52 2013 -0500

    Adds help text to brocade configuration options

    Help text was added to the configuration options defined in the brocade neutron
    plugin. This help text should assist users of the brocade plugin. The OSTYPE
    configuration option was not used in the plugin and was labeled as such so a
    user of the plugin would not worry about the meaning of it.

    The example etc file /etc/neutron/plugins/brocade/brocade.ini has been updated
    to share the same help values where applicable.

    Fixes-Bug: #1157511

    Change-Id: Ie068396e77902037a7a0b5334d4dfb0dcdd3575e