[autoscaler] Bug for file mounts for tilde

[richardliaw]

filemount: {"x" : "~/directory/"} currently does not work because the path isn't expanded.

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/6592/
Test PASSed.

[ericl]

It's not correct to expand on the client side I believe, since the server home dir may be different. Cc @hartikainen

[richardliaw]

What is an example of the incorrect case? In any case, right now setting the local path as `~/dir/file` is fine while `~/dir/` errors.

…

On Tue, Jul 10, 2018 at 9:41 PM Eric Liang ***@***.***> wrote: It's not correct to expand on the client side I believe, since the server home dir may be different. Cc @hartikainen <https://github.com/hartikainen> — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#2382 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEUc5WZf6HvSLNmEPCkqW42y1xuufiwhks5uFQNJgaJpZM4VIRtj> .

[ericl]

Hm I see, this is the read side path expansion only, so it looks fine. The mount point is resolved on the server side only.