Skip to content

Commit

Permalink
Restart zookeeper after config and remove neutron conf modify task fr…
Browse files Browse the repository at this point in the history
…om chain

After running the zookeeper setup script service was not started.
Adding zookeeper restart

Removing neutron.conf updation related task from config manifest chain to avoid dependency cycle
Closes-Bug: #1391107
Closes-Bug: #1397813

Change-Id: I452324aaa4a022b7daaa32d05fd5acb48abd875f
  • Loading branch information
Chhandak committed Dec 1, 2014
1 parent 038dc79 commit 76f6d83
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions contrail/manifests/contrail_config.pp
Expand Up @@ -230,9 +230,9 @@
# Increase header size accepted as keystone v3 generates large ones.
if ! defined(Exec["neutron-conf-max-header"]) {
exec { "neutron-conf-max-header":
command => " openstack-config --set /etc/neutron/neutron.conf DEFAULT max_header_line 65536 && service neutron-server restart && echo neutron-conf-max-header >> /etc/contrail/contrail-config-exec.out",
command => " openstack-config --set /etc/neutron/neutron.conf DEFAULT max_header_line 65536 && service neutron-server restart && echo neutron-conf-max-header >> /etc/contrail/contrail_config_exec.out",
onlyif => "test -f /etc/neutron/neutron.conf",
unless => "grep -qx neutron-conf-max-header /etc/contrail/contrail-config-exec.out",
unless => "grep -qx neutron-conf-max-header /etc/contrail/contrail_config_exec.out",
provider => shell,
logoutput => "true"
}
Expand Down Expand Up @@ -540,7 +540,7 @@
}


Package["contrail-openstack-config"]->Setup-haproxy["setup_haproxy"]->Exec["stop-neutron"]->Service["haproxy"]->Exec["exec-cfg-rabbitmq"]->Exec["setup-rabbitmq-cluster"]->Exec["check-rabbitmq-cluster"]->Config-scripts["config-server-setup"]->Config-scripts["quantum-server-setup"]->Exec["setup-verify-quantum-in-keystone"]->Exec["config-neutron-server"]->Exec["neutron-conf-exec"]->Exec["neutron-conf-admin-tenant-exec"]->Exec["provision-metadata-services"]->Exec["provision-encap-type"]->Exec["exec-provision-control"]->Exec["provision-external-bgp"]
Package["contrail-openstack-config"]->Setup-haproxy["setup_haproxy"]->Exec["stop-neutron"]->Service["haproxy"]->Exec["exec-cfg-rabbitmq"]->Exec["setup-rabbitmq-cluster"]->Exec["check-rabbitmq-cluster"]->Config-scripts["config-server-setup"]->Config-scripts["quantum-server-setup"]->Exec["setup-verify-quantum-in-keystone"]->Exec["config-neutron-server"]->Exec["provision-metadata-services"]->Exec["provision-encap-type"]->Exec["exec-provision-control"]->Exec["provision-external-bgp"]

# Below is temporary to work-around in Ubuntu as Service resource fails
# as upstart is not correctly linked to /etc/init.d/service-name
Expand Down
2 changes: 1 addition & 1 deletion contrail/manifests/contrail_database.pp
Expand Up @@ -122,7 +122,7 @@
source => "puppet:///modules/$module_name/config-zk-files-setup.sh"
}
$contrail_zk_ip_list_for_shell = inline_template('<%= contrail_zookeeper_ip_list.map{ |ip| "#{ip}" }.join(" ") %>')
$contrail_zk_exec_cmd = "/bin/bash /etc/contrail/contrail_setup_utils/config-zk-files-setup.sh $operatingsystem $contrail_database_index $contrail_zk_ip_list_for_shell && echo setup-config-zk-files-setup >> /etc/contrail/contrail_database_exec.out"
$contrail_zk_exec_cmd = "/bin/bash /etc/contrail/contrail_setup_utils/config-zk-files-setup.sh $operatingsystem $contrail_database_index $contrail_zk_ip_list_for_shell && service zookeeper restart && echo setup-config-zk-files-setup >> /etc/contrail/contrail_database_exec.out"
notify { "contrail contrail_zk_exec_cmd is $contrail_zk_exec_cmd":; }
exec { "setup-config-zk-files-setup" :
command => $contrail_zk_exec_cmd,
Expand Down

0 comments on commit 76f6d83

Please sign in to comment.