fix(chart): fully remove parts of non-global redis

[olevski]

So these call-nested helpers were meant (through some witchery) to call the template of a requirement/child helm chart and pull out the redis release name which is then used in the label to allow client applications to connect to redis.

Now since we rely on a global redis instance we should not use this anymore. There is no local redis anymore.

This was causing the helm chart building/publishing step to fail: https://github.com/SwissDataScienceCenter/renku-gateway/runs/5108179637?check_suite_focus=true

/deploy #persist

[olevski]

In addition to the issue with the values extracted from child charts, I also did the following:

• cleaned up the messy defaults/required stuff around the secrets, now the secrets have dummy defaults with comments that indicate that they should be set to proper values for prod
• enabled the tests for the helm chart for every PR

[olevski]

... also we should probably not use helm v2 to test the chart.

[Panaetius]

thank you!

[ableuler]

Thanks for taking the initiative of cleaning things up. There's still a few things that we need to figure out though.

[ableuler]

... also we should probably not use helm v2 to test the chart.

🙈

[RenkuBot]

You can access the deployment of this PR at https://renku-ci-gw-537.dev.renku.ch

[olevski]

@ableuler this is ready now.