Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LAG interfaces can be connected via CSV import #2127

Closed
mandarg opened this issue May 29, 2018 · 1 comment
Closed

LAG interfaces can be connected via CSV import #2127

mandarg opened this issue May 29, 2018 · 1 comment
Labels
status: accepted This issue has been accepted for implementation type: bug A confirmed report of unexpected behavior in the application

Comments

@mandarg
Copy link
Contributor

mandarg commented May 29, 2018

An example scenario:

  • There are two devices, test1 and test2.
  • test1 has a LAG bundle po1 consisting of intf0/1 and intf0/2
  • test2 has a LAG bundle po2 consisting of intf0/1 and intf0/2
  • test1:intf0/1 is connected to test2:intf0/1
  • test1:intf0/2 is connected to test2:intf0/2

In order to represent that their LAG bundles are also "connected", I connected up test1:po1 to test2:po2 as well (connections displayed as below):

interface-connections-view

Fig 1: Devices > Interface connections view


Note that this is not possible to easily do using the standard UI controls (there is no connection button on a LAG interface). However, it works as expected through "Import Connections" or the API.

At this point, I can view test1:po1 and test2:po2 as connected in the Interface Connections view, but if I go to the test1 interface view or the test2 interface view (both shown below), I cannot see the connection.

test1-interfaces

Fig 2: test1 Interfaces section


test2-interfaces

Fig 3: test2 Interfaces section


Should this be rectified? For example, I would expect a connection to test2:po1 to be shown against po1 in the interfaces view of test1 above (Fig 2.)

@jeremystretch
Copy link
Member

This is a bug: it should not be possible to connect LAG interfaces, as they are virtual. It seems the validation is not being enforced during import.

@jeremystretch jeremystretch added type: bug A confirmed report of unexpected behavior in the application status: accepted This issue has been accepted for implementation labels Jun 6, 2018
@jeremystretch jeremystretch changed the title LAG bundle connections are not visible in the UI LAG interfaces can be connected via CSV import Jun 6, 2018
@lock lock bot locked as resolved and limited conversation to collaborators Jan 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: accepted This issue has been accepted for implementation type: bug A confirmed report of unexpected behavior in the application
Projects
None yet
Development

No branches or pull requests

2 participants