Page MenuHomeVyOS Platform
Create Task
Maniphest T3871

Resolve unexpected interface name reordering
Needs testing, NormalPublicBUG
Actions

Description

It is a recurring problem that on certain platforms ethernet interfaces will have changed associations: interface name <-> hw-id, after a reboot or upgrade. There has been active discussion regarding the problem recently, and consensus on a preliminary design to resolve the issue. This task will serve as meta-task for investigation and resolution.

An incomplete collection of related tasks:

https://phabricator.vyos.net/T291
https://phabricator.vyos.net/T577
https://phabricator.vyos.net/T1499
https://phabricator.vyos.net/T1671
https://phabricator.vyos.net/T2838

Details

Difficulty level
Unknown (require assessment)
Version
vyos-1.4
Why the issue appeared?
Will be filled on close
Is it a breaking change?
Unspecified (possibly destroys the router)
Issue type
Bug (incorrect behavior)

Related Objects
Search...

Event Timeline

jestabro created this task.Sep 28 2021, 5:02 PM
jestabro created this object in space S1 VyOS Public.
jestabro updated the task description. (Show Details)
jestabro added a subtask: T3869: Rewrite vyatta_net_name/vyatta_interface_rescan in Python.
jestabro mentioned this in T3869: Rewrite vyatta_net_name/vyatta_interface_rescan in Python.
pasik added a subscriber: pasik.Sep 28 2021, 7:44 PM
jestabro closed subtask T3869: Rewrite vyatta_net_name/vyatta_interface_rescan in Python as Resolved.Sep 30 2021, 5:58 PM
jestabro mentioned this in T3876: Replace vyos-netplug with a VyOS link state monitor service.Sep 30 2021, 7:26 PM
jestabro added a subtask: T3876: Replace vyos-netplug with a VyOS link state monitor service.
remyvv added a subscriber: remyvv.Oct 20 2021, 9:45 AM
sarthurdev added a subscriber: sarthurdev.Oct 26 2021, 5:09 PM
c-po added a subtask: T4171: Interface config migration error on 1.2.8 -> 1.4 upgrade.Jan 20 2022, 7:44 PM
c-po closed subtask T4171: Interface config migration error on 1.2.8 -> 1.4 upgrade as Resolved.
eronlloyd added a subscriber: eronlloyd.Feb 20 2022, 11:51 AM
marvin added a subscriber: marvin.Apr 29 2022, 5:23 PM
daniil mentioned this in T4030: SR-IOV and interface renaming bug .Jul 18 2022, 8:38 AM
stingalleman added a subscriber: stingalleman.Aug 3 2022, 1:16 PM
syncer changed the task status from Open to In progress.Feb 2 2023, 7:38 PM
syncer triaged this task as High priority.
jestabro added a comment.Feb 2 2023, 8:01 PM

The approach in commit ee02ca93 of vyos-1x was considered a reasonable workaround for all but a few corner cases. However, the issue pointed out in

https://phabricator.vyos.net/T4966

and PR:

https://github.com/vyos/vyos-build/pull/302

re-raises the question.

syncer reassigned this task from jestabro to Viacheslav.Feb 2 2023, 9:57 PM
jestabro added a comment.Feb 3 2023, 5:49 PM

Before adopting the approach mentioned above, there was development of an alternative using pyudev within an 'interface-monitor' daemon; the following branches contain (a version of) the rebased code. It would need a few hours of attention to check the logic and add the is_persistent check from vyos-interface-rescan.py; it could use some refactoring as well.

https://github.com/vyos/vyos-1x/compare/current...jestabro:vyos-1x:interface-monitor
https://github.com/vyos/vyatta-cfg/compare/current...jestabro:vyatta-cfg:interface-monitor
https://github.com/vyos/vyos-build/compare/current...jestabro:vyos-build:interface-monitor

sarthurdev added a comment.Feb 3 2023, 9:33 PM
In T3871#141847, @jestabro wrote:

Before adopting the approach mentioned above, there was development of an alternative using pyudev within an 'interface-monitor' daemon; the following branches contain (a version of) the rebased code. It would need a few hours of attention to check the logic and add the is_persistent check from vyos-interface-rescan.py; it could use some refactoring as well.

https://github.com/vyos/vyos-1x/compare/current...jestabro:vyos-1x:interface-monitor
https://github.com/vyos/vyatta-cfg/compare/current...jestabro:vyatta-cfg:interface-monitor
https://github.com/vyos/vyos-build/compare/current...jestabro:vyos-build:interface-monitor

I think your interface-monitordaemon looks like a good long term solution to this.

jestabro added a comment.Feb 23 2023, 7:58 PM

The above repos are suitable for testing: basic tests in vm's and a Protecli device look fine. Note that the vyos-1x repo artificially disables the potential config mode reordering in order to confirm tests of vyos-interface-monitor.:
https://github.com/vyos/vyos-1x/commit/02d874e65262cbd53b413b3eef659b8e7d78bf31

It remains an open question whether the wrapper around libudev (pyudev) will satisfy all our use cases, but it a reasonable approach for testing.

Debugging info is logged if the system is booted with parameter 'vyos-interface-debug'.

The tools are in place to support hot-plugged devices, though that will be a separate round of implementation/testing, as there will again be interactions with conf_mode ifconfig/interface.py on initialization, which will need to be reconciled.

jestabro mentioned this in T2838: Ethernet device names changing, multiple hw-id being added.Mar 7 2023, 5:03 PM
syncer changed the subtype of this task from "Task" to "Bug".Jun 14 2023, 10:43 AM
stingalleman added a comment.Sep 24 2023, 4:16 PM

When will this bug be fixed? I am having a lot of issues with this.

jestabro added a comment.Sep 24 2023, 11:52 PM

@stingalleman As mentioned above (and confirmed in discussions earlier this week), we've had few if any reports of issues with the udev approach, so we would be very interested to hear details of your case.

yun added a subscriber: yun.Jan 5 2024, 9:53 PM
dmbaturin merged a task: T1671: rewrite udev script logic /lib/udev/vyatta_net_name.Jan 9 2024, 3:54 PM
dmbaturin added a subscriber: hagbard.
stingalleman added a comment.Jan 10 2024, 2:03 AM
In T3871#160827, @jestabro wrote:

@stingalleman As mentioned above (and confirmed in discussions earlier this week), we've had few if any reports of issues with the udev approach, so we would be very interested to hear details of your case.

Hi,

I was doing something stupid - whoops:)

It's working like expecting.

Thanks!

jestabro lowered the priority of this task from High to Normal.Jan 10 2024, 3:31 PM

Lowering priority to normal to proceed with adding the interface-monitor daemon development, mentioned above, for 1.5.

remyvv removed a subscriber: remyvv.Jan 10 2024, 3:33 PM
Viacheslav changed the task status from In progress to Needs testing.Jan 19 2024, 10:06 PM