From 8548f12eb2e37565a0ca7b78e9a22275430ce489 Mon Sep 17 00:00:00 2001
From: Robert Nishihara <robertnishihara@gmail.com>
Date: Tue, 26 Mar 2019 14:54:32 -0700
Subject: [PATCH] Give better error when include_webui=1 and webui  can't be
 started. (#4471)

---
 python/ray/node.py     | 4 ++--
 python/ray/services.py | 3 +--
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/python/ray/node.py b/python/ray/node.py
index 875d15799..e5deec232 100644
--- a/python/ray/node.py
+++ b/python/ray/node.py
@@ -316,8 +316,8 @@ class Node(object):
             self.all_processes[ray_constants.PROCESS_TYPE_DASHBOARD] = [
                 process_info
             ]
-        redis_client = self.create_redis_client()
-        redis_client.hmset("webui", {"url": self._webui_url})
+            redis_client = self.create_redis_client()
+            redis_client.hmset("webui", {"url": self._webui_url})
 
     def start_plasma_store(self):
         """Start the plasma store."""
diff --git a/python/ray/services.py b/python/ray/services.py
index 4bfa4719c..61e4fb9a6 100644
--- a/python/ray/services.py
+++ b/python/ray/services.py
@@ -924,10 +924,9 @@ def start_dashboard(redis_address,
         import aiohttp  # noqa: F401
         import psutil  # noqa: F401
     except ImportError:
-        logger.warning(
+        raise ImportError(
             "Failed to start the dashboard. The dashboard requires Python 3 "
             "as well as 'pip install aiohttp psutil'.")
-        return None, None
 
     process_info = start_ray_process(
         command,