fix: analytics client not started correctly by fl-operator (#39)

The fl-operator expects any FL client image to be started with "python /app/main.py". The analytics client image has been restructured to be started that way.
katulu-io · Sep 20, 2022 · 8ed7cbb · 8ed7cbb
1 parent 3a64156
commit 8ed7cbb
Show file tree

Hide file tree

Showing 4 changed files with 134 additions and 133 deletions.
diff --git a/components/kubeflow-pipeline/pipeline_components/analytics-client/Dockerfile b/components/kubeflow-pipeline/pipeline_components/analytics-client/Dockerfile
@@ -2,12 +2,13 @@ FROM python:3.10
 
 LABEL org.opencontainers.image.source https://github.com/katulu-io/fl-suite
 
-COPY requirements.txt /pipelines/
-RUN python3 -m pip install --no-cache-dir -r /pipelines/requirements.txt && \
-    rm /pipelines/requirements.txt
+COPY requirements.txt /app/
+RUN python3 -m pip install --no-cache-dir -r /app/requirements.txt && \
+    rm /app/requirements.txt
 
-COPY flwr_analytics_client /pipelines/component/flwr_analytics_client
+COPY flwr_analytics_client /app/flwr_analytics_client/
+COPY main.py /app/
 
-ENV PYTHONPATH "${PYTHONPATH}:/pipelines/component"
+ENV PYTHONPATH="${PYTHONPATH}:/app"
 
-ENTRYPOINT [ "python3", "-m", "flwr_analytics_client" ]
+ENTRYPOINT [ "python3", "/app/main.py" ]
diff --git a/...-client/flwr_analytics_client/__main__.py → ...eline_components/analytics-client/main.py b/...-client/flwr_analytics_client/__main__.py → ...eline_components/analytics-client/main.py