After re-writing list_interfaces.py to use Interfaces() pseudo-ethernet is missing
Closed, ResolvedPublicBUG
Actions

Assigned To

Authored By

	c-po
	Apr 7 2020, 7:50 PM

Description

Pseudo-Ethernet/MACVLAN interfaces should be listed on the list_interfaces.py --bridgable completion helper

Interface can be bridged, thus it should show up on the completion helper:

[email protected]# run show bridge
bridge name     bridge id               STP enabled     interfaces
br0             8000.0af4308d4071       no              l2tpeth10
                                                        peth0

Details

Difficulty level: Easy (less than an hour)
Version: 1.3-rolling-202004070927
Why the issue appeared?: Will be filled on close
Is it a breaking change?: Unspecified (possibly destroys the router)
Issue type: Bug (incorrect behavior)

Revisions and Commits

Restricted Diffusion Commit

Related Objects

Mentioned In: T2171: Unify creation and manipulation of interfaces

Event Timeline

c-po assigned this task to thomas-mangin.Apr 7 2020, 7:50 PM

c-po triaged this task as Normal priority.

c-po created this task.

c-po updated the task description. (Show Details)

Hi @cpo,

Yes, it is missing, nothing to do with me but the fact the code does not yet find what is what from the interface data as we discussed.
https://github.com/vyos/vyos-1x/commit/bbea850ea5f8ff0402cd276ab63963ece7e0c763#diff-667867449bff9faf1ac285125ceada77

It will auto-magically get fixed when I finish the migration of show interfaces to python and update the file as the work is related and both will be done at the same time.
Hopefully within the week-end.

pasik added a subscriber: pasik.Apr 11 2020, 9:31 AM

https://github.com/vyos/vyos-1x/pull/331

thomas-mangin closed this task as Resolved.Apr 11 2020, 7:47 PM

thomas-mangin added a comment.Apr 11 2020, 9:50 PM

This comment was removed by thomas-mangin.

thomas-mangin added a commit: Restricted Diffusion Commit.Apr 11 2020, 9:52 PM

thomas-mangin mentioned this in T2171: Unify creation and manipulation of interfaces.Apr 11 2020, 10:30 PM

c-po moved this task from Need Triage to Finished on the VyOS 1.3 Equuleus board.May 21 2020, 5:24 PM

erkin set Issue type to Bug (incorrect behavior).Aug 30 2021, 7:36 AM

erkin removed a subscriber: Active contributors.

syncer edited projects, added VyOS 1.3 Equuleus (1.3.0); removed VyOS 1.3 Equuleus.Aug 30 2022, 2:01 PM

syncer moved this task from Need Triage to Finished on the VyOS 1.3 Equuleus (1.3.0) board.