Allow also yaml configuration for nodeSelector, tolerations and affinities

[FWest98]

Similar to #272 (and somewhat also #213 ), it would be great to add support for using yaml instead of a string to configure the tolerations, affinity, and nodeSelector for the various components (injector, server). This would also be in line with how this is done in many other popular charts (which then ties in to #55 ).

[jasonodonnell]

I believe these are all supported, so closing this ticket. Feel free to reopen if you think I made an error!

[erks]

vault-helm/templates/_helpers.tpl

Line 216 in dcb4b10

{{ tpl .Values.server.affinity . | nindent 8 | trim }}

I don't see these fields supporting YAML yet on master. We still need to use string here. Can you reopen?

[FWest98]

I am not sure if I see what was changed in this. According to the docs all nodeSelector, tolerations, and affinities fields still require a multiline string instead of a yaml object. A quick look into the code reveals no clues that this has changed.

Could you maybe clarify what you changed to incorporate this?

[jasonodonnell]

My apologies, just misremembering. We added YAML support for a lot of other configurables, but missed these. We'll get these added.

[benashz]

Fixed in #565