Stopping RQD killframe exception when frame is not running (#171)

* Fixes Issue #168 . Dont have rqd throw an exception when the frame is not running

rqd/rqd/rqcore.py

@@ -802,9 +802,9 @@ def launchFrame(self, runFrame):
     def getRunningFrame(self, frameId):
         try:
             return self.__cache[frameId]
-        except:
-            raise KeyError("frameId {} is not running on this"
-                           "machine".format(frameId))
+        except KeyError:
+            log.info("frameId {} is not running on this machine".format(frameId))
+            return None
 
     def reportStatus(self, current=None):
         """Replies with hostReport"""

rqd/rqd/rqdservicers.py

@@ -1,6 +1,8 @@
 
 import logging as log
 
+import grpc
+
 from compiled_proto import rqd_pb2
 from compiled_proto import rqd_pb2_grpc
 
@@ -26,14 +28,22 @@ def GetRunningFrameStatus(self, request, context):
         """RPC call to return the frame info for the given frame id"""
         log.info("Request received: getRunningFrameStatus")
         frame = self.rqCore.getRunningFrame(request.frameId)
-        return rqd_pb2.RqdStaticGetRunningFrameStatusResponse(
-            running_frame_info=frame.runningFrameInfo())
+        if frame:
+            return rqd_pb2.RqdStaticGetRunningFrameStatusResponse(
+                running_frame_info=frame.runningFrameInfo())
+        else:
+            context.set_details(
+                "The requested frame was not found. frameId: {}".format(request.frameId))
+            context.set_code(grpc.StatusCode.NOT_FOUND)
+            return rqd_pb2.RqdStaticGetRunningFrameStatusResponse()
+
 
     def KillRunningFrame(self, request, context):
         """RPC call that kills the running frame with the given id"""
         log.info("Request received: killRunningFrame")
         frame = self.rqCore.getRunningFrame(request.frame_id)
-        frame.kill()
+        if frame:
+            frame.kill()
         return rqd_pb2.RqdStaticKillRunningFrameResponse()
 
     def ShutdownRqdNow(self, request, context):