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.

Sign up for GitHub

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

Jump to bottom

Mlflow model server #49

Merged
merged 1 commit into from
Aug 12, 2019
Merged

Mlflow model server #49

merged 1 commit into from
Aug 12, 2019

Conversation

axsaucedo
Copy link
Contributor

@axsaucedo axsaucedo commented Aug 9, 2019 •
edited
Loading

Pull request to support MLFlow server in PR #774, to close #771

@axsaucedo axsaucedo force-pushed the mlflow_model_server branch from 05ce4e0 to 7e79846 Compare August 10, 2019 10:17
@axsaucedo
Copy link
Contributor Author

@cliveseldon updated PR, now fully functional, and updated with master. It was tested by building the image, then changing the helm values in the seldon-core repo to IfNotPresent to ensure image is not overwritten. I then tested it with the latest Seldon Engine image built in #774 which contains support for the MLFlow server as well. The MLFlow server works as expected when sending requests.

@axsaucedo axsaucedo mentioned this pull request Aug 10, 2019
Merged
6 tasks
@axsaucedo axsaucedo changed the title WIP: Mlflow model server Mlflow model server Aug 12, 2019
@ukclivecox ukclivecox merged commit 03e2c6e into SeldonIO:master Aug 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@axsaucedo @ukclivecox