Unify nodeSelector and env variables

[guidoiaquinti]

Description

• make sure deployments & migrate job have the option to configure affinity, tolerations and nodeSelector
• rename .Values.hooks.migrate to .Values.migration

Type of change

• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.

Checklist

• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• I have added tests that prove my fix is effective or that my feature works

[klamas1]

This pull request breaks the backward compatibility of values.
Can we start by simply adding tolerations and nodeSelector to migrations?

[guidoiaquinti]

This pull request breaks the backward compatibility of values. Can we start by simply adding tolerations and nodeSelector to migrations?

This is an old PR that we still might want to merge that is supposed to be a breaking change for the chart.

[klamas1]

Then maybe make a separate PR with only tolerations and nodeSelector to migrations ?
I need these options now, but doing it only in my fork is not convenient

[guidoiaquinti]

Then maybe make a separate PR with only tolerations and nodeSelector to migrations ? I need these options now, but doing it only in my fork is not convenient

Feel free to open another PR and we'll be happy to review it

[klamas1]

Thanks, i did it #564

[klamas1]

I created a new PR, in which I added the ability to globally specify affinity, tolerations and nodeSelector for all deployments, of which there are already quite a lot.

#620
Please check it out, maybe it will replace the current PR