rancherfederal / rke2-ansible

RKE2 cluster provisioning via Ansible.
Apache License 2.0
228 stars 129 forks source link

#213 Render templated variables within rke2_config prior to updating #214

Closed onceinarow closed 6 months ago

onceinarow commented 6 months ago

What type of PR is this?

(REQUIRED)

What this PR does / why we need it:

This PR address #213 - specifically, any templated variables within the rke2_config variable will not be rendered correctly when written to the config.yaml file. Based on the known issue when using update_fact, the best solution is to use set_fact to render all templated variables.

Which issue(s) this PR fixes:

Fixes #213

Testing

I performed two tests, each with the following rke2_config:

selinux: true
rke2_config:
  selinux: "{{ selinux_enabled }}"

Release Notes

None

mddamato commented 6 months ago

215