[openstack-dev] [Keystone] Proposal to remove token body from Database tables

Adam Young ayoung at redhat.com
Tue Sep 4 17:59:56 UTC 2012


On 09/04/2012 12:54 PM, Dolph Mathews wrote:
> +1
>
> How do we need to handle the migration reversal? 
> https://github.com/openstack/keystone/blob/master/keystone/common/sql/migrate_repo/versions/002_token_id_hash.py

Just add another one on.  I'm writing it as part of the patch.

>
> -Dolph
>
>
> On Tue, Sep 4, 2012 at 8:55 AM, Adam Young <ayoung at redhat.com 
> <mailto:ayoung at redhat.com>> wrote:
>
>     The PKI token change introduced an unnecessary change to the
>     database table for tokens.  I'd like to revert it to the layout
>     that exists in Essex.  To that end,  I'd like to:
>
>     1.  Remove the id_hash column
>     2.  Change the id column to  sql.Column(sql.String(64),
>     primary_key=True)
>     3.  Stop persisting the PKI token body in the table.
>
>     The changes to do this are minimal.  Is this an acceptable change
>     to push through between now and the Folsom release?  Including the
>     tokens and the extra columns in the database long term will likely
>     cause trouble.
>
>
>
>     _______________________________________________
>     OpenStack-dev mailing list
>     OpenStack-dev at lists.openstack.org
>     <mailto:OpenStack-dev at lists.openstack.org>
>     http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
>
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20120904/096fe9c5/attachment.html>


More information about the OpenStack-dev mailing list