Deduplicate remote_api.py code.

Deduplicate remote_api.py code.

Isolate and swarming apps used to have different implementations of
remote_api.py. Now they defer to components/tools/remote_api.py (that
can also be invoked as a standalone script).

Also add readline support with command history, just like in standard
remote_api_shell.py GAE tool.

R=csharp@chromium.org, maruel@chromium.org
BUG=

Committed: https://code.google.com/p/swarming/source/detail?repo=default&r=5323d95

[vadimsh, 2014-04-25 03:21:26]

https://codereview.appspot.com/92770043/diff/1/services/components/tools/remo...
File services/components/tools/remote_api.py (right):

https://codereview.appspot.com/92770043/diff/1/services/components/tools/remo...
services/components/tools/remote_api.py:144: auth =
KeyringAuth('remote_api_pwd_%s' % socket.getfqdn())
You'd have to reenter your ASP once to create a new key ring entry. There were
"remote_api_isolate_%s" here.

On a bright side, it will be reused for other remote_api invocation (i.e.
isolate and swarming share same keyring entry).

I looked at using OAuth2 here. It's not really supported :(

https://codereview.appspot.com/92770043/diff/1/services/components/tools/remo...
services/components/tools/remote_api.py:149: remote_api_stub.ConfigureRemoteApi(
It's a modern name of ConfigureRemoteDatastore. They point to exact same
function.

[vadimsh, 2014-04-25 18:18:47]

ping in case you've missed this

[csharp, 2014-04-25 18:30:54]

(reviewing for M-A so he can focus on all the swarm cl comments I gave him today
:)

https://codereview.appspot.com/92770043/diff/60001/services/components/tools/...
File services/components/tools/remote_api.py (right):

https://codereview.appspot.com/92770043/diff/60001/services/components/tools/...
services/components/tools/remote_api.py:198: if not app_dir:
Could you swap the if statement bodies so this line can be "if app_dir:"

https://codereview.appspot.com/92770043/diff/60001/services/components/tools/...
services/components/tools/remote_api.py:221: #
https://developers.google.com/appengine/docs/python/modules/
I'm not sure what the point of this comment is, could you add an explanation.

https://codereview.appspot.com/92770043/diff/60001/services/components/tools/...
services/components/tools/remote_api.py:232: # Import all modules app wants to
be present in interactive console.
nit: "app" -> "the app"

https://codereview.appspot.com/92770043/diff/60001/services/components/tools/...
services/components/tools/remote_api.py:234: if setup_context_callback:
Move this to line 238 to prevent call update on None.

[vadimsh, 2014-04-25 19:43:48]

https://codereview.appspot.com/92770043/diff/60001/services/components/tools/...
File services/components/tools/remote_api.py (right):

https://codereview.appspot.com/92770043/diff/60001/services/components/tools/...
services/components/tools/remote_api.py:198: if not app_dir:
On 2014/04/25 18:30:54, csharp wrote:
> Could you swap the if statement bodies so this line can be "if app_dir:"

Done. Here and in another file. I'll probably move that copy-pasted chunk of
code into a separate module once I have all the tools in components/tools/

https://codereview.appspot.com/92770043/diff/60001/services/components/tools/...
services/components/tools/remote_api.py:221: #
https://developers.google.com/appengine/docs/python/modules/
On 2014/04/25 18:30:54, csharp wrote:
> I'm not sure what the point of this comment is, could you add an explanation. 

Removed.

It was present in original remote_api.py file in isolate/tools. I think it's an
explanation for '<version>-dot-<module>-dot-<app_id>' syntax for host name.

https://codereview.appspot.com/92770043/diff/60001/services/components/tools/...
services/components/tools/remote_api.py:232: # Import all modules app wants to
be present in interactive console.
On 2014/04/25 18:30:54, csharp wrote:
> nit: "app" -> "the app"

Done.

https://codereview.appspot.com/92770043/diff/60001/services/components/tools/...
services/components/tools/remote_api.py:234: if setup_context_callback:
On 2014/04/25 18:30:54, csharp wrote:
> Move this to line 238 to prevent call update on None.
Removed the check, |context| can't be None.

[csharp, 2014-04-25 19:49:09]

lgtm

[vadimsh, 2014-04-25 20:46:45]

Committed patchset #5 manually as r5323d95 (presubmit successful).