<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Mon, Jun 10, 2013 at 9:23 AM, Sean Dague <span dir="ltr"><<a href="mailto:sean@dague.net" target="_blank">sean@dague.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 06/10/2013 09:48 AM, Alvaro Lopez wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi all.<br>
<br>
I've submitted a change [1] for Keystone whose tests are failing, since<br>
some tempest changes are made asumptions bases on the old behaviour.<br>
<br>
The proposed change increases the length of the user's name in the DB<br>
up to 128 characters, instead of 64. Tempest is trying to create a user<br>
with a username of lenght 65 and expects it to fail.<br>
<br>
How should I proceed? Should I submit a change to tempest to fix this,<br>
even before knowing if my original change seems reasonable to be merged?<br>
Should I wait for my change to get some feedback before sending the<br>
modified test for tempest?<br>
<br>
Regards,<br>
Alvaro.<br>
<br>
[1] <a href="https://review.openstack.org/#/c/22694/" target="_blank">https://review.openstack.org/#<u></u>/c/22694/</a><br>
</blockquote>
<br></div>
Typically the route is propose the tempest change, and get core contributors from the core project (this time keystone) to +1 it (saying that they agree with the change).<br>
<br>
The core team for keystone would need to believe that this doesn't represent an API change without a version bump. And that python-keystoneclient would do the right things with a new client on an older system. But I'll leave that up to them.</blockquote>
<div><br></div><div style>This definitely doesn't represent an API change, and would be backwards compatible with existing clients (this validation is only performed server side, anyway).</div><div style><br></div><div style>
Following Chmouel's comment on the review, I'm wondering if it would be possible to preserve the existing default but make this value configurable moving forward (which would mean no changes in tempest). I'll save that discussion for the code review, but wanted to mention the possibility of not affecting tempest here.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888"><br>
<br>
        -Sean<br>
<br>
-- <br>
Sean Dague<br>
<a href="http://dague.net" target="_blank">http://dague.net</a></font></span><div class="HOEnZb"><div class="h5"><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>
</div></div></blockquote></div><br></div></div>