Comment 7 for bug 1536753

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/15138
Committed: http://github.org/Juniper/contrail-puppet/commit/6fa8beb53cfbbd85c30bfc978e985c32418fb5cb
Submitter: Zuul
Branch: master

commit 6fa8beb53cfbbd85c30bfc978e985c32418fb5cb
Author: nitishkrishna <email address hidden>
Date: Mon Feb 1 10:10:12 2016 -0800

Server-Manager Provisioning changes for rabbit ip:port list change

Changing the contrail config files to use ip:port list
for rabbit_server knob instead of rabbit_server and rabbit_port
with vip and haproxy frontend port.
Closes-Bug: #1515773 - Rabbit HA change
Closes-Bug: #1536753 - Rabbit HA queues never set

Accompanies commit: https://review.opencontrail.org/#/c/15076/

Patch 2:
Added openstack files deleting the setting of rabbit_port per Thilak response

Patch 3:
Changed all config of rabbit_host to rabbit_hosts, verified that rabbit_hosts is available as a config option

Patch 4:
Added user specified amqp_ip_list and amqp_port to params (Used to set contrail_rabbit_servers)

Patch 5:
Added rabbitmq.pp as separate manifest as part of config to allow rabbitmq to be provisioned conditionally based on amqp_ip_list supplied by user.
Removed rabbit mq provisioning from config/config.pp and moved to config/rabbitmq.pp

Patch 6:
Added separate openstack_amqp_ip_list and contrail_amqp_ip_list for user to be able to configure separate rabbitmq clusters for config and opentack nodes.

Patch 7:
Redid after puppet refactoring changes

Patch 8:
Fixed bugs in uuid (rabbitmq clustering), params, init.pp, neutron and haproxy

Patch 9:
Changed defaults for amqp_port,list,etc - new params introduced.
Defaults changed from false to ''.
Change-Id: I1c916ca4bcc455f877d3f77fda9a69d3442d6754