search

datarevenue-berlin / OpenMLOps

MIT License
697 stars 101 forks source link

MLOPS 349: Seldon storage initializer image and values.yaml #51

Closed bernardolk closed 3 years ago

bernardolk commented 3 years ago

I have moved the set value from main.tf to the values.yaml file and added a different image to the storageInitializer thing, so that we don't have to change it after deploying using kubectl get configmaps -n seldon

bernardolk commented 3 years ago

Needs to be tested and a review

tvinagre commented 3 years ago

What's this storageInitializer for? Didn't get it

bernardolk commented 3 years ago

What's this storageInitializer for? Didn't get it

That's something we had to update manually before, not sure what it is for, but maybe @pipatth or @carloszanella can explain it better

carloszanella commented 3 years ago

@bernardolk @tvinagre https://docs.seldon.io/projects/seldon-core/en/v1.6.0/servers/overview.html#init-containers