From ed2d46bc9eec69ebc3123b4157f510f98f1dd05a Mon Sep 17 00:00:00 2001
From: Peter Sprygada <psprygada@ansible.com>
Date: Fri, 11 Mar 2016 22:08:12 -0500
Subject: [PATCH] bugfix in eos_template for backing up config

eos_template would connect to the remote device to get the running config
for backup even if backup is not needed.
---
 .../modules/network/eos/eos_template.py       | 29 +++++++++++++------
 1 file changed, 20 insertions(+), 9 deletions(-)

diff --git a/lib/ansible/modules/network/eos/eos_template.py b/lib/ansible/modules/network/eos/eos_template.py
index 51d7aeef5de..a4b38347d5c 100644
--- a/lib/ansible/modules/network/eos/eos_template.py
+++ b/lib/ansible/modules/network/eos/eos_template.py
@@ -36,8 +36,7 @@ options:
         file with config or a template that will be merged during
         runtime.  By default the task will search for the source
         file in role or playbook root folder in templates directory.
-    required: false
-    default: null
+    required: true
   force:
     description:
       - The force argument instructs the module to not consider the
@@ -46,7 +45,16 @@ options:
         without first checking if already configured.
     required: false
     default: false
-    choices: BOOLEANS
+    choices: ['yes', 'no']
+  include_defaults:
+    description:
+      - By default when the M(eos_template) connects to the remote
+        device to retrieve the configuration it will issue the `show
+        running-config` command.  If this option is set to True then
+        the issued command will be `show running-config all`
+    required: false
+    default: false
+    choices: ['yes', 'no']
   backup:
     description:
       - When this argument is configured true, the module will backup
@@ -55,7 +63,7 @@ options:
         the root of the playbook directory.
     required: false
     default: false
-    choices: BOOLEANS
+    choices: ['yes', 'no']
   replace:
     description:
       - This argument will cause the provided configuration to be replaced
@@ -65,7 +73,7 @@ options:
         is eapi.
     required: false
     default: false
-    choice: BOOLEANS
+    choices: ['yes', 'no']
   config:
     description:
       - The module, by default, will connect to the remote device and
@@ -138,7 +146,7 @@ def flatten(data, obj):
     return obj
 
 def get_config(module):
-    config = module.params['config'] or dict()
+    config = module.params.get('config')
     if not config and not module.params['force']:
         config = module.config
     return config
@@ -148,7 +156,7 @@ def main():
     """
 
     argument_spec = dict(
-        src=dict(),
+        src=dict(required=True),
         force=dict(default=False, type='bool'),
         include_defaults=dict(default=False, type='bool'),
         backup=dict(default=False, type='bool'),
@@ -169,9 +177,12 @@ def main():
     candidate = module.parse_config(module.params['src'])
 
     contents = get_config(module)
-    result['_backup'] = module.config
 
-    config = module.parse_config(contents)
+    if contents:
+        config = module.parse_config(contents)
+        result['_backup'] = contents
+    else:
+        config = dict()
 
     commands = collections.OrderedDict()
     toplevel = [c.text for c in config]