config_helper.py:set_general_configs should be removed

Bug #1235689 reported by Dmitry Mescheryakov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Sahara
Fix Released
Critical
Nikolay Makhotkin

Bug Description

Exact location:

savanna/plugins/vanilla/config_helper.py
  method set_general_configs()

It is a very bad idea to change constant GENERAL_CONFS. Besides right now it is not thread-safe as it sets constant based on request-scoped variables hive_hostname and passwd_mysql

We should remove the method completely and do not change constants for each request

Tags: 0.3
tags: added: 0.3
Changed in savanna:
milestone: none → 0.3
importance: Undecided → Critical
status: New → Triaged
Changed in savanna:
assignee: nobody → Nikolay Makhotkin (nmakhotkin)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to savanna (master)

Fix proposed to branch: master
Review: https://review.openstack.org/50061

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to savanna (master)

Reviewed: https://review.openstack.org/50061
Committed: http://github.com/openstack/savanna/commit/052244f9323f7465ff7b7c2678e53616fe09e580
Submitter: Jenkins
Branch: master

commit 052244f9323f7465ff7b7c2678e53616fe09e580
Author: Nikolay Mahotkin <email address hidden>
Date: Mon Oct 7 16:46:20 2013 +0400

    Delete constant 'GENERAL_CONFS' from config_helper.py

    Fixes Bug #1235689

    Change-Id: I51c548cc373498b799e1d7cf11736fe6dcef3394

Changed in savanna:
status: In Progress → Fix Committed
Changed in savanna:
milestone: 0.3 → 0.3-rc2
Changed in savanna:
status: Fix Committed → Fix Released
Changed in savanna:
milestone: 0.3-rc2 → 0.3
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.