search

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

Expand secure_path with support for Suse #381

Closed balazshasprai closed 1 year ago

balazshasprai commented 1 year ago

Proposed Changes

Expanded the prereq role's secure_path task with support for Suse. Added a default var (secure_path) for that task too, with separate values for RedHat and Suse. (Didn't want to override Suse's default path order, nor RedHat's, I'm sure someone smarter than me decided on those orders for a reason) Tested both site and reset.yml. (Everything worked perfectly for openSUSE Leap 15.5 and SLES 15 SP5)

Checklist

balazshasprai commented 1 year ago

Hey, not sure what happened with that test, "Failed to start the VM" seems like maybe the runner had some issues? I've tried running these checks on my fork, it completed fine. I think a retry would end up being successful?

timothystewart6 commented 1 year ago

No worries! Yeah, it happens from time to time. I just kicked it off again!