<div dir="ltr">Thanks Adrian, some questions and comments in-line.<br><div class="gmail_extra"><br><div class="gmail_quote">2015-06-03 10:29 GMT+08:00 Adrian Otto <span dir="ltr"><<a href="mailto:adrian.otto@rackspace.com" target="_blank">adrian.otto@rackspace.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div style="word-wrap:break-word">
I have reflected on this further and offer this suggestion:
<div><br>
</div>
<div>1) Add a feature to Magnum to auto-generate human readable names, like Docker does for un-named containers, and ElasticSearch does for naming cluster nodes. Use this feature if no name is specified upon the creation of a Bay or Baymodel.</div></div></blockquote><div>+1 on this <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word">
<div><br>
</div>
<div>-and-</div>
<div><br>
</div>
<div>2) Add a configuration directives (default=FALSE) for allow_duplicate_bay_name and allow_duplicate_baymodel_name. If TRUE, duplicate named Bay and BayModel resources will be allowed, as they are today.</div>
<div><br>
</div>
<div>This way, by default Magnum requires a unique name, and if none is specified, it will automatically generate a name. This way no additional burden is put on users who want to act on containers exclusively using UUIDs, and cloud operators can decide
if they want to enforce name uniqueness or not. </div>
<div><br>
</div>
<div>In the case of clouds that want to allow sharing access to a BayModel between multiple tenants (example: a global BayModel named “kubernetes”) with allow_duplicate_baymodel_name set to FALSE, a user will still be allowed to</div></div></blockquote><div>Here should be allow_duplicate_baymodel set to TRUE? </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><div> create a BayModel with
the name “kubernetes” and it will override the global one. If a user-supplied BayModel is present with the same name as a global one, we shall automatically select the one owned by the tenant.</div></div></blockquote><div>+1 on this , one question is what does a "global BayModel" means? In Magnum, all BayModel belong to a tenant and seems there is no global BayModel?<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word">
<div><br>
</div>
<div>About Sharing of BayModel Resources:</div>
<div><br>
</div>
<div>Similarly, if we add features to allow one tenant to share a BayModel with another tenant (pending acceptance of the offered share), and duplicate names are allowed, then prefer in this order: 1) Use the resource owned by the same tenant, 2) Use
the resource shared by the other tenant (post acceptance only), 3) Use the global resource. If duplicates exist in the same scope of ownership, then raise an exception requiring the use of a UUID in that case to resolve the ambiguity.</div></div></blockquote><div>We can file a bp to trace this. <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word">
<div><br>
</div>
<div>One expected drawback of this approach is that tools designed to integrate with one Magnum may not work the same with another Magnum if the allow_duplicate_bay* settings are changed from the default values on one but not the other. This should
be made clear in the comments above the configuration directive in the example config file.</div></div></blockquote><div>Just curious why do we need this feature? Different Magnum clusters might using different CoE engine. So you are mentioning the case all of the Magnum clusters are using same CoE engine? If so, yes, this should be made clear in configuration file.<br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><span class=""><font color="#888888"><div>
</div>
<div>Adrian</div></font></span><div><div class="h5">
<div><br>
</div>
<div>
<div>
<blockquote type="cite">
<div>On Jun 2, 2015, at 8:44 PM, Jay Lau <<a href="mailto:jay.lau.513@gmail.com" target="_blank">jay.lau.513@gmail.com</a>> wrote:</div>
<br>
<div>
<div dir="ltr">
<div>
<div>
<div>I think that we did not come to a conclusion in today's IRC meeting.<br>
<br>
</div>
Adrian proposed that Magnum generate a unique name just like what docker is doing for "docker run", the problem mentioned by Andrew Melton is that Magnum support multi tenant, we should support the case that bay/baymodel under different tenant can have same
name, the unique name is not required.<br>
<br>
</div>
Also we may need support name update as well if the end user specify a name by mistake and want to update it after the bay/baymodel was created.<br>
<br>
</div>
Hmm.., looking forward to more comments from you. Thanks.<br>
</div>
<div class="gmail_extra"><br>
<div class="gmail_quote">2015-06-02 23:34 GMT+08:00 Fox, Kevin M <span dir="ltr">
<<a href="mailto:Kevin.Fox@pnnl.gov" target="_blank">Kevin.Fox@pnnl.gov</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div>
<div style="direction:ltr;font-family:Tahoma;font-size:10pt">Names can make writing generic orchestration templates that would go in the applications catalog easier. Humans are much better at inputting a name rather then a uuid. You can even
default a name in the text box and if they don't change any of the defaults, it will just work. You can't do that with a UUID since it is different on every cloud.<br>
<br>
Thanks,<br>
Kevin<br>
<div style="font-family:"Times New Roman";font-size:16px">
<hr>
<div style="direction:ltr"><font size="2" face="Tahoma"><b>From:</b> Jay Lau [<a href="mailto:jay.lau.513@gmail.com" target="_blank">jay.lau.513@gmail.com</a>]<br>
<b>Sent:</b> Tuesday, June 02, 2015 12:33 AM<span><br>
<b>To:</b> OpenStack Development Mailing List (not for usage questions)<br>
<b>Subject:</b> Re: [openstack-dev] [Magnum] Does Bay/Baymodel name should be a required option when creating a Bay/Baymodel<br>
</span></font><br>
</div>
<div>
<div>
<div></div>
<div>
<div dir="ltr">Thanks Adrian, imho making name as required can bring more convenient to end users because UUID is difficult to use. Without name, the end user need to retrieve the UUID of the bay/baymodel first before he did some operations for the
bay/baymodel, its really time consuming. We can discuss more in this week's IRC meeting. Thanks.<br>
<br>
</div>
<div class="gmail_extra"><br>
<div class="gmail_quote">2015-06-02 14:08 GMT+08:00 Adrian Otto <span dir="ltr">
<<a href="mailto:adrian.otto@rackspace.com" target="_blank">adrian.otto@rackspace.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div dir="auto">
<div>-1. I disagree.</div>
<div><br>
</div>
<div>I am not convinced that requiring names is a good idea. I've asked several times why there is a desire to require names, and I'm not seeing any persuasive arguments that are not already addressed by UUIDs. We have UUID values to allow for acting
upon an individual resource. Names are there as a convenience. Requiring names, especially unique names, would make Magnum harder to use for API users driving Magnum from other systems. I want to keep the friction as low as possible.<br>
<br>
I'm fine with replacing "None" with an empty string.</div>
<div><br>
</div>
<div>Consistency with Nova would be a valid argument if we were being more restrictive, but that's not the case. We are more permissive. You can use Magnum in the same way you use Nova if you want, by adding names to all resources. I don't see the
wisdom in forcing that style of use without a technical reason for it.</div>
<div><br>
Thanks,</div>
<div><br>
<div>Adrian</div>
</div>
<div>
<div>
<div><br>
On May 31, 2015, at 4:43 PM, Jay Lau <<a href="mailto:jay.lau.513@gmail.com" target="_blank">jay.lau.513@gmail.com</a>> wrote:<br>
<br>
</div>
<blockquote type="cite">
<div>
<div dir="ltr">
<div>
<div>
<div><br>
</div>
Just want to use ML to trigger more discussion here. There are now bugs/patches tracing this, but seems more discussions are needed before we come to a conclusion.<br>
<br>
<a href="https://bugs.launchpad.net/magnum/+bug/1453732" target="_blank">https://bugs.launchpad.net/magnum/+bug/1453732</a><br>
<a href="https://review.openstack.org/#/c/181839/" target="_blank">https://review.openstack.org/#/c/181839/</a><br>
<a href="https://review.openstack.org/#/c/181837/" target="_blank">https://review.openstack.org/#/c/181837/</a><br>
<a href="https://review.openstack.org/#/c/181847/" target="_blank">https://review.openstack.org/#/c/181847/</a><br>
<a href="https://review.openstack.org/#/c/181843/" target="_blank">https://review.openstack.org/#/c/181843/</a><br>
<br>
</div>
IMHO, making the Bay/Baymodel name as a MUST will bring more flexibility to end user as Magnum also support operating Bay/Baymodel via names and the name might be more meaningful to end users.<br>
<p>Perhaps we can borrow some iead from nova, the concept in magnum can be mapped to nova as following:</p>
<p>1) instance => bay<br>
2) flavor => baymodel</p>
<p>So I think that a solution might be as following:<br>
1) Make name as a MUST for both bay/baymodel<br>
2) Update magnum client to use following style for bay-create and baymodel-create: DO NOT add "--name" option</p>
<p>root@devstack007:/tmp# nova boot<br>
usage: nova boot [--flavor <flavor>] [--image <image>]<br>
[--image-with <key=value>] [--boot-volume <volume_id>]<br>
[--snapshot <snapshot_id>] [--min-count <number>]<br>
[--max-count <number>] [--meta <key=value>]<br>
[--file <dst-path=src-path>] [--key-name <key-name>]<br>
[--user-data <user-data>]<br>
[--availability-zone <availability-zone>]<br>
[--security-groups <security-groups>]<br>
[--block-device-mapping <dev-name=mapping>]<br>
[--block-device key1=value1[,key2=value2...]]<br>
[--swap <swap_size>]<br>
[--ephemeral size=<size>[,format=<format>]]<br>
[--hint <key=value>]<br>
[--nic <net-id=net-uuid,v4-fixed-ip=ip-addr,v6-fixed-ip=ip-addr,port-id=port-uuid>]<br>
[--config-drive <value>] [--poll]<br>
<name><br>
error: too few arguments<br>
Try 'nova help boot' for more information.<br>
root@devstack007:/tmp# nova flavor-create<br>
usage: nova flavor-create [--ephemeral <ephemeral>] [--swap <swap>]<br>
[--rxtx-factor <factor>] [--is-public <is-public>]<br>
<name> <id> <ram> <disk> <vcpus></p>
Please show your comments if any.<br>
</div>
<div>
<div>
<div><br>
-- <br>
<div>
<div dir="ltr">
<div>
<div dir="ltr">
<div>Thanks,<br>
<br>
</div>
Jay Lau (Guangya Liu)<br>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</blockquote>
</div>
</div>
<blockquote type="cite">
<div><span>__________________________________________________________________________</span><span><br>
<span>OpenStack Development Mailing List (not for usage questions)</span><br>
<span>Unsubscribe: <a href="mailto:OpenStack-dev-request@lists.openstack.org" target="_blank">
OpenStack-dev-request@lists.openstack.org</a>?subject:unsubscribe</span><br>
<span><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></span><br>
</span></div>
</blockquote>
</div>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org/?subject:unsubscribe" target="_blank">
OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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>
<br clear="all">
<br>
-- <br>
<div>
<div dir="ltr">
<div>
<div dir="ltr">
<div>Thanks,<br>
<br>
</div>
Jay Lau (Guangya Liu)<br>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org/?subject:unsubscribe" target="_blank">
OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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>
<br clear="all">
<br>
-- <br>
<div>
<div dir="ltr">
<div>
<div dir="ltr">
<div>Thanks,<br>
<br>
</div>
Jay Lau (Guangya Liu)<br>
</div>
</div>
</div>
</div>
</div>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="mailto:OpenStack-dev-request@lists.openstack.org" target="_blank">
OpenStack-dev-request@lists.openstack.org</a>?subject:unsubscribe<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>
</div>
</blockquote>
</div>
<br>
</div>
</div></div></div>
<br>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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><br clear="all"><br>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Thanks,<br><br></div>Jay Lau (Guangya Liu)<br></div></div></div></div>
</div></div>