Jaime Caamaño Ruiz
Jaime Caamaño Ruiz
> Failed job: [e2e (control-plane, noHA, local, ipv6, noSnatGW, 1br, ic-single-node-zones)](https://github.com/ovn-org/ovn-kubernetes/actions/runs/11325594330/job/31494087027?pr=4759#logs) > > https://github.com/ovn-org/ovn-kubernetes/actions/runs/11325594330/job/31494087027?pr=4759 > > ``` > Summarizing 7 Failures: > [FAIL] EgressService Multiple Networks, external clients sharing ip...
> > We will most likely see this more. Thinking what to do but we will probably have to stay with 1.30.2 at runtime for a bit. > > TY...
> LGTM overall, just some minor comments. Can we use 1.31.1 consistently in kind and e2es? there is no e2es at version 1.31.1, just 1.31.0 > What is the reason...
Added a bit more information to the comment where `service-lb-controller` is disabled and rebased.
Had to resolve a couple of conflicts after rebase: - https://github.com/ovn-org/ovn-kubernetes/pull/4647 applied a new condition that to be adapted to the new API https://github.com/ovn-org/ovn-kubernetes/pull/4759/files#diff-891f2bd2e1eea7ee0df79d1ed3d55e72df60b9a2bbb60fe64374e8131fb7a269R397-R405 - https://github.com/ovn-org/ovn-kubernetes/pull/4749 intriduced changes to go.mod...
@kyrtapz PTAL
/hold While I would like relevant feedback, this may still not be set on its final form. The option proposed with two flags is clean but it may appear evident...
@JoelSpeed PTAL