Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding support for Docker build/run #42

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 15 additions & 0 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
FROM ubuntu:14.04
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't this a bit old?


WORKDIR /app

RUN apt-get update &&\
apt-get install -y git python2.7-dev python-virtualenv fuse

ADD . /app

RUN virtualenv .env && \
. .env/bin/activate

RUN pip install .

ENTRYPOINT [ "/app/malaria" ]
10 changes: 10 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@ Contributions welcome!
Install
=======

If using Docker, just do ```docker build .```

Requires python2, paho-mqtt python library 1.1 or greater, and fusepy.

```
Expand Down Expand Up @@ -70,6 +72,9 @@ optional arguments:
(multiprocessing) (default: 1)
```

If using Docker version,
docker run flaviostutz/mqtt-malaria [options]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not going to link to third party builds I'm sorry.


Examples
--------

Expand All @@ -79,6 +84,11 @@ sending as fast as the code allows.
malaria publish -P 8 -n 10000 -H mqtt.example.org -s 100
```

For doing the same as above, now using the containerized version:
```
docker run flaviostutz/mqtt-malaria publish -P 8 -n 10000 -H mqtt.example.org -s 100
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here, either convert this to syntax for a locally built docker, or drop it entirely, require container people know what they're doing.

```

To fire up 500 processes, each sending 5 messages per second, each sending
1000 messages, with time in flight tracking information
```
Expand Down
Empty file added wc
Empty file.