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

Use sample inventory for Vagrant #60

Closed sleiner closed 2 years ago

sleiner commented 2 years ago

Proposed Changes

Previously, Vagrantfile redefined all group and host variables. So changes to inventory/sample were not tested in CI.
This PR uses inventory/sample for Vagrant.

Checklist

[x] Tested locally
[x] Ran site.yml playbook
[ ] Ran reset.yml playbook
[x] Did not add any unnecessary changes
[x] 🚀

twistedgrim commented 2 years ago

Why not use Molecule? https://molecule.readthedocs.io/en/latest/

timothystewart6 commented 2 years ago

side note, it seems the the CI tasks still fails often but I think this is another race condition and I will try to fix it

sleiner commented 2 years ago

side note, it seems the the CI tasks still fails often but I think this is another race condition and I will try to fix it

Nice, thank you! 😊 And now with #57 being merged into master, the cache should save the need of downloading boxes from HashiCorp - so no more 429s (as long as we stick to the same image) 🙌

sleiner commented 2 years ago

Concerning molecule: I'm still experimenting with it. However, if it works, I will open a separate PR for it. The core part of this PR are the changes to the host vars, which will be needed to use the sample inventory in CI - regardless of whether we use molecule or Vagrant directly.

sleiner commented 2 years ago

closing this in favor of #67...