The "file://" storage type does not work

/kind bug In the document and the python message it’s said the “file://” storage type is supported, but there’s no way to use it. It should be deprecated since we already have the pvc type.

What steps did you take and what happened: [A clear and concise description of what the bug is.] No matter what I set, there’s kind of message like:

INFO:root:Initializing, args: src_uri [file://test] dest_path[ [/mnt/models]
INFO:root:Copying contents of file://test to local
Traceback (most recent call last):
  File "/storage-initializer/scripts/initializer-entrypoint", line 14, in <module>
    kfserving.Storage.download(src_uri, dest_path)
  File "/usr/local/lib/python3.7/site-packages/kfserving/storage.py", line 55, in download
    return Storage._download_local(uri, out_dir)
  File "/usr/local/lib/python3.7/site-packages/kfserving/storage.py", line 179, in _download_local
    raise Exception("Local path %s does not exist." % (uri))
Exception: Local path file://test does not exist.

What did you expect to happen: Remove the “file://” type.

Anything else you would like to add: [Miscellaneous information that will assist in solving the issue.]

Environment:

• Istio Version:
• Knative Version:
• KFServing Version:
• Kubeflow version:
• Minikube version:
• Kubernetes version: (use kubectl version):
• OS (e.g. from /etc/os-release):

[pugangxa]

I see. After PR #156 the local file in fact means pvc. The “file://” type storage is not supported so I removed it in PR #622

[aurorazl]

I submit a PR #1081 to support storageUri like file://.