Unicast option for hacluster charm
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
hacluster (Juju Charms Collection) |
Fix Released
|
Medium
|
Liam Young |
Bug Description
Very often multicast is not allowed in production environments. Purpose of multicast in corosync is to enable easy discovery of the nodes that want to join the cluster. That seems a bit redundant within juju environment. All the nodes that will form the cluster are already known and given by juju. So, instead of trashing multicast on the network, ideally charm would setup corosync in unicast mode, and node information would be pulled from juju.
What hacluster charm needs to do is replace interface {} section in corosync.conf. When unicast is enabled, that part shoud be like this:
interface {
}
}
}
}
}
Replace $variables with proper values. As a side effect this might allow HA within dislocated networks, like AWS.
Related branches
- Liam Young (community): Disapprove
- James Page: Needs Fixing
-
Diff: 1127 lines (+592/-115)15 files modified.bzrignore (+2/-0)
Makefile (+7/-2)
config.yaml (+8/-0)
hooks/charmhelpers/contrib/hahelpers/cluster.py (+12/-2)
hooks/charmhelpers/contrib/storage/linux/ceph.py (+1/-1)
hooks/charmhelpers/contrib/storage/linux/lvm.py (+1/-1)
hooks/charmhelpers/contrib/storage/linux/utils.py (+29/-5)
hooks/charmhelpers/core/fstab.py (+116/-0)
hooks/charmhelpers/core/hookenv.py (+103/-5)
hooks/charmhelpers/core/host.py (+42/-8)
hooks/charmhelpers/fetch/__init__.py (+130/-81)
hooks/charmhelpers/fetch/bzrurl.py (+2/-1)
hooks/hooks.py (+66/-8)
revision (+0/-1)
templates/corosync.conf.udpu (+73/-0)
- Edward Hope-Morley: Approve
- James Page: Approve
-
Diff: 708 lines (+557/-10)7 files modified.bzrignore (+1/-0)
Makefile (+11/-2)
charm-helpers.yaml (+1/-0)
config.yaml (+5/-0)
hooks/charmhelpers/contrib/openstack/utils.py (+486/-0)
hooks/hooks.py (+39/-8)
templates/corosync.conf (+14/-0)
Changed in charms: | |
importance: | Undecided → Medium |
assignee: | nobody → Liam Young (gnuoy) |
affects: | charms → hacluster (Juju Charms Collection) |
tags: | added: openstack |
Changed in hacluster (Juju Charms Collection): | |
status: | New → In Progress |
tags: | added: cts |
Changed in hacluster (Juju Charms Collection): | |
status: | In Progress → Fix Committed |
Changed in hacluster (Juju Charms Collection): | |
milestone: | none → 15.01 |
Changed in hacluster (Juju Charms Collection): | |
status: | Fix Committed → Fix Released |
Oh, this format is still valid in Trusty, but newer versions of corosync might require newer syntax:
http:// clusterlabs. org/doc/ en-US/Pacemaker /1.1/html/ Clusters_ from_Scratch/ _sample_ corosync_ configuration. html