← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1527042] Re: All customizable configurations should be above "from local.local_settings import *"

 

Reviewed:  https://review.openstack.org/258833
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=15ac2261eb508dc8cb86046304da4e57ef3c678d
Submitter: Jenkins
Branch:    master

commit 15ac2261eb508dc8cb86046304da4e57ef3c678d
Author: Bo Wang <bo.wang@xxxxxxxxxxxx>
Date:   Thu Dec 17 14:16:35 2015 +0800

    Ensure horizon configurations could be customized
    
    Users could customize horizon configuration items in two ways:
    1. overwrite item values in "local_settings.py".
    2. add *.py into local_settings.d/.
       ref:https://review.openstack.org/#/c/243974/6
    To ensure the two ways work, we should move all customizable
    configurations above the two ways in settings.py.
    Otherwise, configurations in settings.py are the real final values.
    
    Change-Id: I3b6a04d8bd8ec540b1f8efac93465317a0c1008d
    Closes-Bug: #1527042


** Changed in: horizon
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1527042

Title:
  All customizable configurations should be above "from
  local.local_settings import *"

Status in OpenStack Dashboard (Horizon):
  Fix Released

Bug description:
  In my deployed environment, I add some configure items at end of file
  dashboard/settings.py as default values. Try to overwrite some in
  openstack_dashboard/local/local_settings.py but failed.  I find the
  following codes do the overwriting work:

  try:
      from local.local_settings import *  # noqa
  except ImportError:
      logging.warning("No local_settings file found.")

  To ensure overwrite working, all the default configurations in
  settings.py(could be customized in local_settings.py) should be above
  line:"from local.local_settings import *".

  So we should move all customizable configurations above "from
  local.local_settings import *".

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1527042/+subscriptions


References