fuel-dev team mailing list archive

[Igor Kalnitsky <ikalnitsky@xxxxxxxxxxxx>]

Hi there,

Actually, I think it's a bad idea. As you already said, interface
names are come from nailgun agents and if we change them in database
(by means UI or CLI) the nodes will know nothing about these changes.

This leads us to the fact, that we can loose a node if nailgun agent
doesn't send a mac address (which happens time to time), since we
can't rely on interface names anymore [1].

If this topic was raised only for UI improvements, we can add a new
field - "label" - which will be shown instead of interface name. What
do you think? Will be it enough for customers?

[1]: https://github.com/stackforge/fuel-web/blob/stable/5.1/nailgun/nailgun/network/manager.py#L649

Thanks,
Igor

On Thu, Oct 9, 2014 at 2:34 PM, Vyacheslav Valyavskiy
<vvalyavskiy@xxxxxxxxxxxx> wrote:
> Hello everyone,
>
> Recently, we had request from our customer to provide feature to change
> node's interfaces names.
>
> I have researched it a little bit and it seems like there is no big problem
> to implement it.
>
> I have hacked nailgun agent to send non-standard names to api and node was
> provisioned successfully with
>
> custom interfaces names.
>
>
> This part of the code updates interfaces-networks assignment and we can
> update interfaces names here, too.
> https://github.com/stackforge/fuel-web/blob/stable/5.1/nailgun/nailgun/network/manager.py#L586-L600
>
> Then, we can update interfaces names using cli immediately.
> ###############
> - assigned_networks: []
>   current_speed: 100
>   id: 10
>   mac: 52:bf:dd:5e:f3:13
>   max_speed: 100
>   name: eth0 # here we can replace iface name with something else
>   state: up
>   type: ether
> ###############
>
> Fuel UI. Here, type of the interface name field should be modified to be
> changeable. And, than, UI can generate proper
>
> api request.
>
> One thing here which make sense it's that bootstraped nodes send data
> request with real interfaces names from time to time. And, conflict may be
> occurred. I didn't check it.
>
> What do think about this feature guys?
>
>
>
> --
> --
> With best regards,
> Viacheslav A. Valyavskiy
> Deployment engineer
> Mirantis, Inc. http://www.mirantis.com
>
> --
> Mailing list: https://launchpad.net/~fuel-dev
> Post to     : fuel-dev@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~fuel-dev
> More help   : https://help.launchpad.net/ListHelp
>