Skip to content

test(e2e): add e2e check for interface order after ClusterNetwork removal#2124

Merged
LopatinDmitr merged 1 commit into
mainfrom
test/e2e/order-interfaces
Mar 24, 2026
Merged

test(e2e): add e2e check for interface order after ClusterNetwork removal#2124
LopatinDmitr merged 1 commit into
mainfrom
test/e2e/order-interfaces

Conversation

@LopatinDmitr
Copy link
Copy Markdown
Contributor

@LopatinDmitr LopatinDmitr commented Mar 22, 2026

Description

Add e2e coverage for additional network interface ordering when ClusterNetworks are removed.

The tests now support checking multiple VLAN-based ClusterNetworks and verify that guest interface names stay stable after removing the middle ClusterNetwork and rebooting the VM.

Why do we need it, and what problem does it solve?

We need regression coverage for interface renumbering/order issues in VMs with multiple additional networks.

Without this check, removing one ClusterNetwork could silently change interface naming/order in guest OS after reboot, which may break network configuration relying on persistent interface names.

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

Changelog entries

section: test
type: fix
summary: Add e2e regression test for preserving VM additional interface naming after middle ClusterNetwork removal and reboot.
impact_level: low

@LopatinDmitr LopatinDmitr requested a review from Isteb4k as a code owner March 22, 2026 19:01
@LopatinDmitr LopatinDmitr added this to the v1.8.0 milestone Mar 22, 2026
@LopatinDmitr LopatinDmitr changed the title test(network): add e2e check for interface order after cn removal test(e2e): add e2e check for interface order after ClusterNetwork removal Mar 22, 2026
@LopatinDmitr LopatinDmitr requested a review from danilrwx March 22, 2026 19:03
Signed-off-by: Dmitry Lopatin <dmitry.lopatin@flant.com>
@LopatinDmitr LopatinDmitr force-pushed the test/e2e/order-interfaces branch from e727c82 to e2edb99 Compare March 22, 2026 19:04
@LopatinDmitr LopatinDmitr merged commit 6fede87 into main Mar 24, 2026
29 of 30 checks passed
@LopatinDmitr LopatinDmitr deleted the test/e2e/order-interfaces branch March 24, 2026 07:52
Isteb4k pushed a commit that referenced this pull request Mar 25, 2026
…oval (#2124)

Signed-off-by: Dmitry Lopatin <dmitry.lopatin@flant.com>
(cherry picked from commit 6fede87)
@Isteb4k Isteb4k modified the milestones: v1.8.0, v1.7.0 Mar 31, 2026
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.

3 participants