chore: switch to a minimal requirements file

The old requirements file was bloated and not used by the docker image. We only installed required dependencies as a transitive dependencies. This setup should allow for code completion and better dependency management for the locust containers.
codecentric · Jan 5, 2024 · ede380f · ede380f
1 parent 9c36212
commit ede380f
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 42 deletions.
diff --git a/.github/workflows/lpt-locust.yaml b/.github/workflows/lpt-locust.yaml
@@ -31,9 +31,11 @@ jobs:
         with:
           python-version: '3.11'
           cache: 'pip'
-          cache-dependency-path: 'test/lpt-locust/requirements.txt'
+          cache-dependency-path: 'test/lpt-locust/src/requirements.txt'
       - name: 👨🏻‍💻 Install dependencies
-        run: pip install -r test/lpt-locust/requirements.txt
+        run: pip install -r test/lpt-locust/src/requirements.txt
+      - name: Install license reporting tool
+        run: pip install pip-licenses
       - name: 📋 Generate license report
         working-directory: ./test/lpt-locust
         run: |

diff --git a/test/lpt-locust/Dockerfile b/test/lpt-locust/Dockerfile
@@ -1,4 +1,4 @@
 FROM locustio/locust:2.16.1
-RUN pip3 install kubernetes
 COPY src /locust-config
+RUN pip3 install -r /locust-config/requirements.txt
 ENV LOCUST_LOCUSTFILE /locust-config/locustfile.py
diff --git a/test/lpt-locust/requirements.txt b/test/lpt-locust/requirements.txt
diff --git a/test/lpt-locust/src/requirements.txt b/test/lpt-locust/src/requirements.txt
@@ -0,0 +1,3 @@
+locust
+oauthlib
+requests-oauthlib