Tracking server image #6731

oojo12 · 2022-09-08T04:49:07Z

Related Issues/PRs

What changes are proposed in this pull request?

This PR introduces a minimal Dockerfile for the MlFLow tracking server as well as a docker compose file that can be invoked for building and pushing all MlFlow docker images.

How is this patch tested?

Requires maintainer testing to ensure the associated workflow PR that goes along with this properly automatically tags and pushes the image to GitHub container registry.

Does this PR change the documentation?

No. You can skip the rest of this section.

Release Notes

Is this a user-facing change?

[X ] Yes. Give a description of this change to be included in the release notes for MLflow users.

Introduction of Docker image for the MlFlow tracking server.

What component(s), interfaces, languages, and integrations does this PR affect?

Components

area/build: Build and test infrastructure for MLflow

Interface

area/docker: Docker use across MLflow's components, such as MLflow Projects and MLflow Models

How should the PR be classified in the release notes? Choose one:

rn/feature - A new user-facing feature worth mentioning in the release notes

mlflow/tracking/Dockerfile

mlflow/docker-compose.yml

mlflow/tracking/Dockerfile

harupy · 2022-09-12T01:13:25Z

docker/docker-compose.yml

+version: "3.9"  # optional since v1.27.0
+services:
+  tracking:
+    build: .
+    args:
+        VERSION: ${GITHUB_REF_NAME}
+    entrypoint: ["mlflow", "server"]


Is this file used in the workflow added in ~~#6735~~ #6732?

you mean #6732? No, it isn't and has been deleted.

harupy · 2022-09-13T07:30:18Z

@oojo12 Thanks for the update! Can you fix the DCO check failure?

Signed-off-by: Femi <47154698+oojo12@users.noreply.github.com> Signed-off-by: Femi Oho <47154698+oojo12@users.noreply.github.com>

Signed-off-by: Femi Oho <47154698+oojo12@users.noreply.github.com>

Signed-off-by: Femi <47154698+oojo12@users.noreply.github.com> Signed-off-by: Femi Oho <47154698+oojo12@users.noreply.github.com>

oojo12 · 2022-09-13T14:48:02Z

@harupy DCO failure resolved.

harupy

LGTM!

Rusteam · 2022-10-06T04:30:41Z

docker/Dockerfile

@@ -0,0 +1,3 @@
+FROM python:3.10.6-slim-bullseye
+ARG VERSION
+RUN pip install --no-cache mlflow==$VERSION


It complains about not having psycopg2 installed when using with postgres

@Rusteam can you create an issue with a reproducible example, since this PR has already been merged? With the reproducible example I can identify and rectify any missing dependencies.

@harupy do you think it is better to add this to the requirements.txt for mlflow?

* Create docker-compose.yml Signed-off-by: Femi <47154698+oojo12@users.noreply.github.com> Signed-off-by: Femi Oho <47154698+oojo12@users.noreply.github.com> * Create Dockerfile Signed-off-by: Femi <47154698+oojo12@users.noreply.github.com> Signed-off-by: Femi Oho <47154698+oojo12@users.noreply.github.com> * pip install from Pypi Signed-off-by: Femi <47154698+oojo12@users.noreply.github.com> Signed-off-by: Femi Oho <47154698+oojo12@users.noreply.github.com> * organized docker folder Signed-off-by: Femi Oho <47154698+oojo12@users.noreply.github.com> * remove unnecessary comment Signed-off-by: Femi <47154698+oojo12@users.noreply.github.com> Signed-off-by: Femi Oho <47154698+oojo12@users.noreply.github.com> * Delete docker-compose.yml Signed-off-by: Femi <47154698+oojo12@users.noreply.github.com> Signed-off-by: Femi Oho <47154698+oojo12@users.noreply.github.com> Signed-off-by: Femi <47154698+oojo12@users.noreply.github.com> Signed-off-by: Femi Oho <47154698+oojo12@users.noreply.github.com>

github-actions bot added area/build Build and test infrastructure for MLflow area/docker Docker use anywhere, such as MLprojects and MLmodels rn/feature Mention under Features in Changelogs. labels Sep 8, 2022