Riak Charm

[Dean Proctor]

This is a charm for the Riak NoSQL database. The code for the charm is located here: https://code.launchpad.net/~deanproctor/charms/precise/riak/trunk

[Adam Gandelman (gandelman-a)]

Hi Dean-

Thanks for submitting.

This looks like a nice, simple and clean charm.

Couple of things,
 - trivial typo in config.yaml (last line, 'instnace')
 - Would it make sense to also include a 'ring-relation-departed' hook to remove departed nodes from the ring? As I understand it this isn't necessary, but disappearing nodes causes some overhead if they are not removed clean?
 - I'd like to see at least a few lines in the README that describes basic usage, requirements, etc.

Unfortunately, I can't actually get the service started in a 12.04 m1.small instance on EC2. Deploying the charm with default config ends in a 'start-error'.

The charm log shows the following: http://paste.ubuntu.com/1083613/

Manually trying to start riak results in the same error. Setting WAIT_FOR_ERLANG in my environment has no effect (seems to be ignoring the value if I increase to 60 sec)

The former points are minor nits that shouldn't block this charm but this needs to reliably produce a functioning service before it can be promulgated. If I've overlooked a config option or if there are any other special requirements to get this working, please note them in the README.

[Kapil Thangavelu (hazmat)]

  node_name:

shouldn't be in service config, a service can have many nodes/machines.

[Clint Byrum (clint-fewbar)]

Hi Dean. When you've addressed Adam and Kapil's concerns, can you set the status to "Fix Committed". Then we will review it again. Thanks!

[Jorge Castro (jorge)]

Of note: http://basho.com/riak-cs-is-now-open-source/