memcache_servers configuration option

[Kevin L. Mitchell]

The memcache_servers configuration option is already declared as a ListOpt, but when used in AuthProtocol.__init__(), a .split() is attempted, which of course fails.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/17413
Committed: http://github.com/openstack/python-keystoneclient/commit/6b72fb626af57a85d1b3411d0c7e8b3d8fdc33b2
Submitter: Jenkins
Branch: master

commit 6b72fb626af57a85d1b3411d0c7e8b3d8fdc33b2
Author: Kevin L. Mitchell <email address hidden>
Date: Mon Dec 3 14:24:38 2012 -0600

    Don't try to split a list of memcache servers

    The memcache_servers configuration option is declared as a ListOpt,
    but when used in AuthProtocol.__init__(), we treat it as a string
    to be split on ',', which is wrong. Remove the .split().

    No test is added because the memcache package may not be installed,
    and I cannot see an easy method to only test this clause if it is
    available.

    Fixes bug 1086125.

    Change-Id: Ifb0a18017d2407b3ccb2188b1a704c26997ba594

[Alan Pevec (apevec)]

BTW this bug was a side-effect of https://github.com/openstack/keystone/commit/174964498ba098f206d27119ce58d9fa6f43d302

One issue with the merged fix is that it won't parse multiple servers when using filter parameters from paste.ini which is supported for backward compatibility. I'll propose a followup fix.