Unify sqlalchemy Dockerfiles (#1585)

* Unify sqlalchemy Dockerfiles Signed-off-by: eduardo apolinario <eapolinario@users.noreply.github.com> * Simplifyte sqlalchemy Dockerfile Signed-off-by: eduardo apolinario <eapolinario@users.noreply.github.com> * Set pythonpath to /root Signed-off-by: eduardo apolinario <eapolinario@users.noreply.github.com> --------- Signed-off-by: eduardo apolinario <eapolinario@users.noreply.github.com> Co-authored-by: eduardo apolinario <eapolinario@users.noreply.github.com>
flyteorg · Apr 7, 2023 · e3cee83 · e3cee83
1 parent 0063b29
commit e3cee83
Show file tree

Hide file tree

Showing 6 changed files with 21 additions and 101 deletions.
diff --git a/.github/workflows/pythonpublish.yml b/.github/workflows/pythonpublish.yml
@@ -137,6 +137,7 @@ jobs:
  tags: ${{ steps.sqlalchemy-names.outputs.tags }}
  build-args: |
  VERSION=${{ needs.deploy.outputs.version }}
- file: ./plugins/flytekit-sqlalchemy/Dockerfile.py${{ matrix.python-version }}
+ PYTHON_VERSION=${{ matrix.python-version }}
+ file: ./plugins/flytekit-sqlalchemy/Dockerfile
  cache-from: type=gha
  cache-to: type=gha,mode=max
diff --git a/plugins/flytekit-sqlalchemy/Dockerfile b/plugins/flytekit-sqlalchemy/Dockerfile
@@ -0,0 +1,19 @@
+ARG PYTHON_VERSION
+FROM python:${PYTHON_VERSION}-slim-buster
+
+WORKDIR /root
+ENV LANG C.UTF-8
+ENV LC_ALL C.UTF-8
+ENV PYTHONPATH /root
+
+ARG VERSION
+
+RUN pip install sqlalchemy \
+ psycopg2-binary \
+ pymysql \
+ flytekitplugins-sqlalchemy==$VERSION \
+ flytekit==$VERSION
+
+RUN useradd -u 1000 flytekit
+RUN chown flytekit: /root
+USER flytekit
diff --git a/plugins/flytekit-sqlalchemy/Dockerfile.py3.10 b/plugins/flytekit-sqlalchemy/Dockerfile.py3.10
diff --git a/plugins/flytekit-sqlalchemy/Dockerfile.py3.7 b/plugins/flytekit-sqlalchemy/Dockerfile.py3.7
diff --git a/plugins/flytekit-sqlalchemy/Dockerfile.py3.8 b/plugins/flytekit-sqlalchemy/Dockerfile.py3.8
diff --git a/plugins/flytekit-sqlalchemy/Dockerfile.py3.9 b/plugins/flytekit-sqlalchemy/Dockerfile.py3.9