<div dir="ltr">Also, how about checking for duplicated name? That it matches standard for DNS?<div><br></div><div>> <span style="font-size:12.8000001907349px;font-family:arial,sans-serif">return u"{node_id}".format(node_id=</span><span style="font-size:12.8000001907349px;font-family:arial,sans-serif">instance.name.lower())</span></div><div><span style="font-size:12.8000001907349px;font-family:arial,sans-serif">you could just "return </span><span style="font-family:arial,sans-serif;font-size:12.8000001907349px">instance.name.lower()" I believe..</span></div><div><span style="font-family:arial,sans-serif;font-size:12.8000001907349px"><br></span></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 10, 2014 at 7:00 PM, Igor Kalnitsky <span dir="ltr"><<a href="mailto:ikalnitsky@mirantis.com" target="_blank">ikalnitsky@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Dmitry,<br>
<br>
It's not as easy as you think. The node's name may be reseted during<br>
some of our actions. For example, if you remove node from a cluster a<br>
node's name will be reseted.<br>
<br>
So the idea of the blueprint [1] is to provide mechanism when you once<br>
set names and they will not be reseted.<br>
<br>
[1]: <a href="https://blueprints.launchpad.net/fuel/+spec/node-naming" target="_blank">https://blueprints.launchpad.net/fuel/+spec/node-naming</a><br>
<br>
- Igor<br>
<div><div class="h5"><br>
On Wed, Sep 10, 2014 at 4:19 PM, Dmitry Ukov <<a href="mailto:dukov@mirantis.com">dukov@mirantis.com</a>> wrote:<br>
> Hello Fuelers,<br>
> I've figured out that there is a blueprint regarding node naming<br>
> (<a href="https://blueprints.launchpad.net/fuel/+spec/node-naming" target="_blank">https://blueprints.launchpad.net/fuel/+spec/node-naming</a>). It looks like<br>
> this this feature really has not been started. That being said all necessary<br>
> preparations in Nailgun code have been already done. The only thing that is<br>
> needed to be done is to enable this feature (in fact 4 lines of code).<br>
> --- a/nailgun/nailgun/objects/node.py<br>
> +++ b/nailgun/nailgun/objects/node.py<br>
> @@ -654,7 +654,10 @@ class Node(NailgunObject):<br>
><br>
>      @classmethod<br>
>      def make_slave_name(cls, instance):<br>
> -        return u"node-{node_id}".format(node_id=<a href="http://instance.id" target="_blank">instance.id</a>)<br>
> +        if instance.name.startswith("Untitled"):<br>
> +            return u"node-{node_id}".format(node_id=<a href="http://instance.id" target="_blank">instance.id</a>)<br>
> +        else:<br>
> +            return u"{node_id}".format(node_id=instance.name.lower())<br>
><br>
>      @classmethod<br>
>      def make_slave_fqdn(cls, instance):<br>
><br>
> Should I consider this as a bug or this blueprint will be implemented in the<br>
> nearest future?<br>
><br>
><br>
> --<br>
> Kind regards<br>
> Dmitry Ukov<br>
> IT Engineer<br>
> Mirantis, Inc.<br>
><br>
><br>
</div></div>> _______________________________________________<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>
<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>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Mike Scherbakov<br>#mihgen<br><br></div>
</div>