Add support for 'nice' argument.

manifests/service.pp

@@ -38,6 +38,7 @@
 #   $access_times   - optional
 #   $log_type       - optional
 #   $bind           - optional
+#   $nice           - optional
 #
 # Actions:
 #   setups up a xinetd service by creating a file in /etc/xinetd.d/
@@ -57,6 +58,7 @@
 #     cps         => '100 2',
 #     flags       => 'IPv4',
 #     per_source  => '11',
+#     nice        => '19',
 #   } # xinetd::service
 #
 define xinetd::service (
@@ -86,7 +88,8 @@
   $no_access               = undef,
   $access_times            = undef,
   $log_type                = undef,
-  $bind                    = undef
+  $bind                    = undef,
+  $nice                    = undef
 ) {
 
   include xinetd
@@ -101,6 +104,13 @@
     }
   }
 
+  if $nice {
+    validate_re($nice,'^-?[0-9]+$')
+    if !is_numeric($nice) or $nice < -19 or $nice > 19 {
+      fail("Invalid value for nice, ${nice}")
+    }
+  }
+
   # Template uses:
   # - $port
   # - $disable
@@ -126,6 +136,7 @@
   # - $no_access
   # - $access_types
   # - $log_type
+  # - $nice
   file { "${xinetd::confdir}/${title}":
     ensure  => $ensure,
     owner   => 'root',

spec/defines/xinetd_service_spec.rb

@@ -86,4 +86,25 @@
     }
   end
 
+  # nice values, good
+  ['-19','9','19'].each do |i|
+    describe "with nice #{i}" do
+      let :params do
+        default_params.merge({ :nice => i })
+      end
+      it {
+        should contain_file('/etc/xinetd.d/httpd').with_content(
+          /nice\s*=\s*#{i}/)
+      }
+    end
+  end
+  # nice values, bad
+  ['-20','90','foo'].each do |i|
+    describe "with nice #{i}" do
+      let :params do
+        default_params.merge({ :nice => i })
+      end
+      it { expect { should compile }.to raise_error(Puppet::Error) }
+    end
+  end
 end

templates/service.erb

@@ -51,4 +51,7 @@ service <%= @service_name %>
 <% if @log_type -%>
         log_type        = <%= @log_type %>
 <% end -%>
+<% if @nice -%>
+        nice            = <%= @nice %>
+<% end -%>
 }