From c3ccf26b7027e7c282d3313d2dd58571b7431e84 Mon Sep 17 00:00:00 2001 From: Brian Coca Date: Sun, 14 Jun 2015 23:45:56 -0400 Subject: [PATCH] added become check back to connections --- lib/ansible/plugins/connections/__init__.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/lib/ansible/plugins/connections/__init__.py b/lib/ansible/plugins/connections/__init__.py index 449d1379ef6..921c4e38825 100644 --- a/lib/ansible/plugins/connections/__init__.py +++ b/lib/ansible/plugins/connections/__init__.py @@ -63,10 +63,10 @@ class ConnectionBase(with_metaclass(ABCMeta, object)): if not hasattr(self, '_connected'): self._connected = False - def _become_method_supported(self, become_method): + def _become_method_supported(self): ''' Checks if the current class supports this privilege escalation method ''' - if become_method in self.__class__.become_methods: + if self._connection_info.become_method in self.__class__.become_methods: return True raise AnsibleError("Internal Error: this connection module does not support running commands via %s" % become_method) @@ -90,7 +90,10 @@ class ConnectionBase(with_metaclass(ABCMeta, object)): @abstractmethod def _connect(self): """Connect to the host we've been initialized with""" - pass + + # Check if PE is supported + if self._connection_info.become: + self.__become_method_supported() @ensure_connect @abstractmethod