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

Add missing curl to engine Dockerfile #285

Merged
merged 1 commit into from
Nov 5, 2018
Merged

Add missing curl to engine Dockerfile #285

merged 1 commit into from
Nov 5, 2018

Conversation

ukclivecox
Copy link
Contributor

@ukclivecox ukclivecox commented Nov 5, 2018

Used in preStop to get Tomcat server to pause and no longer be "ready"

Fixes #283

@ukclivecox ukclivecox merged commit 323de50 into SeldonIO:master Nov 5, 2018
@ukclivecox ukclivecox deleted the curl_missing branch February 14, 2020 11:36
agrski pushed a commit that referenced this pull request Dec 2, 2022
* small update to docs

* optimise computation of reserved memory

* reconfigure model gateway for 1 consumer

* kafka k8s config

* update kafka config

* add publish model as go routine

* simplify goroutine call

* add lock around sort and update to make it atomic

* tidy up

* remove pprof from scheduler

* remove redundant struct

* add a test for working out reserved memory

* remove go routine

* small mod to k8s Makefile

* fix lint

* add map for unique models

* abstract add / delete to struct

* add option for shallow copy

* optimise create server status

* fix test

* add consistency lock for computing len

* Revert "update kafka config"

This reverts commit 2449d3cfbe0ac18e3050a28c9db9bc5890532aef.

* Revert "small mod to k8s Makefile"

This reverts commit 82a99bef126c15495157e4f062375926b81d9c10.

* Revert "kafka k8s config"

This reverts commit ee6b5dddb0fd1ffc3c9cdcb9b552494ac9fe39cd.

* Revert "reconfigure model gateway for 1 consumer"

This reverts commit e3295fa3792aeaaa1b0fdacad6d225ced5973710.

* Review comments

* add note about object leakage

* add simple test

* fix lint

Co-authored-by: Clive Cox <cc@seldon.io>
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

Successfully merging this pull request may close these issues.

curl not found error
1 participant