lwolf / kube-cleanup-operator

Kubernetes Operator to automatically delete completed Jobs and their Pods
MIT License
498 stars 109 forks source link

helm chart #60

Closed dududko closed 3 years ago

dududko commented 3 years ago

Is there the official helm chart for this operator ?

lwolf commented 3 years ago

No, the chart felt like overkill for this simple thing. Do you think it needs one?

dududko commented 3 years ago

@lwolf I believe it is good to pack it into a helm chart. Even with the small project like this, it would be very convenient to have a helm chart with a specified version and the list of configurable parameters. It will also be easier to rollout changes if the list of kubernetes objects will change (for example if you will decide to use a statefulset instead of deployment). Also by making a helm chart and by uploading it to the helm hub the kube-cleanup-operator project will attract more users/contributors.

lwolf commented 3 years ago

Thanks for the suggestion, I'll keep it in mind for the next release.

qbasicer commented 3 years ago

+1 for a helm chart specifically to parameterize the namespace config

haarchri commented 3 years ago

added a helm chart for kube-cleanup-operator - please have a look if its fits ;)

lwolf commented 3 years ago

thanks for the PR, will take a look over weekend

lwolf commented 3 years ago

Thanks @haarchri. PR is merged, I'll close the issue after I publish it to the registry.

pranas commented 3 years ago

looking forward to the Helm chart

lwolf commented 3 years ago

Chart is published to https://charts.lwolf.org/ (https://github.com/lwolf/kube-charts)

$ helm repo add lwolf-charts http://charts.lwolf.org
"lwolf-charts" has been added to your repositories
$ helm search kube-cleanup
NAME                                CHART VERSION   APP VERSION DESCRIPTION
lwolf-charts/kube-cleanup-operator  1.0.0           v0.8.1      Kubernetes Operator to automatically delete completed Job...