maorfr commented 5 years ago

Hello,

As maintaining this repository is growing harder by the day, this is a call for maintainers!

This issue is about 'recruiting' maintainers for charts that have no OWNERS file. The list of charts can be found at the end of this message.

If you wish to be added as a maintainer to one of the charts in this issue, please open a PR to add an OWNERS file, and add yourself to that OWNERS file. In addition, maintainers in Chart.yaml and OWNERS should match.

We would suggest to include in the PR references to past contributions to the chart you are applying for.

If you are a planning to apply for being a maintainer and know someone who also fits this (and is interested) - feel free to approach them and include them! we aim to have 2 maintainers per chart.

Once a PR exists, please add a comment referencing this issue, and ping one of the maintainers to get added as a read-only user to the helm/charts repository.

Thanks in advance to everyone who applies! We need your help.

All new OWNERS should be familiar with the following documents: https://github.com/helm/charts/blob/master/REVIEW_GUIDELINES.md https://github.com/helm/charts/blob/master/CONTRIBUTING.md https://helm.sh/docs/chart_best_practices/

Here is the list of charts:

~~incubator/artifactory~~ ~~incubator/burrow~~ incubator/check-mk incubator/common incubator/couchdb incubator/distribution ~~incubator/drone~~ incubator/etcd incubator/fluentd-cloudwatch ~~incubator/fluentd~~ incubator/goldfish ~~incubator/istio~~ ~~incubator/jaeger~~ incubator/jenkins-operator incubator/keycloak-proxy ~~incubator/keycloak~~ incubator/kube-janitor incubator/kube-registry-proxy incubator/kubernetes-vault ~~incubator/kubewatch~~ incubator/mysqlha ~~incubator/oauth-proxy~~ incubator/patroni incubator/puppet-forge incubator/riemann incubator/spring-cloud-data-flow incubator/tensorflow-inception incubator/vault incubator/vaultingkube incubator/webpagetest-agent incubator/webpagetest-server incubator/xray incubator/orientdb incubator/solr ~~stable/acs-engine-autoscaler~~ ~~stable/aerospike~~ (moved #13867) stable/ark ~~stable/aws-cluster-autoscaler~~ stable/aws-iam-authenticator stable/bitcoind stable/buildkite ~~stable/cert-manager~~ stable/cluster-autoscaler stable/consul stable/cosbench stable/coscale stable/dask-distributed stable/dask stable/distributed-tensorflow stable/ethereum stable/express-gateway stable/external-dns stable/factorio stable/gcloud-endpoints stable/gcp-night-king ~~stable/gitlab-ce~~ ~~stable/gitlab-ee~~ stable/hackmd stable/hadoop stable/heapster stable/heartbeat stable/hubot stable/ignite stable/ipfs stable/janusgraph stable/k8s-spot-termination-handler stable/kanister-operator ~~stable/kanister-operator/charts/profile~~ stable/kapacitor stable/katafygio stable/keel ~~stable/kiam~~ ~~stable/kube-lego~~ stable/kube-ops-view stable/kube-slack ~~stable/kube-state-metrics~~ (moved #13867) stable/kube2iam stable/kuberos stable/kured stable/lamp stable/linkerd stable/locust stable/luigi ~~stable/mailhog~~ (moved #13315) stable/mcrouter stable/metabase stable/metallb stable/minecraft ~~stable/mission-control/charts/elasticsearch~~ stable/msoms stable/namerd stable/neo4j stable/nginx-ldapauth-proxy stable/nginx-lego stable/openldap stable/percona stable/presto ~~stable/prometheus-adapter~~ stable/prometheus-consul-exporter stable/prometheus-couchdb-exporter stable/prometheus-mysql-exporter stable/prometheus-postgres-exporter stable/prometheus-rabbitmq-exporter stable/prometheus-redis-exporter stable/prometheus-to-sd stable/rethinkdb stable/risk-advisor stable/rookout stable/sapho stable/sensu ~~stable/sentry~~ (moved #13867) ~~stable/signalfx-agent~~ ~~stable/spark-history-server/charts/nfs~~ stable/spark stable/spartakus stable/spotify-docker-gc ~~stable/spring-cloud-data-flow~~ stable/stackdriver-exporter stable/stolon stable/sumokube stable/superset stable/tomcat stable/uchiwa stable/unbound stable/verdaccio ~~stable/weave-scope~~ stable/zeppelin stable/zetcd stable/logdna-agent stable/socat-tunneller ~~stable/velero~~

okgolove commented 5 years ago

@maorfr stable/aws-cluster-autoscaler has migrated to helm hub https://hub.helm.sh/charts/jetstack/cert-manager

srueg commented 5 years ago

incubator/keycloak is deprecated and moved to codecentric/keycloak

monotek commented 5 years ago

@unguiculus told me once, that incubator repo should / will die in the future, as unfinished or untested charts should not be published at all, so imho these charts should rather be deleted or moved to stable with a version bump, instead of looking for a new maintainer.

Maybe it's also an idea to find people to be in the Github Helm organization / or in the root OWNERS file (https://github.com/helm/charts/blob/master/OWNERS) so they can help out everywhere without to be bound to a specific chart.

unguiculus commented 5 years ago

@monotek Well, it's not gonna die. I just prefer not to add new stuff there. IMO, we should try and promote existing charts to stable or deprecate them.

zakkg3 commented 5 years ago

incubator/drone have a stable/drone working version

tariq1890 commented 5 years ago

stable/acs-engine-autoscaler is deprecated. stable/kube-state-metrics can be struck out.

dperetti commented 5 years ago

What makes maintaining the repository difficult is also probably related to the fact that all the projects and their issues are mixed there. As a quite recent Helm user, I really dislike this!

desaintmartin commented 5 years ago

incubator/fluentd is deprecated (https://github.com/helm/charts/pull/8477)

Comradin commented 5 years ago

incubator/istio is deprecated according to their README.md users are encouraged to use https://github.com/istio/istio/tree/master/install/kubernetes/helm/istio from their repository instead.

jeff-knurek commented 5 years ago

incubator/oauth-proxy & stable/kube-lego are deprecated - I'd be inclined to think that several other of the charts in this list are already deprecated too

desaintmartin commented 5 years ago

stable/gitlab-ce and stable/gitlab-ee are deprecated as well.

Comradin commented 5 years ago

compleatang and k8s-ci-robot move burrow to stable repo (#3481) Latest commit 0abd574 on 23 Mar 2018

I guess it should be deleted from the incubator path, then? And it can be removed from your list.

naseemkullah commented 5 years ago

PR to add OWNERS to incubator/jaeger https://github.com/helm/charts/pull/13693

EricLemieux commented 5 years ago

stable/signalfx-agent was deprecated in #13586

EricLemieux commented 5 years ago

@maorfr I made a pretty basic script to get the current number of maintainers, so you don't need to rely on comments letting you know whenever someone is added/removed.

This is a sample out the output displaying the number of maintainers, approvers, and reviewers per chart.

https://github.com/EricLemieux/Helm-Chart-Maintainers/blob/master/current-number-of-maintainers.md

deepaksood619 commented 5 years ago

PR to add OWNERS to stable/kapacitor #14622

koolamusic commented 5 years ago

@maorfr any progress with stable/lamp I would like to join the maintainers for that project

monotek commented 5 years ago

added myself to prometheus-mysql-exporter: https://github.com/helm/charts/pull/14776

deepaksood619 commented 5 years ago

incubator/kubewatch is deprecated and moved to stable/kubewatch

Hashfyre commented 5 years ago

@cristinanegrean, it'd be great if you could maintain stable/stackdriver-exporter, I could pitch in too.

At work I mostly end up using your version: https://github.com/cristinanegrean/charts/tree/master/stable/stackdriver-exporter

Also, great blog post: https://blog.trifork.com/2019/02/07/centralized-metrics-with-stackdriver-prometheus-exporter/

stale[bot] commented 5 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Any further update will cause the issue/pull request to no longer be considered stale. Thank you for your contributions.