[server-manager] Heat Auth Encryption Key set incorrectly

Bug #1478101 reported by Nitish Krishna Kaveri
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.20
Fix Committed
High
Nitish Krishna Kaveri
Trunk
Fix Committed
High
Nitish Krishna Kaveri

Bug Description

This bug is for setups provisioned with Heat using Server-Manager

Heat Auth Encryption Key is explicitly set as empty.
This causes heat stack-create to fail with below error:
ValueError: Key cannot be the null string

Fix is not setting this parameter and letting the default be used.

no longer affects: juniperopenstack/trunk
Abhay Joshi (abhayj)
Changed in juniperopenstack:
milestone: r2.21 → r2.30-fcs
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

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

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

Reviewed: https://review.opencontrail.org/12649
Committed: http://github.org/Juniper/contrail-server-manager/commit/9289abd7ebeac721ed0a0e1aebd26b87949911e0
Submitter: Zuul
Branch: master

commit 9289abd7ebeac721ed0a0e1aebd26b87949911e0
Author: nitishkrishna <email address hidden>
Date: Mon Jul 27 14:44:18 2015 -0700

Partial-Bug: #1478101 - Heat Encryption Key set incorrectly

The Parameter heat_encryption_key has been added to cluster params
If this value is not set, the default value used by heat internally is used instead

Change-Id: I3a94b0c24b89873fa5db94c75d32ec9364c57b5a

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

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

commit 6b9aabf164381f702aefeca4c97bb28ef0fd22d4
Author: nitishkrishna <email address hidden>
Date: Fri Jul 24 15:55:04 2015 -0700

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/12623
Committed: http://github.org/Juniper/contrail-puppet/commit/3c7c93a4207a1dcdb6ff198b3e43dcd355b04d17
Submitter: Zuul
Branch: R2.20

commit 3c7c93a4207a1dcdb6ff198b3e43dcd355b04d17
Author: nitishkrishna <email address hidden>
Date: Fri Jul 24 15:55:04 2015 -0700

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/12648
Committed: http://github.org/Juniper/contrail-server-manager/commit/2cdc1428f9a3ed8715a91a6be85a6638157a840b
Submitter: Zuul
Branch: R2.20

commit 2cdc1428f9a3ed8715a91a6be85a6638157a840b
Author: nitishkrishna <email address hidden>
Date: Mon Jul 27 14:44:18 2015 -0700

Partial-Bug: #1478101 - Heat Encryption Key set incorrectly

The Parameter heat_encryption_key has been added to cluster params
If this value is not set, the default value used by heat internally is used instead

Change-Id: I3a94b0c24b89873fa5db94c75d32ec9364c57b5a

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.22-dev

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

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

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

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/14352
Committed: http://github.org/Juniper/contrail-server-manager/commit/da7988cafa9b614a06334217ab4901381cd2983b
Submitter: Zuul
Branch: R2.22-dev

commit da7988cafa9b614a06334217ab4901381cd2983b
Author: nitishkrishna <email address hidden>
Date: Mon Jul 27 14:44:18 2015 -0700

Partial-Bug: #1478101 - Heat Encryption Key set incorrectly

The Parameter heat_encryption_key has been added to cluster params
If this value is not set, the default value used by heat internally is used instead

Change-Id: I3a94b0c24b89873fa5db94c75d32ec9364c57b5a

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.