add a hubot docker image #42
Conversation
|
This is now r4r. Tested by adding |
hubot/data/Procfile
Outdated
| @@ -0,0 +1 @@ | |||
| web: bin/hubot -a irc | |||
hubot/data/external-scripts.json
Outdated
| @@ -0,0 +1,2 @@ | |||
| [ | |||
There was a problem hiding this comment.
This is generated by yo script that makes hubot configs, but I don't know if it's actually required (similarly, not sure about procfile). I'll experiment and see quickly.
|
Actually, before we merge this, please create a |
|
Information about what the credentials need to be? Do you want that to live in the hubot dir? |
|
No, in teh root of the repo please. |
hubot/Dockerfile
Outdated
|
|
||
| COPY ["data/package.json", "/home/hubot/bot/"] | ||
| COPY ["data/scripts/", "/home/hubot/bot/scripts"] | ||
| COPY ["data/bin/", "/home/hubot/bot/bin"] |
There was a problem hiding this comment.
These lines don't need the array syntax, do they? COPY data/bin/ /home/hubot/bot/bin works right?
hubot/data/bin/hubot
Outdated
|
|
||
| set -e | ||
|
|
||
| npm install |
There was a problem hiding this comment.
Why does running hubot trigger an npm install?
There was a problem hiding this comment.
No idea. the yo script that creates a hubot does this. We could potentially remove it.
| RUN apt-get install -qq -y runit | ||
|
|
||
| ENV HUBOT_IRC_SERVER irc.freenode.net | ||
| ENV HUBOT_IRC_ROOMS #cryptography-dev |
There was a problem hiding this comment.
"The ENV instruction has two forms. The first form, ENV key value, will set a single variable to a value. The entire string after the first space will be treated as the value - including characters such as spaces and quotes."
This requires a token passed as an environment variable to work properly. We'll need to figure out how we want to provide that.