azimuth-cloud / azimuth-caas-operator

K8s operator to create ansible based clusters using K8s CRDs
Apache License 2.0
1 stars 2 forks source link

Allow cluster spec to set ansible runner envvars #75

Closed sjpb closed 10 months ago

sjpb commented 10 months ago

Provides new optional parameter envVars to ClusterTypeSpec. This is a dict containing environment variables for ansible-runner runs.

Note that if envVars contains ANSIBLE_INVENTORY then the default runner inventory defining openstack as localhost is not created (as if it is ansible-runner ignores ANSIBLE_INVENTORY).

sjpb commented 10 months ago

@JohnGarbutt this is ready for review - it's failing lint on line lengths, on a line I've changed, but there's other unchanged lines in the same file which also exceed 88 chars. Do you want me to fix all of those as part of this PR?? Don't understand how they've got through CI before.