syntasso / helm-charts

Helm charts for Kratix
Apache License 2.0
0 stars 2 forks source link

feat: add authMethod to kratix chart #7

Closed Knapsack44 closed 2 months ago

Knapsack44 commented 2 months ago

This PR will add the possibility to define the authMethod for the Kratix helm chart. If no value is defined, the value is automatically set to basicAuth. With my change, this will also be explicit visible.

CLAassistant commented 2 months ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Markus Feilen seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

ChunyiLyu commented 2 months ago

👋 @Knapsack44 thanks for your contribution! Could you please take a look and sign the CLA?

Knapsack44 commented 2 months ago

Sry, it seems to be a bit messy on my site. Please see https://github.com/syntasso/helm-charts/pull/8