Add missing puppetdoc and lint all parameter documentation

Un-pin puppet-lint gem and add puppet-lint-param-docs, this commit also
add missing puppetdoc.

Change-Id: Id9843816fc3106f4d2e2052e1cdf4fd18efc194b

Gemfile

@@ -4,7 +4,7 @@ group :development, :test do
   gem 'puppetlabs_spec_helper', :require => false
 
   gem 'metadata-json-lint'
-  gem 'puppet-lint', '~> 1.1.0'
+  gem 'puppet-lint-param-docs'
   gem 'puppet-lint-absolute_classname-check'
   gem 'puppet-lint-absolute_template_path'
   gem 'puppet-lint-trailing_newline-check'

manifests/cells.pp

@@ -16,9 +16,12 @@
 # License for the specific language governing permissions and limitations
 # under the License.
 #
-# The nova::cells class installs the Nova Cells
+# == Class: nova::cells
+#
+# Installs the Nova Cells
+#
+# === Parameters:
 #
-# == Parameters
 #  [*enabled*]
 #    Use Nova Cells or not
 #    Defaults to 'False'
@@ -35,6 +38,10 @@
 #    Cells communication driver to use
 #    Defaults to 'nova.cells.rpc_driver.CellsRPCDriver'
 #
+#  [*ensure_package*]
+#    Desired ensure state of packages.
+#    Defaults to present
+#
 #  [*instance_updated_at_threshold*]
 #    Number of seconds after an instance was updated or deleted to continue to update cells
 #    Defaults to '3600'
@@ -133,7 +140,6 @@
 #    It might be used by some cell scheduling code in the future
 #    Defaults to '1.0'
 #
-
 class nova::cells (
   $bandwidth_update_interval     = '600',
   $call_timeout                  = '60',

manifests/compute.pp

@@ -99,7 +99,7 @@
 #   (optional) The availability zone to show internal services under.
 #   Defaults to internal
 #
-#  [*pci_passthrough_whitelist*]
+#  [*pci_passthrough*]
 #   (optional) Pci passthrough hash in format of:
 #   Defaults to undef
 #   Example

manifests/compute/serial.pp

@@ -4,7 +4,7 @@
 #
 # === Parameters:
 #
-# [*port_range]
+# [*port_range*]
 #   (optional) Range of TCP ports to use for serial ports on compute hosts
 #   Defaults to 10000:20000
 #
@@ -12,7 +12,7 @@
 #   (optional) URL that gets passed to the clients
 #   Defaults to 'ws://127.0.0.1:6083/'
 #
-# [*listen]
+# [*listen*]
 #   IP address on which instance serial console should listen
 #   Defaults to 127.0.0.1
 #
@@ -20,8 +20,7 @@
 #   The address to which proxy clients (like nova-serialproxy)
 #   should connect (string value)
 #   Defaults to 127.0.0.1
-
+#
-
 class nova::compute::serial(
   $port_range            = '10000:20000',
   $base_url              = 'ws://127.0.0.1:6083/',

manifests/config.pp

@@ -20,6 +20,13 @@
 #   NOTE: The configuration MUST NOT be already handled by this module
 #   or Puppet catalog compilation will fail with duplicate resources.
 #
+# [*nova_paste_api_ini*]
+#   (optional) Allow configuration of arbitrary Nova paste api configurations.
+#   The value is an hash of nova_paste_api_ini resources. Example:
+#   { 'DEFAULT/foo' => { value => 'fooValue'},
+#     'DEFAULT/bar' => { value => 'barValue'}
+#   }
+#
 class nova::config (
   $nova_config        = {},
   $nova_paste_api_ini = {},

manifests/init.pp

@@ -68,6 +68,10 @@
 #   (optional) Connect over SSL for RabbitMQ
 #   Defaults to false
 #
+# [*rabbit_ha_queues*]
+#   (optional) Use HA queues in RabbitMQ.
+#   Defaults to undef
+#
 # [*kombu_ssl_ca_certs*]
 #   (optional) SSL certification authority file (valid only if SSL enabled).
 #   Defaults to undef
@@ -122,6 +126,10 @@
 #   (optional) Disable Nagle algorithm
 #   Defaults to true
 #
+# [*auth_strategy*]
+#   (optional) The strategy to use for auth: noauth or keystone.
+#   Defaults to 'keystone'
+#
 # [*service_down_time*]
 #   (optional) Maximum time since last check-in for up service.
 #   Defaults to 60
@@ -144,6 +152,10 @@
 #   (optional) Set log output to verbose output.
 #   Defaults to false
 #
+# [*debug*]
+#   (optional) Set log output to debug output.
+#   Defaults to false
+#
 # [*periodic_interval*]
 #   (optional) Seconds between running periodic tasks.
 #   Defaults to '60'
@@ -152,6 +164,10 @@
 #   (optional) Interval at which nodes report to data store.
 #    Defaults to '10'
 #
+# [*rootwrap_config*]
+#   (optional) Path to the rootwrap configuration file to use for running commands as root
+#   Defaults to '/etc/nova/rootwrap.conf'
+#
 # [*monitoring_notifications*]
 #   (optional) Whether or not to send system usage data notifications out on the message queue. Only valid for stable/essex.
 #   Defaults to false
@@ -164,6 +180,10 @@
 #   (optional) Syslog facility to receive log lines.
 #   Defaults to 'LOG_USER'
 #
+# [*install_utilities*]
+#   (optional) Install nova utilities (Extra packages used by nova tools)
+#   Defaults to true,
+#
 # [*use_ssl*]
 #   (optional) Enable SSL on the API server
 #   Defaults to false, not set
@@ -228,6 +248,7 @@
 #   and you have multiple endpoints, you will get AmbiguousEndpoint
 #   exceptions in the nova API service.
 #   Defaults to undef
+#
 class nova(
   $ensure_package           = 'present',
   $database_connection      = false,

manifests/network.pp

@@ -26,6 +26,10 @@
 #   split into.
 #   Defaults to 1
 #
+# [*network_size*]
+#   (optional) Number of addresses in each private subnet.
+#   Defaults to 255
+#
 # [*floating_range*]
 #   (optional) Range of floating ip addresses to create.
 #   Defaults to false

manifests/network/flat.pp

@@ -7,7 +7,7 @@
 # [*fixed_range*]
 #   (required) The IPv4 CIDR for the network
 #
-# [flat_interface]
+# [*flat_interface*]
 #   (optional) Interface that flat network will use for bridging
 #   Defaults to undef
 #
@@ -15,7 +15,7 @@
 #   (optional) The interface to use for public traffic
 #   Defaults to undef
 #
-# [flat_network_bridge]
+# [*flat_network_bridge*]
 #   (optional) The name of the bridge to use
 #   Defaults to 'br100'
 #

manifests/scheduler/filter.pp

@@ -40,7 +40,7 @@
 #   (optional) Ignore hosts that have too many instances
 #   Defaults to '50'
 #
-# [*ram_allocation_ratio:*]
+# [*ram_allocation_ratio*]
 #   (optional) Virtual ram to physical ram allocation ratio
 #   Defaults to '1.5'
 #