]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Typographical correction of Arista ML2 help
authorStephen Gordon <sgordon@redhat.com>
Tue, 8 Apr 2014 01:43:51 +0000 (21:43 -0400)
committerStephen Gordon <sgordon@redhat.com>
Tue, 8 Apr 2014 01:45:49 +0000 (21:45 -0400)
The Configuration Reference Guide on docs.openstack.org contains
configuration tables generated based on the help strings defined
in the Python code.

Typographical errors in the source for the Arista ML2 plugin,
particularly a lack of spaces between the start and end of
continuing lines, were resulting in errors in the guide.

Change-Id: I793274d9bf631db890af57f242b99de99db6f1f6

neutron/plugins/ml2/drivers/mech_arista/config.py

index c4bdfe7c6329f39a9237c95b6291f32b8b434681..2f968c874ddb6a840fa62abd6f6faeff83e561d0 100644 (file)
@@ -28,42 +28,43 @@ OpenStack/Neutron controllers.
 ARISTA_DRIVER_OPTS = [
     cfg.StrOpt('eapi_username',
                default='',
-               help=_('Username for Arista EOS. This is required field.'
-                      'if not set, all communications to Arista EOS'
-                      'will fail')),
+               help=_('Username for Arista EOS. This is required field. '
+                      'If not set, all communications to Arista EOS'
+                      'will fail.')),
     cfg.StrOpt('eapi_password',
                default='',
                secret=True,  # do not expose value in the logs
-               help=_('Password for Arista EOS. This is required field.'
-                      'if not set, all communications to Arista EOS'
-                      'will fail')),
+               help=_('Password for Arista EOS. This is required field. '
+                      'If not set, all communications to Arista EOS '
+                      'will fail.')),
     cfg.StrOpt('eapi_host',
                default='',
-               help=_('Arista EOS IP address. This is required field.'
+               help=_('Arista EOS IP address. This is required field. '
                       'If not set, all communications to Arista EOS'
-                      'will fail')),
+                      'will fail.')),
     cfg.BoolOpt('use_fqdn',
                 default=True,
-                help=_('Defines if hostnames are sent to Arista EOS as FQDNs'
-                       '("node1.domain.com") or as short names ("node1").'
-                       'This is optional. If not set, a value of "True"'
+                help=_('Defines if hostnames are sent to Arista EOS as FQDNs '
+                       '("node1.domain.com") or as short names ("node1"). '
+                       'This is optional. If not set, a value of "True" '
                        'is assumed.')),
     cfg.IntOpt('sync_interval',
                default=180,
-               help=_('Sync interval in seconds between Neutron plugin and'
-                      'EOS. This interval defines how often the'
-                      'synchronization is performed. This is an optional'
-                      'field. If not set, a value of 180 seconds is assumed')),
+               help=_('Sync interval in seconds between Neutron plugin and '
+                      'EOS. This interval defines how often the '
+                      'synchronization is performed. This is an optional '
+                      'field. If not set, a value of 180 seconds is '
+                      'assumed.')),
     cfg.StrOpt('region_name',
                default='RegionOne',
-               help=_('Defines Region Name that is assigned to this OpenStack'
-                      'Controller. This is useful when multiple'
-                      'OpenStack/Neutron controllers are managing the same'
-                      'Arista HW clusters. Note that this name must match with'
-                      'the region name registered (or known) to keystone'
-                      'service. Authentication with Keysotne is performed by'
-                      'EOS. This is optional. If not set, a value of'
-                      '"RegionOne" is assumed'))
+               help=_('Defines Region Name that is assigned to this OpenStack '
+                      'Controller. This is useful when multiple '
+                      'OpenStack/Neutron controllers are managing the same '
+                      'Arista HW clusters. Note that this name must match '
+                      'with the region name registered (or known) to keystone '
+                      'service. Authentication with Keysotne is performed by '
+                      'EOS. This is optional. If not set, a value of '
+                      '"RegionOne" is assumed.'))
 ]
 
 cfg.CONF.register_opts(ARISTA_DRIVER_OPTS, "ml2_arista")