7 Commits

Author SHA1 Message Date
Pete Birley
c6ff0dd075 Revert "RabbitMQ: use DNS for cluster discovery, not etcd"
This reverts commit 0982a823e139560fd97ecc9390f0cd249cc7982e.

Change-Id: I59fe221e7ecea54fbfde1b3fee72d59a7b1a1cc2
2017-05-05 21:47:37 +00:00
Mark Burnett
0982a823e1 RabbitMQ: use DNS for cluster discovery, not etcd
Implements: blueprint rabbitmq-dns-discovery

Some useful things to note:

1. This uses a StatefulSet instead of a Deployment.  The reason for this
   is that when RabbitMQ uses DNS for peer discovery, the first thing it
   does when trying to join a node is attempt a reverse-dns lookup.
   This reverse lookup works when using a StatefulSet, but not a
   Deployment.
2. The RabbitMQ configuration was updated to use the new sysctl-style
   format.  It seems that the new format is required to configure the
   new autoclustering features.  Additionally, I found that this
   generate much clearer error messages than the straight erlang format.
3. I removed the `is-node-properly-clustered` test in the liveness and
   readiness probes.  This probe isn't directly supported in 3.7.0,
   and it wasn't clear that a clustering check was appropriate for each
   node.

Change-Id: Ieefbb2205bd77fbac04abcd051fb06fce62e8d97
2017-05-05 09:48:18 -05:00
Wil Reichert
7321ded7d5 case replicas to int for comparison, thanks @larryrensing 2017-04-08 06:09:16 +09:00
Wil Reichert
844173049c Don't execute the rabbit cluster readiness check if is there is only 1 replica 2017-04-08 06:04:29 +09:00
Brandon B. Jozsa
1ff7bcac7b add license headers to rabbitmq (#226) 2017-02-20 07:10:35 -08:00
Tomasz Paszkowski
e3f1389f60 Add rabbitmq deployment with HA. (#123)
This works is based on http://github.com/openstack-fuel-ccp-rabbitmq and it mostly adjusts this solution
to a helm based deployment.
2017-02-17 10:49:32 -08:00
Alan Meadows
877e0cd46f aic-helm normalization
* As of helm 2.0.0, it no longer seems necessary to
copy a "globals.tpl" into each child chart, defines in
child charts now seem to be reliably rendered in the
parent.  This was not working correctly in 2.0.0rc2
and so after a lot of testing, the globals define
insertion has been removed from the Makefiles. Also,
includes piped to b64encode are working now, meaning
we no longer have to do Makefile magic to base64
encode successfully.  That has been removed.  Once
.Files.Get works properly in a child chart context
we can remove all Make help.

* The openstack-base chart has been renamed to common
to better reflect that it is common to everything in
aic-helm, even non-openstack projects like ceph.  All
charts now include it as a requirement.

* A first pass at a "cleaner" template directory approach
has been applied to rabbitmq as a test chart.  This allows
files such as _start_rabbit to live in their raw form
and organizes files by config vs script in 'etc' or 'bin'
directories.  If accepted, will apply to all other charts
for consistency.
2016-11-29 16:20:06 -08:00