Nadia Pinaeva
Nadia Pinaeva
not sure if you would like to include this, but we were going to add some CEL for the UDN CRD, there are TODOs like this https://github.com/ovn-org/ovn-kubernetes/blob/master/go-controller/pkg/crd/userdefinednetwork/v1/types.go#L213 it just needs...
> Are there no tests for those CELs? not really :(
reported 1 flake github.com/ovn-kubernetes/ovn-kubernetes/issues/5309 and 1 fix github.com/ovn-kubernetes/ovn-kubernetes/issues/5308
/retest-failed
Converted flake to bug https://github.com/ovn-kubernetes/ovn-kubernetes/issues/5309 we need to fix it, but this PR can go in
/retest-failed
cc @kyrtapz @tssurya https://github.com/ovn-kubernetes/ovn-kubernetes/pull/5202#issuecomment-3023764889 if that is ok, should be ready for another review/merge
I also think we need to describe the tunnel-key allocation here, I have already explained the problem and some potential solutions here https://github.com/ovn-kubernetes/ovn-kubernetes/pull/5485/files#diff-8e8bdbb5f432becc3618489c074f91b6d35097cc75bc1a353e4263a1cf311a22R466
/retest-failed