Migrate L2 announcements and LB-IPAM to cilium #505
Conversation
No need to use MetalLB since Cilium can do this as well.
There was a problem hiding this comment.
+@MarcusSorealheis +@adam-singer +@allada
Reviewable status: 0 of 3 files reviewed, all discussions resolved (waiting on @adam-singer, @allada, and @MarcusSorealheis)
There was a problem hiding this comment.
Reviewable status: 0 of 3 files reviewed, 2 unresolved discussions (waiting on @adam-singer, @allada, and @MarcusSorealheis)
deployment-examples/kubernetes/00_infra.sh line 104 at r1 (raw file):
Previously, MarcusSorealheis (Marcus Eagan) wrote…
This is less stable not more stable, right? I think it's fine, but we need to ensure that neither our CI our future releases are impeded by an Alpha announcement policy.
From what I've tested it amounts to the same stability as in "just works", but the cilium variant is much faster to set up (e.g. in CI) since we don't need to fetch/deploy another helm chart.
deployment-examples/kubernetes/00_infra.sh line 117 at r1 (raw file):
Previously, MarcusSorealheis (Marcus Eagan) wrote…
Replacing MetalLb with Cilium is obviously an improvement.
Done.
No need to use MetalLB since Cilium can do this as well.
This change is