Skip to content

EXOS Normalize interface#3339

Merged
ipspace merged 3 commits intoipspace:devfrom
sdargoeuves:exos-interface
Apr 25, 2026
Merged

EXOS Normalize interface#3339
ipspace merged 3 commits intoipspace:devfrom
sdargoeuves:exos-interface

Conversation

@sdargoeuves
Copy link
Copy Markdown
Collaborator

@ipspace I'm not sure how to run the test to see if this branch fixes the issue #3252 , do you mind having a look, or sharing how you did the test?

This PR disabled the physical ports as part of the normalization, and will re-enable those interfaces at the end of the initial task, so hopefully that should be enough to sort this issue.

I've also added a caveat, as I've realised the EXOS initial configuration is not idempotent due to the Vlan creation. I'm not sure how to workaround this at the moment, so caveat it is!

@ipspace
Copy link
Copy Markdown
Owner

ipspace commented Apr 23, 2026

@ipspace I'm not sure how to run the test to see if this branch fixes the issue #3252 , do you mind having a look, or sharing how you did the test?

The issue pops up in tests with two IPv6 segments where a device sending RAs on one segment is configured before the tested device. Will add the test.

This PR disabled the physical ports as part of the normalization, and will re-enable those interfaces at the end of the initial task, so hopefully that should be enough to sort this issue.

I would also remove them from VLAN 1, but I guess we already have the command to do that in one of the templates, so I can add that.

I've also added a caveat, as I've realised the EXOS initial configuration is not idempotent due to the Vlan creation.

Thank you!

It might take a day or two before this gets merged, I'm busy with a totally unrelated project 🤷🏻‍♂️

ipspace added a commit that referenced this pull request Apr 25, 2026
Copy link
Copy Markdown
Owner

@ipspace ipspace left a comment

Choose a reason for hiding this comment

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

Works like a charm. Thank you!

@ipspace ipspace merged commit 6bb31d5 into ipspace:dev Apr 25, 2026
5 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.

2 participants