openstack/puppet-vitrage
Code Issues Proposed changes

Fix parameters comments

Browse Source 
Fix comments format.
Add default values.

Change-Id: I000c184b573a9b6f487d8fd4f06e656131ac4af4
This commit is contained in:
ZhongShengping 2016-06-02 11:25:35 +08:00
parent 46171713bc
commit caa4e47bea
10 changed files with 133 additions and 124 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
manifests/api.pp
View File

@ -12,7 +12,7 @@
# #
# [*keystone_user*] # [*keystone_user*]
# (optional) The name of the auth user # (optional) The name of the auth user
# Defaults to vitrage # Defaults to 'vitrage'
# #
# [*keystone_tenant*] # [*keystone_tenant*]
# (optional) Tenant to authenticate with. # (optional) Tenant to authenticate with.
@ -21,22 +21,23 @@
# [*keystone_password*] # [*keystone_password*]
# Password to authenticate with. # Password to authenticate with.
# Mandatory. # Mandatory.
# Defaults to false.
# #
# [*keystone_auth_uri*] # [*keystone_auth_uri*]
# (optional) Public Identity API endpoint. # (optional) Public Identity API endpoint.
# Defaults to 'false'. # Defaults to false.
# #
# [*keystone_identity_uri*] # [*keystone_identity_uri*]
# (optional) Complete admin Identity API endpoint. # (optional) Complete admin Identity API endpoint.
# Defaults to: false # Defaults to false
# #
# [*host*] # [*host*]
# (optional) The vitrage api bind address. # (optional) The vitrage api bind address.
# Defaults to 0.0.0.0 # Defaults to '0.0.0.0'
# #
# [*port*] # [*port*]
# (optional) The vitrage api port. # (optional) The vitrage api port.
# Defaults to 8999 # Defaults to '8999'
# #
# [*package_ensure*] # [*package_ensure*]
# (optional) ensure state for package. # (optional) ensure state for package.
@ -51,8 +52,6 @@
# to make vitrage-api be a web app using apache mod_wsgi. # to make vitrage-api be a web app using apache mod_wsgi.
# Defaults to '$::vitrage::params::api_service_name' # Defaults to '$::vitrage::params::api_service_name'
# #
class vitrage::api ( class vitrage::api (
$manage_service = true, $manage_service = true,
$enabled = true, $enabled = true,

31
manifests/auth.pp
View File

@ -2,37 +2,36 @@
# #
# == Parameters # == Parameters
# [*auth_url*] # [*auth_url*]
# the keystone public endpoint # (Optional) The keystone public endpoint
# Optional. Defaults to 'http://localhost:5000' # Defaults to 'http://localhost:5000'
# #
# [*auth_region*] # [*auth_region*]
# the keystone region of this node # (Optional) The keystone region of this node
# Optional. Defaults to 'RegionOne' # Defaults to 'RegionOne'
# #
# [*auth_user*] # [*auth_user*]
# the keystone user for vitrage services # (Optional) The keystone user for vitrage services
# Optional. Defaults to 'vitrage' # Defaults to 'vitrage'
# #
# [*auth_password*] # [*auth_password*]
# the keystone password for vitrage services # (Required) The keystone password for vitrage services
# Required.
# #
# [*auth_tenant_name*] # [*auth_tenant_name*]
# the keystone tenant name for vitrage services # (Optional) The keystone tenant name for vitrage services
# Optional. Defaults to 'services' # Defaults to 'services'
# #
# [*auth_tenant_id*] # [*auth_tenant_id*]
# the keystone tenant id for vitrage services. # (Optional) The keystone tenant id for vitrage services.
# Optional. Defaults to $::os_service_default. # Defaults to $::os_service_default.
# #
# [*auth_cacert*] # [*auth_cacert*]
# Certificate chain for SSL validation. # (Optional) Certificate chain for SSL validation.
# Optional. Defaults to $::os_service_default. # Defaults to $::os_service_default.
# #
# [*auth_endpoint_type*] # [*auth_endpoint_type*]
# Type of endpoint in Identity service catalog to use for # (Optional) Type of endpoint in Identity service catalog to use for
# communication with OpenStack services. # communication with OpenStack services.
# Optional. Defaults to $::os_service_default. # Defaults to $::os_service_default.
# #
class vitrage::auth ( class vitrage::auth (
$auth_password, $auth_password,

3
manifests/client.pp
View File

@ -3,7 +3,8 @@
# #
# == parameters # == parameters
# [*ensure*] # [*ensure*]
# ensure state for package. # (Optional) Ensure state for package.
# Defaults to 'present'
# #
class vitrage::client ( class vitrage::client (
$ensure = 'present' $ensure = 'present'

4
manifests/db.pp
View File

@ -5,8 +5,8 @@
# === Parameters # === Parameters
# #
# [*database_connection*] # [*database_connection*]
# Url used to connect to database. # (Optional) Url used to connect to database.
# (Optional) Defaults to "mysql://vitrage:secrete@localhost:3306/vitrage". # Defaults to "mysql://vitrage:secrete@localhost:3306/vitrage".
# #
# [*database_max_retries*] # [*database_max_retries*]
# (Optional) Maximum number of database connection retries during startup. # (Optional) Maximum number of database connection retries during startup.

3
manifests/db/mysql.pp
View File

@ -7,7 +7,6 @@
# #
# [*password*] # [*password*]
# (Mandatory) Password to connect to the database. # (Mandatory) Password to connect to the database.
# Defaults to 'false'.
# #
# [*dbname*] # [*dbname*]
# (Optional) Name of the database. # (Optional) Name of the database.
@ -23,7 +22,7 @@
# #
# [*allowed_hosts*] # [*allowed_hosts*]
# (Optional) Other hosts the user is allowed to connect from. # (Optional) Other hosts the user is allowed to connect from.
# Defaults to 'undef'. # Defaults to undef.
# #
# [*charset*] # [*charset*]
# (Optional) The database charset. # (Optional) The database charset.

31
manifests/keystone/auth.pp
View File

@ -8,46 +8,55 @@
# (required) Password for vitrage user. # (required) Password for vitrage user.
# #
# [*auth_name*] # [*auth_name*]
# Username for vitrage service. Defaults to 'vitrage'. # (Optional) Username for vitrage service.
# Defaults to 'vitrage'.
# #
# [*email*] # [*email*]
# Email for vitrage user. Defaults to 'vitrage@localhost'. # (Optional) Email for vitrage user.
# Defaults to 'vitrage@localhost'.
# #
# [*tenant*] # [*tenant*]
# Tenant for vitrage user. Defaults to 'services'. # (Optional) Tenant for vitrage user.
# Defaults to 'services'.
# #
# [*configure_endpoint*] # [*configure_endpoint*]
# Should vitrage endpoint be configured? Defaults to 'true'. # (Optional) Should vitrage endpoint be configured?
# Defaults to true.
# #
# [*configure_user*] # [*configure_user*]
# (Optional) Should the service user be configured? # (Optional) Should the service user be configured?
# Defaults to 'true'. # Defaults to true.
# #
# [*configure_user_role*] # [*configure_user_role*]
# (Optional) Should the admin role be configured for the service user? # (Optional) Should the admin role be configured for the service user?
# Defaults to 'true'. # Defaults to true.
# #
# [*service_type*] # [*service_type*]
# Type of service. Defaults to 'rca'. # (Optional) Type of service.
# Defaults to 'rca'.
# #
# [*region*] # [*region*]
# Region for endpoint. Defaults to 'RegionOne'. # (Optional) Region for endpoint.
# Defaults to 'RegionOne'.
# #
# [*service_name*] # [*service_name*]
# (optional) Name of the service. # (optional) Name of the service.
# Defaults to the value of auth_name. # Defaults to the value of auth_name.
# #
# [*public_url*] # [*public_url*]
# (optional) The endpoint's public url. (Defaults to 'http://127.0.0.1:8999') # (optional) The endpoint's public url.
# This url should *not* contain any trailing '/'. # This url should *not* contain any trailing '/'.
# Defaults to 'http://127.0.0.1:8999'
# #
# [*admin_url*] # [*admin_url*]
# (optional) The endpoint's admin url. (Defaults to 'http://127.0.0.1:8999') # (optional) The endpoint's admin url.
# This url should *not* contain any trailing '/'. # This url should *not* contain any trailing '/'.
# Defaults to 'http://127.0.0.1:8999'
# #
# [*internal_url*] # [*internal_url*]
# (optional) The endpoint's internal url. (Defaults to 'http://127.0.0.1:8999') # (optional) The endpoint's internal url.
# This url should *not* contain any trailing '/'. # This url should *not* contain any trailing '/'.
# Defaults to 'http://127.0.0.1:8999'
# #
class vitrage::keystone::auth ( class vitrage::keystone::auth (
$password, $password,

4
manifests/logging.pp
View File

@ -81,12 +81,12 @@
# it like this (string value). # it like this (string value).
# Defaults to $::os_service_default # Defaults to $::os_service_default
# Example: instance_uuid_format='[instance: %(uuid)s] ' # Example: instance_uuid_format='[instance: %(uuid)s] '
#
# [*log_date_format*] # [*log_date_format*]
# (optional) Format string for %%(asctime)s in log records. # (optional) Format string for %%(asctime)s in log records.
# Defaults to $::os_service_default # Defaults to $::os_service_default
# Example: 'Y-%m-%d %H:%M:%S' # Example: 'Y-%m-%d %H:%M:%S'
#
class vitrage::logging( class vitrage::logging(
$use_syslog = $::os_service_default, $use_syslog = $::os_service_default,
$use_stderr = $::os_service_default, $use_stderr = $::os_service_default,

2
manifests/params.pp
View File

@ -1,3 +1,5 @@
# == Class: vitrage::params
#
# Parameters for puppet-vitrage # Parameters for puppet-vitrage
# #
class vitrage::params { class vitrage::params {

2
manifests/policy.pp
View File

@ -21,7 +21,7 @@
# #
# [*policy_path*] # [*policy_path*]
# (optional) Path to the nova policy.json file # (optional) Path to the nova policy.json file
# Defaults to /etc/vitrage/policy.json # Defaults to '/etc/vitrage/policy.json'
# #
class vitrage::policy ( class vitrage::policy (
$policies = {}, $policies = {},