feat: support multi-chart deployment

[thomaspetit]

Description

All information is referenced in #161

As I would like to already incrementally fix this setup and ensure that Zitadel can easily be deployed the current proposal is to move these annotations:

helm.sh/hook: pre-install,pre-upgrade
helm.sh/hook-delete-policy: before-hook-creation
helm.sh/hook-weight: "1"

.. to the values.yaml as was basically already done for some resources already like the initJob values.yaml#134

Goal for the initial PR is to make a non-breaking change on the chart

Definition of Ready

• I am happy with the code
• Short description of the feature/issue is added in the pr description
• PR is linked to the corresponding user story
• Acceptance criteria are met
• All open todos and follow ups are defined in a new ticket and justified
• Deviations from the acceptance criteria and design are agreed with the PO and documented.
• No debug or dead code
• My code has no repetitions
• Documentation/examples are up-to-date
• All non-functional requirements are met
• If possible, the test configuration is adjusted so acceptance tests cover my changes

[thomaspetit]

@arnouthoebreckx as we've been discussing this offline, feel free to comment too 🚀

[eliobischof]

hey @thomaspetit thanks for this very useful feature! Please have a look at the small suggestion above.

If you'd like to have a small gift in return, please send us a mail to [email protected]. We will send you a form with questions about your address and shirt size.

[thomaspetit]

hey @thomaspetit thanks for this very useful feature! Please have a look at the small suggestion above.

If you'd like to have a small gift in return, please send us a mail to [email protected]. We will send you a form with questions about your address and shirt size.

I had a good reason to not put it in the values as appararntly dbSslRootCrt has been deprecated when the switch to v5 happened on Zitadel:

zitadel-charts/README.md

Line 52 in d4d9977

| `zitadel.dbSslRootCrt` | `zitadel.dbSslCaCrt` |

If I would move it back to the values.yaml it would mean that we re-introduce a dbSslRootCrt (which was removed): https://github.com/zitadel/zitadel-charts/blob/zitadel-5.0.0/charts/zitadel/values.yaml#L57 (ref to old code)

Let me know if you still want it in there and I'll do the necessary work 😁 @eliobischof

[eliobischof]

hey @thomaspetit thanks for this very useful feature! Please have a look at the small suggestion above.

If you'd like to have a small gift in return, please send us a mail to [email protected]. We will send you a form with questions about your address and shirt size.

I had a good reason to not put it in the values as appararntly dbSslRootCrt has been deprecated when the switch to v5 happened on Zitadel:

zitadel-charts/README.md

Line 52 in d4d9977

| `zitadel.dbSslRootCrt` | `zitadel.dbSslCaCrt` |

If I would move it back to the values.yaml it would mean that we re-introduce a dbSslRootCrt (which was removed): https://github.com/zitadel/zitadel-charts/blob/zitadel-5.0.0/charts/zitadel/values.yaml#L57 (ref to old code)

Let me know if you still want it in there and I'll do the necessary work 😁 @eliobischof

Makes perfect sense 😃