refactor(ruby): refactor patch_server.rb task, add timeout to apt get patching

Author: rwaffen

REFERENCE.md

@@ -353,25 +353,25 @@ Any additional parameters to include in the zypper update command
 
 ##### `reboot`
 
-Data type: `Optional[Variant[Boolean, Enum['always', 'never', 'patched', 'smart']]]`
+Data type: `Variant[Boolean, Enum['always', 'never', 'patched', 'smart']]`
 
 Should the server reboot after patching has been applied? (Defaults to 'never')
 
 ##### `timeout`
 
-Data type: `Optional[Integer]`
+Data type: `Integer`
 
 How many seconds should we wait until timing out the patch run? (Defaults to 3600 seconds)
 
 ##### `security_only`
 
-Data type: `Optional[Boolean]`
+Data type: `Boolean`
 
 Limit patches to those tagged as security related? (Defaults to false)
 
 ##### `clean_cache`
 
-Data type: `Optional[Boolean]`
+Data type: `Boolean`
 
 Should the yum/dpkg caches be cleaned at the start of the task? (Defaults to false)
 

tasks/patch_server.json

@@ -15,19 +15,23 @@
     },
     "reboot": {
       "description": "Should the server reboot after patching has been applied? (Defaults to 'never')",
-      "type": "Optional[Variant[Boolean, Enum['always', 'never', 'patched', 'smart']]]"
+      "type": "Variant[Boolean, Enum['always', 'never', 'patched', 'smart']]",
+      "default": "never"
     },
     "timeout": {
       "description": "How many seconds should we wait until timing out the patch run? (Defaults to 3600 seconds)",
-      "type": "Optional[Integer]"
+      "type": "Integer",
+      "default": 3600
     },
     "security_only": {
       "description": "Limit patches to those tagged as security related? (Defaults to false)",
-      "type": "Optional[Boolean]"
+      "type": "Boolean",
+      "default": false
     },
     "clean_cache": {
       "description": "Should the yum/dpkg caches be cleaned at the start of the task? (Defaults to false)",
-      "type": "Optional[Boolean]"
+      "type": "Boolean",
+      "default": false
     }
   },
   "implementations": [