[library] We don't need to cache (in memcached) tokens stored in memcached

Bug #1364401 reported by Yuriy Taraday
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Aleksandr Didenko

Bug Description

Currently configuration of token backend in keystone looks like:

[token]
driver=keystone.token.backends.memcache.Token
caching=true

Which means that tokens are stored in memcached and then cached in (presumably the same) memcached. This doesn't make sense, "caching=true" line should be removed.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

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

Changed in fuel:
assignee: nobody → Vladimir Kuklin (vkuklin)
status: New → In Progress
Changed in fuel:
importance: Undecided → High
milestone: none → 5.1
Changed in fuel:
assignee: Vladimir Kuklin (vkuklin) → Aleksandr Didenko (adidenko)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/118382
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=203ef3179007cffe3236032e61ecbaf1cd20605f
Submitter: Jenkins
Branch: master

commit 203ef3179007cffe3236032e61ecbaf1cd20605f
Author: Vladimir Kuklin <email address hidden>
Date: Tue Sep 2 19:30:36 2014 +0400

    Change keystone memcache backend

    Use keystone memcache pool backend
    in order to fix scalability and
    failover problems

    Also fixing file_line "match" + "after" insertion logic.

    Change-Id: Idfe4b54caa0d96a93e93bfff12d8b6216f83e2f1
    Closes-bug: #1364401
    Closes-bug: #1340657

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/5.0)

Fix proposed to branch: stable/5.0
Review: https://review.openstack.org/122230

Changed in fuel:
status: Fix Committed → Fix Released
Changed in fuel:
status: Fix Released → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-library (stable/5.0)

Change abandoned by Sergii Golovatiuk (<email address hidden>) on branch: stable/5.0
Review: https://review.openstack.org/122230

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/5.0)

Fix proposed to branch: stable/5.0
Review: https://review.openstack.org/143084

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-library (stable/5.0)

Change abandoned by Sergii Golovatiuk (<email address hidden>) on branch: stable/5.0
Review: https://review.openstack.org/143084

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.