This wasn't checking for a key called "Success" instead of "ReturnValue" in the response. Probably just a leftover from early days.

Author: nielsvaes

skyhook/client.py

@@ -91,7 +91,7 @@ def is_host_online(self):
         :return: *bool*
         """
         response = self.execute("is_online", {})
-        return response.get("Success")
+        return response.get(Results.return_value)
 
 
     def execute(self, command, parameters={}, timeout=0):