Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Can we attach pvc to model-initializer(storage-initializer) ? #1146

Closed
hemantha-kumara opened this issue Nov 27, 2019 · 5 comments
Closed

Can we attach pvc to model-initializer(storage-initializer) ? #1146

hemantha-kumara opened this issue Nov 27, 2019 · 5 comments

Comments

@hemantha-kumara
Copy link

Hi,
I am trying to run XGBOOST_SERVER serving by using model present in pv

apiVersion: machinelearning.seldon.io/v1alpha2
kind: SeldonDeployment
metadata:
  name: xgboost
spec:
  name: iris
  predictors:
  - graph:
      children: []
      implementation: XGBOOST_SERVER
      modelUri: file://seldon-models/xgboost/iris
      name: classifier
    name: default
    replicas: 1 

but problem is I am not able to attach pvc to model-initializer and hence model is not accesible to storage-initializer.
Due to mentioned problem i am not able to serve models.

can anyone pls help me to attach pvc to init container?

@hemantha-kumara hemantha-kumara changed the title Can we attach pvc to storage-initializer Can we attach pvc to model-initializer(storage-initializer) ? Nov 27, 2019
@ryandawsonuk
Copy link
Contributor

ryandawsonuk commented Nov 27, 2019

The format is pvc://. It proved tricky to get working with tensorflow in a recent issue but we did manage it.

@hemantha-kumara
Copy link
Author

Thanks @ryandawsonuk
pvc://<claim name> solving initcontainer pvc issue.

@hemantha-kumara
Copy link
Author

hemantha-kumara commented Nov 27, 2019

I am trying serving in on-prem airgap deployments where docker.io is not accessible,
as DefaultXGBoostServerImageNameRest is hardcoded to use docker.io, now serving pod is going to ImagePullBackOff error.

@ryandawsonuk
Copy link
Contributor

@hemantha-kumara
Copy link
Author

Thanks @ryandawsonuk
I was expecting #1118 changes and thanks again for pointing fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants