diff options
author | Andrey Nikitin <anikitin@mirantis.com> | 2016-03-21 13:08:38 +0300 |
---|---|---|
committer | Andrey Nikitin <anikitin@mirantis.com> | 2016-03-21 13:09:04 +0300 |
commit | 1f6efd534c78a729e4f418cd34ff92b0d23b9539 (patch) | |
tree | 85aec4974d193ae29e3a1617dcc4f76ae1f97bc8 | |
parent | b6e8639f67e4fe78228306cc7f70b89bbe4238b5 (diff) |
Order of the class parameters is refactored
Order and intendation of those parameters are changed
to follow Puppet Style Guide recommendation [0].
Moreover, it will allow to an user to find much faster
a variable in a list of variables.
[0]. https://docs.puppetlabs.com/guides/style_guide.html
Change-Id: I4dd4727f94132ce066d42aa87a246dd39c13553e
Notes
Notes (review):
Code-Review+2: yolanda.robla <yolanda.robla-mota@hpe.com>
Code-Review+1: Ricardo Carrillo Cruz <ricardo.carrillo.cruz@gmail.com>
Code-Review+1: Kayu Kiang <kayu.kiang@ericsson.com>
Code-Review+2: Paul Belanger <pabelanger@redhat.com>
Workflow+1: Paul Belanger <pabelanger@redhat.com>
Verified+2: Jenkins
Submitted-by: Jenkins
Submitted-at: Wed, 23 Mar 2016 15:23:17 +0000
Reviewed-on: https://review.openstack.org/295179
Project: openstack-infra/puppet-logrotate
Branch: refs/heads/master
-rw-r--r-- | manifests/file.pp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/manifests/file.pp b/manifests/file.pp index 88df974..3eef1c9 100644 --- a/manifests/file.pp +++ b/manifests/file.pp | |||
@@ -3,11 +3,11 @@ | |||
3 | define logrotate::file ( | 3 | define logrotate::file ( |
4 | $log, | 4 | $log, |
5 | $options, | 5 | $options, |
6 | $ensure=present, | 6 | $ensure = present, |
7 | $prerotate='undef', | 7 | $firstaction = 'undef', |
8 | $postrotate='undef', | 8 | $lastaction = 'undef', |
9 | $firstaction='undef', | 9 | $postrotate = 'undef', |
10 | $lastaction='undef', | 10 | $prerotate = 'undef', |
11 | ) { | 11 | ) { |
12 | 12 | ||
13 | # $options should be an array containing 1 or more logrotate | 13 | # $options should be an array containing 1 or more logrotate |