<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 4 December 2015 at 08:46, Neil Jerram <span dir="ltr"><<a href="mailto:Neil.Jerram@metaswitch.com" target="_blank">Neil.Jerram@metaswitch.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">I'm new to this discussion, but you did ask for any feedback, so ...<br>
<span class=""><br>
On 03/12/15 18:29, Smigiel, Dariusz wrote:<br>
> Hey Neutrinos (thanks armax for this word :),<br>
> Keystone is planning to deprecate V2 API (again :). This time in Mitaka [6], and probably forever. It will stay at least four releases, but we need to decide, how to conquer problem of renaming...<br>
> And more important: consider if it's a problem for Neutron?<br>
><br>
> I'm looking at blueprint [1] about renaming all occurrences of 'tenant' to 'project', and trying to find out all the details.<br>
> First attempt to solve this problem was raised in November 2013 [4][5] but unfortunately, no one finished it. Although Keystone V3 API is already supported in Neutron client [2], there are still some unknowns about Neutron server side. Monty Taylor is trying to address necessary (if any) changes [3].<br>
><br>
> Findings:<br>
> I've focused on two projects: python-neutronclient and neutron.<br>
> grep found 429 occurrences of 'tenant' in Client while Server has 3021 of it. Some of them are just documentation and docstrings, but there are a lot of places, where variables are tangled: defined in DB, used in server, accessed by client. Most of places are just internal usages. The only thing where I've found 'public' information about tenants is 'help' command in neutron client.<br>
><br>
> Suggested plan for conquer:<br>
> 1. First step would be to deal with neutronclient. It's much smaller amount of code to look through, update all places and be successful :)<br>
> 2. Bigger challenge will be to change server side code. I'd suggest to start with renaming db columns. It affects a lot of places, so when finished should significantly lower number of remained "tenants".<br>
> 3. Deal with all other places.<br>
><br>
> Pros:<br>
> - variable names unification in OpenStack code base. Someone needs to start this job.<br>
> - one way to describe the same thing, instead of: tenant/account/project. Helpful, especially for newcomers.<br>
> - alignment with Keystone V3 API<br>
><br>
> Cons:<br>
> - A. Lot. Of. Work.<br>
> - dealing with DB migrations<br>
> - about 2-4 weeks of work for every part of code. Additional, a lot of patchsets to be reviewed.<br>
><br>
> What do you think about this? About proposed way of dealing with all changes?<br>
> Is this change necessary?<br>
<br>
</span>My intuition is that we should just not do this change.  The whole world<br>
says 'tenant' for the 'tenant' concept, particularly in the context of<br>
networking.  Changing to a different term is just silly.<br>
<br></blockquote><div><br></div><div>If you don't know the whole story why are you making these remarks?</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
But I haven't looked into the history.  Perhaps there's some reason we<br>
need to anyway. </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<span class=""><font color="#888888"><br>
    Neil<br>
</font></span><div class=""><div class="h5"><br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div></div>