+1<div><br></div><div>How do we need to handle the migration reversal? <a href="https://github.com/openstack/keystone/blob/master/keystone/common/sql/migrate_repo/versions/002_token_id_hash.py">https://github.com/openstack/keystone/blob/master/keystone/common/sql/migrate_repo/versions/002_token_id_hash.py</a><br clear="all">
</div><div><br></div>-Dolph<br>
<br><br><div class="gmail_quote">On Tue, Sep 4, 2012 at 8:55 AM, Adam Young <span dir="ltr"><<a href="mailto:ayoung@redhat.com" target="_blank">ayoung@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
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:<br>
<br>
1.  Remove the id_hash column<br>
2.  Change the id column to  sql.Column(sql.String(64), primary_key=True)<br>
3.  Stop persisting the PKI token body in the table.<br>
<br>
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.<br>
<br>
<br>
<br>
______________________________<u></u>_________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">OpenStack-dev@lists.openstack.<u></u>org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/<u></u>cgi-bin/mailman/listinfo/<u></u>openstack-dev</a><br>
</blockquote></div><br>