Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add several configuration options #15

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tiagoblackcode
Copy link
Contributor

@tiagoblackcode tiagoblackcode commented Feb 27, 2022

The new configurable options are:

  • cni.chainingMode via cilium_chaining_mode
  • ipam.mode via cilium_ipam_mode
  • ipam.clusterPoolIPv4PodCIDR via cilium_ipam_cluster_pool_pod_cidr when ipam.mode is cluster-pool
  • bgp to enable BGP advertisement
  • kubeProxyReplacement via cilium_kube_proxy_replacement

BGP advertisement uses the cilium_bgp_config to render the configuration supported by Cilium's MetalLB
integration.

The kubeProxyReplacement defaults to probe despite Cilium's Helm chart suggesting it's disabled.

@tiagoblackcode
Copy link
Contributor Author

I've been fiddling with Cilium configuration options to get its MetalLB integration working and there were several customisations I made to the Helm values template. So, I added them to the repository in case anybody wants to customize the "basic" helm values without having to provide a custom cilium_values_user.j2 template.

It also adds support to enable Cilium's MetalLB integration.

The default values use the same defaults that are being used at the moment by the Ansible role to avoid breaking existing playbooks that might update to a new version.

The new configurable options are:
  - `cni.chainingMode` via `cilium_chaining_mode`
  - `ipam.mode` via `cilium_ipam_mode`
  - `ipam.clusterPoolIPv4PodCIDR` via `cilium_ipam_cluster_pool_pod_cidr` when `ipam.mode` is `cluster-pool`
  - `bgp` to enable BGP advertisement
  - `kubeProxyReplacement` via `cilium_kube_proxy_replacement`

BGP advertisement uses the `cilium_bgp_config` to render the configuration supported by Cilium's MetalLB
integration.

The `kubeProxyReplacement` defaults to `probe` despite Cilium's Helm chart suggesting it's `disabled`.
Repository owner deleted a comment from floatingdev Jan 31, 2024
Repository owner deleted a comment from 19890843006 Feb 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant