Smgr Provisioning: Neutron notification_driver set to empty by default

[Nitish Krishna Kaveri]

The notification_driver config in neutron.conf is being set to empty in non-ceilometer case.
The default value that comes with install needs to be preserved.
Causes failure in service chain test case (test_ecmp_svc_in_network_with_3_instance) in Server Manager Sanity

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/12080
Submitter: Nitish Krishna Kaveri (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/12081
Submitter: Nitish Krishna Kaveri (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/12080
Submitter: Nitish Krishna Kaveri (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/12081
Submitter: Nitish Krishna Kaveri (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/12080
Submitter: Nitish Krishna Kaveri (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/12081
Submitter: Nitish Krishna Kaveri (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/12081
Committed: http://github.org/Juniper/contrail-puppet/commit/9ba18195d5badf5cf2cb18551a80e22e3f7c52d3
Submitter: Zuul
Branch: master

commit 9ba18195d5badf5cf2cb18551a80e22e3f7c52d3
Author: nitishkrishna <email address hidden>
Date: Mon Jun 29 10:54:58 2015 -0700

Closes-Bug: #1469264 - Neutron notification_driver param set incorrectly.

Previously it was being set to empty if ceilometer was not enabled.
Patch 2:
- Reverting change to set this config for Ceilometer
- Whatever is the default value when neutron is installed, that will be taken
Patch 3:
- Missed removing one line

Change-Id: I557d7e3bcb8a4d9002fe65ccdc015212534055dd

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/12080
Committed: http://github.org/Juniper/contrail-puppet/commit/228aba84d42e54aad97a0f58ba17aa5fdf8df340
Submitter: Zuul
Branch: R2.20

commit 228aba84d42e54aad97a0f58ba17aa5fdf8df340
Author: nitishkrishna <email address hidden>
Date: Mon Jun 29 10:54:58 2015 -0700

Closes-Bug: #1469264 - Neutron notification_driver param set incorrectly.

Previously it was being set to empty if ceilometer was not enabled.
Patch 2:
- Reverting change to set this config for Ceilometer
- Whatever is the default value when neutron is installed, that will be taken
Patch 3:
- Missed removing one line

Change-Id: I557d7e3bcb8a4d9002fe65ccdc015212534055dd

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/14353
Submitter: Nitish Krishna Kaveri (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/14353
Committed: http://github.org/Juniper/contrail-puppet/commit/97df333ca77c257aab080e2a3ebc774e5ea3dced
Submitter: Zuul
Branch: R2.22-dev

commit 97df333ca77c257aab080e2a3ebc774e5ea3dced
Author: nitishkrishna <email address hidden>
Date: Mon Jun 29 10:54:58 2015 -0700

Closes-Bug: #1469264 - Neutron notification_driver param set incorrectly.

Previously it was being set to empty if ceilometer was not enabled.
Patch 2:
- Reverting change to set this config for Ceilometer
- Whatever is the default value when neutron is installed, that will be taken
Patch 3:
- Missed removing one line

Change-Id: I557d7e3bcb8a4d9002fe65ccdc015212534055dd

Closes-Bug: #1478101 - Heat Auth Encryption Key set incorrectly

Previously the auth_encryption_key param of heat engine was being set as empty.
This is caused heat stack-create to fail.
Fab provisioned Heat uses Default auth key (This param is not set).

Fix:
Since puppet manifest of heat engine requires auth key as mandatory param, we are setting it to the default that Heat internally uses
If we wish to change the auth key, we can provide it in the openstack params as ::openstack::config::heat_encryption_key
There is an accompanying check-in to server-manager-repo to set this param there

Change-Id: Ie2c828fff0a206eec5e963a6902e9413d56026ac

Related-Bug: #1471596 - Smgr changes for Keepliaved independence for Cluster monitor and monitoring jobs

This check-in:
Is the Smgr side changes made for the checkins at:
https://review.opencontrail.org/#/c/12280/
and https://review.opencontrail.org/#/c/12281/

Extra parameters are being passed to the script setup-vnc-galera
These parameters are set or configured in ha_config.pp

Should go in after Thilak's commit:
https://review.opencontrail.org/#/c/12734/4

Small bug fix - Verified setup-vnc-galera ran on HA Setup

Change-Id: I9263248e2f632fc04dbebbb99afbd0abeff45390

Related-Bug: #1471596 - Readding Fab to Server Manager Provisioning

This check-in:
Brings back Fab package to Server Manager provisioning
There are many contrail-provisioning scripts that use fab commands and sm puppet scripts have to be extricated from these before fab can be removed

Change-Id: Ib289948e0e0317cff1a9de064e86b22b6acca265