Skip to content

Conversation

@felix-kaestner
Copy link
Contributor

This change introduces a new custom resource to configure a VXLAN mapping for a VLAN (Bridge Domain) and it's corresponding EVPN control plane context as a MAC-VRF.

@felix-kaestner felix-kaestner requested a review from a team as a code owner November 18, 2025 15:58
@hardikdr hardikdr added the area/metal-automation Automation processes within the Metal project. label Nov 19, 2025
@hardikdr hardikdr added this to Roadmap Nov 19, 2025
@felix-kaestner felix-kaestner force-pushed the evi-l2vni branch 3 times, most recently from 79bdca7 to 5e6fded Compare November 20, 2025 10:08
@felix-kaestner felix-kaestner force-pushed the evi-l2vni branch 2 times, most recently from c985c23 to 9938574 Compare November 26, 2025 13:20
Copy link
Contributor

@nikatza nikatza 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 to me. I've left a comment pointing to a PR that might conflict.

Base automatically changed from ipv4-validation to main December 3, 2025 14:15
This change introduces a new custom resource to configure a VXLAN
mapping for a VLAN (Bridge Domain) and it's corresponding EVPN control
plane context as a MAC-VRF.
@github-actions
Copy link

github-actions bot commented Dec 3, 2025

Merging this branch will increase overall coverage

Impacted Packages Coverage Δ 🤖
github.com/ironcore-dev/network-operator/api/core/v1alpha1 0.00% (ø)
github.com/ironcore-dev/network-operator/cmd 0.00% (ø)
github.com/ironcore-dev/network-operator/internal/controller/core 69.73% (+0.63%) 👍
github.com/ironcore-dev/network-operator/internal/provider 20.00% (ø)
github.com/ironcore-dev/network-operator/internal/provider/cisco/nxos 12.44% (+0.15%) 👍

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/ironcore-dev/network-operator/api/core/v1alpha1/evpninstance_types.go 0.00% (ø) 0 0 0
github.com/ironcore-dev/network-operator/api/core/v1alpha1/groupversion_info.go 0.00% (ø) 0 0 0
github.com/ironcore-dev/network-operator/api/core/v1alpha1/vlan_types.go 0.00% (ø) 0 0 0
github.com/ironcore-dev/network-operator/api/core/v1alpha1/zz_generated.deepcopy.go 0.00% (ø) 0 0 0
github.com/ironcore-dev/network-operator/cmd/main.go 0.00% (ø) 0 0 0
github.com/ironcore-dev/network-operator/internal/controller/core/evpninstance_controller.go 77.02% (+77.02%) 161 (+161) 124 (+124) 37 (+37) 🌟
github.com/ironcore-dev/network-operator/internal/provider/cisco/nxos/evi.go 72.73% (+72.73%) 22 (+22) 16 (+16) 6 (+6) 🌟
github.com/ironcore-dev/network-operator/internal/provider/cisco/nxos/intf.go 11.11% (-0.12%) 90 (+1) 10 80 (+1) 👎
github.com/ironcore-dev/network-operator/internal/provider/cisco/nxos/nve.go 66.67% (-33.33%) 3 (+2) 2 (+1) 1 (+1) 💀 💀 💀
github.com/ironcore-dev/network-operator/internal/provider/cisco/nxos/provider.go 0.09% (-0.01%) 1060 (+82) 1 1059 (+82) 👎
github.com/ironcore-dev/network-operator/internal/provider/cisco/nxos/vlan.go 15.79% (-27.07%) 19 (+12) 3 16 (+12) 💀 💀
github.com/ironcore-dev/network-operator/internal/provider/provider.go 20.00% (ø) 25 5 20

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

Changed unit test files

  • github.com/ironcore-dev/network-operator/internal/controller/core/evpninstance_controller_test.go
  • github.com/ironcore-dev/network-operator/internal/controller/core/suite_test.go
  • github.com/ironcore-dev/network-operator/internal/provider/cisco/nxos/evi_test.go
  • github.com/ironcore-dev/network-operator/internal/provider/cisco/nxos/nve_test.go
  • github.com/ironcore-dev/network-operator/internal/provider/cisco/nxos/vlan_test.go

@felix-kaestner felix-kaestner merged commit 7f1a105 into main Dec 3, 2025
9 checks passed
@github-project-automation github-project-automation bot moved this to Done in Roadmap Dec 3, 2025
@felix-kaestner felix-kaestner deleted the evi-l2vni branch December 3, 2025 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/metal-automation Automation processes within the Metal project.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants