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

Added pre_task to verify minimum Ansible version #454

Closed AngelGarzaDev closed 8 months ago

AngelGarzaDev commented 8 months ago

Proposed Changes

Added a pre_task to site.yml to verify the minimum version of Ansible (2.11).

Note: playbook syntax requires a non-empty "hosts" line, but the version check happens in the local machine (not the hosts in the host.ini).

-

Checklist

[X] Tested locally
[X] Ran site.yml playbook
[X] Ran reset.yml playbook
[X] Did not add any unnecessary changes
[X] Ran pre-commit install at least once before committing
[X] 🚀

timothystewart6 commented 8 months ago

this closes https://github.com/techno-tim/k3s-ansible/issues/251

thank you!

timothystewart6 commented 8 months ago

Looks like there's some lint. See https://github.com/techno-tim/k3s-ansible?tab=readme-ov-file#pre-commit-hooks