From 16c2d3717eb118fd09808a268414f34630410524 Mon Sep 17 00:00:00 2001 From: Ganesh Nalawade Date: Wed, 11 Apr 2018 12:07:56 +0530 Subject: [PATCH] Add junos connection type check (#38527) junos_netconf works only with connection=netconf Add check to report appropriate error if any other connection type is used. --- lib/ansible/plugins/action/junos.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/ansible/plugins/action/junos.py b/lib/ansible/plugins/action/junos.py index 6f61701103..64913c14be 100644 --- a/lib/ansible/plugins/action/junos.py +++ b/lib/ansible/plugins/action/junos.py @@ -90,7 +90,8 @@ class ActionModule(_ActionModule): if any(provider.values()): display.warning('provider is unnecessary when using connection=%s and will be ignored' % self._play_context.connection) - if self._play_context.connection == 'network_cli' and self._task.action not in CLI_SUPPORTED_MODULES: + if (self._play_context.connection == 'network_cli' and self._task.action not in CLI_SUPPORTED_MODULES) or \ + (self._play_context.connection == 'netconf' and self._task.action == 'junos_netconf'): return {'failed': True, 'msg': "Connection type '%s' is not valid for '%s' module. " "Please see http://docs.ansible.com/ansible/latest/network/user_guide/platform_junos.html" % (self._play_context.connection, self._task.action)}