Carlos Gonçalves
Carlos Gonçalves
Thanks for the fix! Would it be possible to add test coverage? E2E would be great, if possible.
@fedepaol @gkopels please review.
@danwinship could you please revisit this PR? I believe Marcin addressed your previous comments. Thank you!
/retest-required
/lgtm
@nicklesimba are you agreeing that https://github.com/k8snetworkplumbingwg/whereabouts/pull/317 is sufficient? If so, this PR could be closed I think.
Tested this PR, seems to fix the issue with the EgressIP being added to both OVN primary network interface and secondary host network interface. Created EgressIP got assigned to expected...