Riak Charm

Reported by Dean Proctor on 2012-07-09
22
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Juju Charms Collection
Undecided
Dean Proctor

Bug Description

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) wrote :

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) wrote :

  node_name:

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

Clint Byrum (clint-fewbar) wrote :

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!

Changed in charms:
status: New → Incomplete
assignee: nobody → Dean Proctor (deanproctor)
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers