SeldonIO / seldon-operator

Seldon Core Operator for Kubernetes
Apache License 2.0
12 stars 10 forks source link

TFServing container: override Docker entrypoint. #67

Closed desaintmartin closed 4 months ago

desaintmartin commented 5 years ago

Default Docker entrypoint is a bash script for easy tfserving start. We don't need it since we re-define all arguments and we don't want it since it will swallow all OOM & co informations. https://github.com/tensorflow/serving/blame/master/tensorflow_serving/tools/docker/Dockerfile\#L59.

Also minor changes (that I can remove and/or dedicate to their PRs): TFServer: add container ports names. prepackaged servers: directly return if implementation is not what we want.

ukclivecox commented 5 years ago

@desaintmartin We have moved the Operator to be part of the main SeldonCore repo now. Can you do the PR there? https://github.com/SeldonIO/seldon-core/tree/master/operator