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

Avoid using VLAN interfaces for physical interface modifications #51

Merged
merged 4 commits into from
Mar 20, 2024

Conversation

atanasdinov
Copy link
Collaborator

@atanasdinov atanasdinov commented Mar 18, 2024

@atanasdinov atanasdinov marked this pull request as ready for review March 18, 2024 12:13
@atanasdinov atanasdinov changed the title Prevent content modification for VLAN interfaces Avoid using VLAN interfaces for physical interface modifications Mar 18, 2024
Copy link

@diconico07 diconico07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, I wonder if the mac address would be better represented as an Option<String> but that might make the code more complicated overall.

@atanasdinov
Copy link
Collaborator Author

This is so much natural and I have no clue why it didn't come to mind at all.. Fixed!

Signed-off-by: Atanas Dinov <[email protected]>
@atanasdinov atanasdinov merged commit 9885d7e into suse-edge:main Mar 20, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for VLAN based interfaces
2 participants