Skip to content

Commit

Permalink
Update molotov version to v2.6, to support newer versions of python
Browse files Browse the repository at this point in the history
  • Loading branch information
julienw committed Jan 30, 2023
1 parent 5536a7a commit 45efa2b
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 4 deletions.
6 changes: 4 additions & 2 deletions loadtest/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
# We set up a python 3 running in latest Debian stable called bullseye, in the
# "slim" flavor because we don't need the big version.
# We use version 3.9 because 3.10 is failing for now, see https://github.com/loads/molotov/issues/137
FROM python:3.9-slim-bullseye
FROM python:3-slim-bullseye

# Create the user we'll run the build commands with. Its home is configured to
# be the directory /app. It helps avoiding warnings when running tests and
Expand Down Expand Up @@ -51,5 +51,7 @@ RUN pip install -r requirements.txt --no-warn-script-location
RUN ls -la

# Lastly configure the command to be run in this docker.
ENTRYPOINT ["/app/.local/bin/molotov"]
# "-c" is to force the console output of molotov, as the default output doesn't
# work in a docker container.
ENTRYPOINT ["/app/.local/bin/molotov", "-c"]
CMD ["--sizing", "publish.py"]
3 changes: 2 additions & 1 deletion loadtest/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ docker build -t profiler-server-molotov:dev .
```

### Running the image
By default the image will run `molotov --sizing publish.py`. There's no default
By default the image will run `molotov -c --sizing publish.py`. There's no default
endpoint so you need to provide one:
```
docker run -e API_ENDPOINT=http://172.17.0.1:5252 profiler-server-molotov:dev
Expand All @@ -112,6 +112,7 @@ You can specify other parameters, eg:
```
docker run -e API_ENDPOINT=http://172.17.0.1:5252 profiler-server-molotov:dev --sizing shorten.py
```
Note that `-c` will be always added, to force the simple console output of molotov.

And run bash, to debug the image:
```
Expand Down
2 changes: 1 addition & 1 deletion loadtest/requirements.txt
Original file line number Diff line number Diff line change
@@ -1 +1 @@
molotov==2.4
molotov==2.6

0 comments on commit 45efa2b

Please sign in to comment.