fixed compatibility for ansible 1.5

lib/battleschool/__init__.py

@@ -1,2 +1,2 @@
-__version__ = '0.3.0'
+__version__ = '0.3.1'
 __author__ = 'Spencer Gibb'

lib/battleschool/printing.py

@@ -72,13 +72,20 @@ class BattleschoolRunnerCallbacks(DefaultRunnerCallbacks):
     """ callbacks for use by battles """
 
     def get_play(self):
+        if self.runner:
+            return self.runner
         if self.task:
             return self.task
         return self.play
 
+    def get_name(self):
+        if self.runner:
+            return self.get_play().module_name
+        return self.get_play().name
+
     def on_failed(self, host, res, ignore_errors=False):
         if not ignore_errors:
-            display("\tTask FAILED: %s %s" % (self.get_play().name, res['msg']), color="red")
+            display("\tTask FAILED: %s %s" % (self.get_name(), res['msg']), color="red")
         super(BattleschoolRunnerCallbacks, self).on_failed(host, res, ignore_errors=ignore_errors)
 
     def on_ok(self, host, res):
@@ -92,7 +99,7 @@ def on_ok(self, host, res):
 
         try:
             if self.get_play():
-                display("\tTask OK: %s%s" % (self.get_play().name, msg))
+                display("\tTask OK: %s%s" % (self.get_name(), msg))
         except AttributeError:
             invocation = res['invocation']
             msg = invocation['module_args']
@@ -122,15 +129,15 @@ def on_unreachable(self, host, results):
         super(BattleschoolRunnerCallbacks, self).on_unreachable(host, results)
 
     def on_skipped(self, host, item=None):
-        display("\tTask skipped: %s" % self.get_play().name, color="yellow")
+        display("\tTask skipped: %s" % self.get_name(), color="yellow")
         super(BattleschoolRunnerCallbacks, self).on_skipped(host, item)
 
     def on_error(self, host, err):
-        display("\tTask ERROR: %s%s" % (self.get_play().name, err), color="red")
+        display("\tTask ERROR: %s%s" % (self.get_name(), err), color="red")
         super(BattleschoolRunnerCallbacks, self).on_error(host, err)
 
     def on_no_hosts(self):
-        display("\tTask NO HOSTS: %s" % self.get_play().name, color="red")
+        display("\tTask NO HOSTS: %s" % self.get_name(), color="red")
         super(BattleschoolRunnerCallbacks, self).on_no_hosts()
 
     def on_async_poll(self, host, res, jid, clock):

lib/battleschool/source/git.py

@@ -23,6 +23,6 @@ def module_args(self, source):
             update = "yes"
             force = "yes"
 
-        module_args = "repo=%s dest=%s force=%s update=%s" % \
+        module_args = "repo=%s dest=%s force=%s update=%s " % \
                       (source['repo'], self.dest_dir(source), force, update)
         return module_args

lib/battleschool/source/url.py

@@ -23,7 +23,7 @@ def module_args(self, source):
         if self.options.update_sources:
             force = "yes"
 
-        module_args = "url=%s dest=%s/%s force=%s" % \
+        module_args = "url=%s dest=%s/%s force=%s validate_certs=no " % \
                       (source["url"], self.dest_dir(source), source["name"], force)
 
         if "playbooks" not in source: