More Code Cleanup

Modified README.md

README.md

@@ -26,6 +26,8 @@ This module was tested and works great within RedHat Satellite 6.1.
 
 Make sure touch is within the path /usr/bin or /sbin
 
+The default command_var variable is set to "/sbin/shutdown -ra now" which will reboot your server. The intended purpose of this was originally to allow scheduled reboots, but I modified it to do more.
+
 ##Beginning with run once
 
 ```puppet

manifests/init.pp

@@ -28,23 +28,23 @@
  $dt_current = regsubst(generate('/bin/date', '+%s'),'\n','')
  $dt_start = regsubst(generate('/bin/date', '-d',$datetime_start,'+%s'),'\n','')
  $dt_end = regsubst(generate('/bin/date', '-d',$datetime_end,'+%s'),'\n','')
- $check_file = '${check_file_prefix}.${dt_start}'
+ $check_file = "${check_file_prefix}.${dt_start}"
 
  # Check if Class is disabled (Helps to disable some hosts)
  if $class_enabled == true {
  # Check if Date/Time is within allowed period
  if $dt_start <= $dt_current and $dt_end >= $dt_current {
  # Prep Command that is to be run
- $command_string = 'touch ${check_file};${command_var}'
- exec { "scheduled_reboot_command":
+ $command_string = "touch ${check_file};${command_var}"
+ exec { 'scheduled_reboot_command':
  command => $command_string,
  user => root,
  onlyif => '/usr/bin/test ! -f ${check_file}',
  path => ['/usr/bin','/sbin'],
  }
- notice('Reboot command will run. ${command_string}')
+ notice("Reboot command will run. ${command_string}")
  } else {
- notice('Not the right time. Current Date/Time: ${dt_current}. Start Date/Time: ${dt_start}. End Date/Time: ${dt_end}.')
+ notice("Not the right time. Current Date/Time: ${dt_current}. Start Date/Time: ${dt_start}. End Date/Time: ${dt_end}.")
  }
  } else {
  notice('Class disabled.')

metadata.json

@@ -5,7 +5,7 @@
  "dependencies": [],
  "license": "Apache-2.0",
  "name": "Ramorous-scheduled_runonce",
- "version": "0.1.15",
+ "version": "0.1.16",
  "operatingsystem_support": [
  {
  "operatingsystem": "RedHat",