search

rocdev-foundation / medera

Is a chat bot the new "Hello, World"?
GNU General Public License v3.0
1 stars 2 forks source link

Initial minion implementation #20

Closed dantswain closed 7 years ago

dantswain commented 7 years ago

I want to add some documentation before merging. Opening PR partly for self-review.

dantswain commented 7 years ago

Ready for review. I ended up giving the README some love as well and adding the test coverage badge (why not?). This would close #9 I think as that doesn't really put any requirements on what the agent does other than connect to the master.

The minion node should automatically connect to the configured master node when it is available, both sides detect disconnects and should automatically reconnect when possible.

I opted for a fairly simple distribution scheme here - nothing fancy like Phoenix.Presence or PubSub. That may be something good to do eventually - especially if we want the "master" to be clusterable - but I'd rather let the requirements sort themselves out before trying to implement anything.