Merge 9fb1623 into 0898dfd

aimmo-game/service.py

@@ -101,9 +101,7 @@ def _find_avatar_id_from_query(self, session_id, query_string):
 
     def _send_logs(self, player_id_to_workers):
         def should_send_logs(logs):
-            LOGGER.info("should_send_logs: " + str(logs))
-
-            return logs is not None and logs != ''
+            return bool(logs)
 
         socket_session_id_to_player_id_copy = self._socket_session_id_to_player_id.copy()
         for sid, player_id in socket_session_id_to_player_id_copy.iteritems():

aimmo_runner/minikube.py

@@ -1,6 +1,8 @@
 #!/user/bin/env python
 from __future__ import print_function
 
+from subprocess import CalledProcessError
+
 import docker
 import kubernetes
 import os
@@ -74,10 +76,10 @@ def start_cluster(minikube):
     Starts the cluster unless it has been already started by the user.
     :param minikube: Executable minikube installed beforehand.
     """
-    status = run_command([minikube, 'status'], True)
-    if 'minikube: Running' in status:
+    try:
+        run_command([minikube, 'status'], True)
         print('Cluster already running')
-    else:
+    except CalledProcessError:
         run_command([minikube, 'start', '--memory=2048', '--cpus=2'])