A terrible idea for software versioning in prov

Will discuss further with Govert.

.dockerignore

@@ -5,7 +5,6 @@
 /tests
 .venv
 .flake8
-.git
 .github
 .mypy_cache
 .pytest_cache

Dockerfile

@@ -12,6 +12,14 @@ COPY ./pyproject.toml ./pyproject.toml
 RUN poetry self add poetry-plugin-export==1.8.0
 RUN poetry export --format requirements.txt --output requirements.txt
 
+# Get git commit info
+RUN apt-get update && \
+    apt-get install -y git
+COPY ./.git/ ./
+RUN git rev-parse HEAD >> commit_hash
+RUN rm -rf .git/ && apt-get purge -y git
+
+
 FROM nvidia/cuda:12.3.2-cudnn9-runtime-ubuntu22.04
 
 # Install Python & ffmpeg
@@ -32,13 +40,10 @@ RUN mkdir /src /data /model /model/.cache
 WORKDIR /src
 
 COPY --from=req ./requirements.txt requirements.txt
+COPY --from=req ./commit_hash commit_hash
 RUN python -m pip install --no-cache-dir -r requirements.txt
 
 # copy the rest into the source dir
 COPY ./ /src
 
-# # Get git commit info, then delete the .git folder
-# RUN git rev-parse HEAD >> git_commit
-# RUN rm -rf .git
-
 ENTRYPOINT ["./docker-entrypoint.sh"]

asr.py

@@ -28,13 +28,11 @@
 
 logger = logging.getLogger(__name__)
 
-# TODO: Get commit hash and use it as version in prov
-# (prev impl didn't work)
 version = ""
-# if os.path.exists("git_commit"):
-#     with open("git_commit", "r") as f:
-#         for line in f:
-#             version = line.strip()
+if os.path.exists("commit_hash"):
+    with open("commit_hash", "r") as f:
+        for line in f:
+            version = f"git-{line[:6]}"
 
 
 def run(input_uri: str, output_uri: str, model=None) -> Optional[str]:

docker-compose.yml

@@ -12,12 +12,12 @@ services:
         max-size: 20m
     restart: no
     # Uncomment below when testing the service locally
-    # ports:
-    #   - "5333:5333"
-    #
+    ports:
+      - "5333:5333"
+
     # Comment below to use the CPU instead of the GPU/CUDA
-    deploy:
-      resources:
-        reservations:
-          devices:
-          - capabilities: [gpu]
+    # deploy:
+    #   resources:
+    #     reservations:
+    #       devices:
+    #       - capabilities: [gpu]