Merge pull request #29 from NilashishC/master

ansible-zuul[bot] · web-flow · commit 4a13ec04be7a · 2019-05-21T07:13:51.000Z
Some minor changes to templates Reviewed-by: Nilashish Chakraborty <nilashishchakraborty8@gmail.com> https://github.com/NilashishC
diff --git a/roles/resource_module/templates/module_utils/network_os/config/resource/resource.py.j2 b/roles/resource_module/templates/module_utils/network_os/config/resource/resource.py.j2
@@ -31,7 +31,7 @@ class {{ resource|capitalize }}(ConfigBase, {{ resource|capitalize }}Args):
     ]
 
     gather_network_resources = [
-        {{ resource }},
+        '{{ resource }}',
     ]
 
     def get_{{ resource }}_facts(self):
diff --git a/roles/resource_module/templates/module_utils/network_os/facts/facts.py.j2 b/roles/resource_module/templates/module_utils/network_os/facts/facts.py.j2
@@ -75,7 +75,8 @@ class Facts(FactsArgs, FactsBase): #pylint: disable=R0903
         :rtype: dict
         :returns: the facts gathered
         """
-	self.ansible_facts['gather_network_resources'] = list()
+        warnings = []
+        self.ansible_facts['gather_network_resources'] = list()
         self.ansible_facts['gather_subset'] = list()
 
         valid_network_resources_subsets = self.argument_spec['gather_network_resources'].get('choices', [])

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ class {{ resource\|capitalize }}(ConfigBase, {{ resource\|capitalize }}Args):`
`31`	`31`	`]`
`32`	`32`
`33`	`33`	`gather_network_resources = [`
`34`		`- {{ resource }},`
	`34`	`+ '{{ resource }}',`
`35`	`35`	`]`
`36`	`36`
`37`	`37`	`def get_{{ resource }}_facts(self):`