Last modified: 2014-01-07 05:49:07 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T61006, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 59006 - Write a redis token driver for keystone
Write a redis token driver for keystone
Status: RESOLVED FIXED
Product: Wikimedia Labs
Classification: Unclassified
Infrastructure (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: Ryan Lane
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-12-27 20:19 UTC by Ryan Lane
Modified: 2014-01-07 05:49 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Ryan Lane 2013-12-27 20:19:24 UTC
Keystone doesn't actually have a redis driver for tokens. It only has some caching code for redis. We need a full driver so that we can replicate tokens across datacenters.
Comment 1 Ryan Lane 2013-12-27 20:21:08 UTC
All functions listed as unimplemented in <https://github.com/openstack/keystone/blob/stable/havana/keystone/token/core.py#L194> must be implemented, except for flush_expired_tokens, which can do nothing, assuming expiration values are properly entered for tokens being added to redis.
Comment 2 Ryan Lane 2013-12-27 20:23:26 UTC
For Folsom we can use this <https://github.com/icgood/keystone-redis> and can likely modify it for Havana.
Comment 3 Ryan Lane 2013-12-31 08:12:36 UTC
So, I've made a fork of <https://github.com/icgood/keystone-redis> at <https://github.com/ryan-lane/keystone-redis>. The fork removes the requirement for python-redis-multiwrite which has a requirement of a much newer version of python-redis and also made the fork itself compatible with the precise version of python-redis. I've also added redis password support to it. I have the changes in labs, but not yet in the fork. It's still a work in progress. When I have it tested and ready I'll push in the changes, make a debian, push it into the repo and deploy it.
Comment 4 Ryan Lane 2014-01-07 05:49:07 UTC
I've added it to the repo and added puppet config for this.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links