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

Incorrect Interface Numbering Order in get_lab_from_api and update_lab_from_api #322

Open
tcaiazzi opened this issue Nov 20, 2024 · 0 comments

Comments

@tcaiazzi
Copy link
Member

Description

Currently, the get_lab_from_api and update_lab_from_api methods assign interface numbers based on the alphabetic order of the collision domains they are attached to. This behavior does not respect the order imposed by:

  • The lab.conf file for static lab configurations.
  • The order of lconfig commands during dynamic updates.

This mismatch can lead to unexpected behavior, inconsistency, and potential errors in scenarios where interface ordering is crucial for proper lab operation.

Proposed Solution

Revise the logic in the get_lab_from_api and update_lab_from_api methods to ensure that interface numbering aligns with the intended configuration order.

To achieve this, leverage the interface numbers stored in the Docker networks' DriverOpts, introduced in #321. This will allow consistent and accurate interface ordering as defined by the configuration

@tcaiazzi tcaiazzi added the bug label Nov 20, 2024
@tcaiazzi tcaiazzi self-assigned this Nov 20, 2024
@tcaiazzi tcaiazzi added this to the Release 3.7.8 milestone Nov 20, 2024
tcaiazzi added a commit that referenced this issue Nov 20, 2024
Fix the interfaces number while getting (updating) the network scenario from the API
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

2 participants