Key-value stores in OpenStack Installation Guide for Red Hat Enterprise Linux, CentOS, and Fedora - icehouse

[Nick Mishin]

Hello!
I think need add a comma after the line: "'BACKEND': 'django.core.cache.backends.memcached.MemcachedCache'"
should be:
Enabled by:

SESSION_ENGINE = 'django.contrib.sessions.backends.cache'
CACHES = {
  'default': {
    'BACKEND': 'django.core.cache.backends.memcached.MemcachedCache',
    'LOCATION': 'my_memcached_host:11211'
  }
}
-----------------------------------
Built: 2014-08-21T22:11:41 00:00
git SHA: d9e3ab1f86d8157cd4a7b0ad5be680aaa300ceb3
URL: http://docs.openstack.org/icehouse/install-guide/install/yum/content/dashboard-session-key-value-store.html
source File: file:/home/jenkins/workspace/openstack-manuals-tox-doc-publishdocs/doc/common/section_dashboard_sessions.xml
xml:id: dashboard-session-key-value-store

[naggappan (naggappan)]

Changes submitted for review,

https://review.openstack.org/#/c/121450/

[naggappan (naggappan)]

Hi ,

Currently when i checked in the code and build i get the following in the document,

SESSION_ENGINE = 'django.contrib.sessions.backends.cache'
CACHES = {
  'default': {
    'BACKEND': 'django.core.cache.backends.memcached.MemcachedCache',
    'LOCATION': 'my_memcached_host:11211'
  }
}

I added a comma in the end of LOCATION. In dictionary format it doesn't matter we have a comma or not for the last variable. So It is proper now. But i don't see this update in ,
http://docs.openstack.org/icehouse/install-guide/install/yum/content/dashboard-session-key-value-store.html

But in the latest icehouse branch its modified. So please let me know do i still need to do any changes on this?

[Tom Fifield (fifieldt)]

Hi naggappan,

This was fixed in juno work, I believe. Thanks for the report!

[Tom Fifield (fifieldt)]

oh - I see you submitted the review to fix it in stable/icehouse as well.

[Tom Fifield (fifieldt)]

oh - it is fixed in icehouse too - looks like someone beat you to it by a whisker. Sorry, naggappan.