Code review comment for lp://staging/~thedac/charms/trusty/rabbitmq-server/le-ignore-min-cluster

Revision history for this message
Chris Glass (tribaal) wrote :

I realize this is already merged, but I'll do my comments anyway.

It would be good to add an amulet test to the branch asserting that a cluster forms with only i.e. 2 units (one less than min-cluster-size) if leader election is available.

Two inline comments.

« Back to merge proposal