set defautStorageClass = nfs-client

[j0hnL]

this will close issue #81

Signed-off-by: John Lockman jlockman3@gmail.com

[lwilson]

Hey @j0hnL, looking at the implementation, it appears as though the kubectl patch command could have been inserted directly into a play using the shell module. Any particular reason for the standalone script?

Also, should we explore some of the more modern Ansible modules for kubernetes? I believe we can use the k8s module (https://docs.ansible.com/ansible/latest/modules/k8s_module.html) and the merge command.

[j0hnL]

it looks like the k8s module is specific to OpenShift

Uses the OpenShift Python client to perform CRUD operations on K8s objects.

Had problems using the shell module because of : in the command. Ansible barfs on that and thinks it's an operator.

[lwilson]

You should be able to escape the colon by surrounding the command in quotes:
shell: "kubectl patch storageclasses.storage.k8s.io nfs-client -p '{\"metadata\": {\"annotations\":{\"storageclass.kubernetes.io/is-default-class\":\"true\"}}}'"

see https://stackoverflow.com/questions/40936360/syntax-error-on-colon-in-ansible-command-argument

[lwilson]

I think this is a much better solution than the helper file. Approve and merged.