reorder params to place optional ones at the end

voxpupuli · Mar 21, 2017 · 01abb70 · 01abb70
1 parent 5c9aecd
commit 01abb70
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/manifests/mount.pp b/manifests/mount.pp
@@ -38,16 +38,16 @@
 #
 define gluster::mount (
   String $volume,
-  Optional[String] $log_level                                           = undef,
-  Optional[String] $log_file                                            = undef,
-  Optional[String] $transport                                           = undef,
-  Optional[String] $direct_io_mode                                      = undef,
-  Optional[Boolean] $readdirp                                           = undef,
   $atboot                                                               = yes,
   String $options                                                       = 'defaults',
   Integer $dump                                                         = 0,
   Integer $pass                                                         = 0,
   Enum['defined', 'present', 'unmounted', 'absent', 'mounted'] $ensure  = 'mounted',
+  Optional[String] $log_level                                           = undef,
+  Optional[String] $log_file                                            = undef,
+  Optional[String] $transport                                           = undef,
+  Optional[String] $direct_io_mode                                      = undef,
+  Optional[Boolean] $readdirp                                           = undef,
 ) {
 
   if $log_level {

diff --git a/manifests/volume.pp b/manifests/volume.pp
@@ -41,14 +41,14 @@
 #
 define gluster::volume (
   Boolean $force                              = false,
-  Optional[Integer] $stripe                   = undef,
-  Optional[Integer] $replica                  = false,
   Enum['tcp', 'rdma', 'tcp,rdma'] $transport  = 'tcp',
   Boolean $rebalance                          = true,
   Boolean $heal                               = true,
   Array $bricks                               = undef,
-  Optional[Array] $options                    = undef,
   Boolean $remove_options                     = false,
+  Optional[Array] $options                    = undef,
+  Optional[Integer] $stripe                   = undef,
+  Optional[Integer] $replica                  = false,
 ) {
 
   if $force {