techno-tim / k3s-ansible

The easiest way to bootstrap a self-hosted High Availability Kubernetes cluster. A fully automated HA k3s etcd install with kube-vip, MetalLB, and more. Build. Destroy. Repeat.
https://technotim.live/posts/k3s-etcd-ansible/
Apache License 2.0
2.41k stars 1.05k forks source link

Unreferenced Variable metal_lb_frr_tag_version #256

Closed timothystewart6 closed 3 months ago

timothystewart6 commented 1 year ago
          Maybe I am missing something or something was overlooked when you created another PR but we added `metal_lb_frr_tag_version` but it looks like it isn't being referenced anywhere.  This tag does exist in the manifests but we aren't overriding it with regex.  Not sure if this is really needed or not.  https://raw.githubusercontent.com/metallb/metallb/v0.13.7/config/manifests/metallb-frr.yaml

Originally posted by @timothystewart6 in https://github.com/techno-tim/k3s-ansible/issues/212#issuecomment-1469245983

timothystewart6 commented 3 months ago

this is no longer valid