<div dir="ltr"><div>Looking at all comments it seems that existing change is reasonable. I will update it with link to this thread.</div><div><br></div><div>Thanks!</div><div><br></div><div>Regards,</div><div>Ann Kamyshnikova</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Dec 13, 2014 at 1:15 AM, Rochelle Grober <span dir="ltr"><<a href="mailto:rochelle.grober@huawei.com" target="_blank">rochelle.grober@huawei.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div lang="EN-US" link="blue" vlink="purple">
<div><div><div class="h5">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
<div style="border:none;border-top:solid #b5c4df 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">Morgan Fainberg [mailto:<a href="mailto:morgan.fainberg@gmail.com" target="_blank">morgan.fainberg@gmail.com</a>]
<b>on</b> Friday, December 12, 2014 2:01 PM wrote:</span><br>
On Friday, December 12, 2014, Sean Dague <<a href="mailto:sean@dague.net" target="_blank">sean@dague.net</a>> wrote:<span style="font-size:10.0pt;font-family:"Tahoma","sans-serif""><u></u><u></u></span></p>
</div>
<p class="MsoNormal" style="margin-bottom:12.0pt">On 12/12/2014 01:05 PM, Maru Newby wrote:<br>
><br>
> On Dec 11, 2014, at 2:27 PM, Sean Dague <<a>sean@dague.net</a>> wrote:<br>
><br>
>> On 12/11/2014 04:16 PM, Jay Pipes wrote:<br>
>>> On 12/11/2014 04:07 PM, Vishvananda Ishaya wrote:<br>
>>>> On Dec 11, 2014, at 1:04 PM, Jay Pipes <<a>jaypipes@gmail.com</a>> wrote:<br>
>>>>> On 12/11/2014 04:01 PM, Vishvananda Ishaya wrote:<br>
>>>>>><br>
>>>>>> On Dec 11, 2014, at 8:00 AM, Henry Gessau <<a>gessau@cisco.com</a>> wrote:<br>
>>>>>><br>
>>>>>>> On Thu, Dec 11, 2014, Mark McClain <mark@mcclain.xyz> wrote:<br>
>>>>>>>><br>
>>>>>>>>> On Dec 11, 2014, at 8:43 AM, Jay Pipes <<a>jaypipes@gmail.com</a><br>
>>>>>>>>> <mailto:<a>jaypipes@gmail.com</a>>> wrote:<br>
>>>>>>>>><br>
>>>>>>>>> I'm generally in favor of making name attributes opaque, utf-8<br>
>>>>>>>>> strings that<br>
>>>>>>>>> are entirely user-defined and have no constraints on them. I<br>
>>>>>>>>> consider the<br>
>>>>>>>>> name to be just a tag that the user places on some resource. It<br>
>>>>>>>>> is the<br>
>>>>>>>>> resource's ID that is unique.<br>
>>>>>>>>><br>
>>>>>>>>> I do realize that Nova takes a different approach to *some*<br>
>>>>>>>>> resources,<br>
>>>>>>>>> including the security group name.<br>
>>>>>>>>><br>
>>>>>>>>> End of the day, it's probably just a personal preference whether<br>
>>>>>>>>> names<br>
>>>>>>>>> should be unique to a tenant/user or not.<br>
>>>>>>>>><br>
>>>>>>>>> Maru had asked me my opinion on whether names should be unique and I<br>
>>>>>>>>> answered my personal opinion that no, they should not be, and if<br>
>>>>>>>>> Neutron<br>
>>>>>>>>> needed to ensure that there was one and only one default security<br>
>>>>>>>>> group for<br>
>>>>>>>>> a tenant, that a way to accomplish such a thing in a race-free<br>
>>>>>>>>> way, without<br>
>>>>>>>>> use of SELECT FOR UPDATE, was to use the approach I put into the<br>
>>>>>>>>> pastebin on<br>
>>>>>>>>> the review above.<br>
>>>>>>>>><br>
>>>>>>>><br>
>>>>>>>> I agree with Jay.  We should not care about how a user names the<br>
>>>>>>>> resource.<br>
>>>>>>>> There other ways to prevent this race and Jay’s suggestion is a<br>
>>>>>>>> good one.<br>
>>>>>>><br>
>>>>>>> However we should open a bug against Horizon because the user<br>
>>>>>>> experience there<br>
>>>>>>> is terrible with duplicate security group names.<br>
>>>>>><br>
>>>>>> The reason security group names are unique is that the ec2 api<br>
>>>>>> supports source<br>
>>>>>> rule specifications by tenant_id (user_id in amazon) and name, so<br>
>>>>>> not enforcing<br>
>>>>>> uniqueness means that invocation in the ec2 api will either fail or be<br>
>>>>>> non-deterministic in some way.<br>
>>>>><br>
>>>>> So we should couple our API evolution to EC2 API then?<br>
>>>>><br>
>>>>> -jay<br>
>>>><br>
>>>> No I was just pointing out the historical reason for uniqueness, and<br>
>>>> hopefully<br>
>>>> encouraging someone to find the best behavior for the ec2 api if we<br>
>>>> are going<br>
>>>> to keep the incompatibility there. Also I personally feel the ux is<br>
>>>> better<br>
>>>> with unique names, but it is only a slight preference.<br>
>>><br>
>>> Sorry for snapping, you made a fair point.<br>
>><br>
>> Yeh, honestly, I agree with Vish. I do feel that the UX of that<br>
>> constraint is useful. Otherwise you get into having to show people UUIDs<br>
>> in a lot more places. While those are good for consistency, they are<br>
>> kind of terrible to show to people.<br>
><br>
> While there is a good case for the UX of unique names - it also makes orchestration via tools like puppet a heck of a lot simpler - the fact is that most OpenStack resources do not require unique names.  That being the case, why would we want security groups
 to deviate from this convention?<br>
<br>
Maybe the other ones are the broken ones?<br>
<br>
Honestly, any sanely usable system makes names unique inside a<br>
container. Like files in a directory. In this case the tenant is the<br>
container, which makes sense.<br>
<br>
It is one of many places that OpenStack is not consistent. But I'd<br>
rather make things consistent and more usable than consistent and less.<u></u><u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">+1. <u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">More consistent and more usable is a good approach. The name uniqueness has prior art in OpenStack - keystone keeps project names unique within a domain (domain is the container), similar usernames can't be duplicated in the same domain.
 It would be silly to auth with the user ID, likewise unique names for the security group in the container (tenant) makes a lot of sense from a UX Perspective. <u></u><u></u></p>
</div>
</div></div><div>
<p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><b><i><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">[Rockyg] +1<u></u><u></u></span></i></b></p>
<p class="MsoNormal"><b><i><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Especially when dealing with domain data that are managed by Humans, human visible unique is important for understanding *and* efficiency.  Tenant security
 is expected to be managed by the tenant admin, not some automated “robot admin” and as such needs to be clear , memorable and seperable between instances.  Unique names is the most straightforward (and easiest to enforce) way do this for humans. Humans read
 differentiate alphanumerics, so that should be the standard differentiator when humans are expected to interact and reason about containers.<u></u><u></u></span></i></b></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d"><u></u> <u></u></span></p>
</div>
<div>
<p class="MsoNormal">--Morgan  <u></u><u></u></p>
</div>
</div>
</div>

<br>_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br></div>