Helm-charts: Deprecate stable prometheus charts

Created on 26 Aug 2020  ·  12Comments  ·  Source: prometheus-community/helm-charts

After #11 and #34

Note these PRs must be done properly, per https://github.com/helm/charts/blob/master/PROCESSES.md#deprecating-a-chart

Most helpful comment

Good job deprecating all charts! Adding this for documentation purpose:

| chart | PR |
| --- | --- |
prometheus-adapter | https://github.com/helm/charts/pull/23694
prometheus-blackbox-exporter | https://github.com/helm/charts/pull/23695
prometheus-cloudwatch-exporter | https://github.com/helm/charts/pull/23696
prometheus-consul-exporter | https://github.com/helm/charts/pull/23697
prometheus-couchdb-exporter | https://github.com/helm/charts/pull/23698
prometheus-mongodb-exporter | https://github.com/helm/charts/pull/23699
prometheus-mysql-exporter | https://github.com/helm/charts/pull/23700
prometheus-nats-exporter | https://github.com/helm/charts/pull/23701
prometheus-node-exporter | https://github.com/helm/charts/pull/23702
prometheus-operator
prometheus-postgres-exporter | https://github.com/helm/charts/pull/23703
prometheus-pushgateway | https://github.com/helm/charts/pull/23704
prometheus-rabbitmq-exporter | https://github.com/helm/charts/pull/23705
prometheus-redis-exporter | https://github.com/helm/charts/pull/23706
prometheus-snmp-exporter | https://github.com/helm/charts/pull/23708
prometheus-to-sd | https://github.com/helm/charts/pull/23707
prometheus | https://github.com/helm/charts/pull/23692

All 12 comments

@scottrigby does this mean, all the existing chart maintainers need to do a PR for helm/stable ?

@gkarthiks Yes, we do.

I will raise a PR for deprecating the stable charts for all the 17 tomorrow for by myself if that's ok with everyone, since we #34 completed

It's fine for prometheus Helm chart.

34 is done, so we're good to go for opening these PR. Feel free to ping me in the PRs and I'll review/merge if other chart maintainers don't get to them first 🏃

@gkarthiks I somehow missed your reply - that would be great!

@naseemkullah ended up submitting PRs tonight

@scottrigby no worries. I see the PR https://github.com/helm/charts/pull/23693 only for prometheus-operator. Will do the rest then.

@scottrigby My bad...!!! I didn't see the merged PRs. That's awesome @naseemkullah

Good job deprecating all charts! Adding this for documentation purpose:

| chart | PR |
| --- | --- |
prometheus-adapter | https://github.com/helm/charts/pull/23694
prometheus-blackbox-exporter | https://github.com/helm/charts/pull/23695
prometheus-cloudwatch-exporter | https://github.com/helm/charts/pull/23696
prometheus-consul-exporter | https://github.com/helm/charts/pull/23697
prometheus-couchdb-exporter | https://github.com/helm/charts/pull/23698
prometheus-mongodb-exporter | https://github.com/helm/charts/pull/23699
prometheus-mysql-exporter | https://github.com/helm/charts/pull/23700
prometheus-nats-exporter | https://github.com/helm/charts/pull/23701
prometheus-node-exporter | https://github.com/helm/charts/pull/23702
prometheus-operator
prometheus-postgres-exporter | https://github.com/helm/charts/pull/23703
prometheus-pushgateway | https://github.com/helm/charts/pull/23704
prometheus-rabbitmq-exporter | https://github.com/helm/charts/pull/23705
prometheus-redis-exporter | https://github.com/helm/charts/pull/23706
prometheus-snmp-exporter | https://github.com/helm/charts/pull/23708
prometheus-to-sd | https://github.com/helm/charts/pull/23707
prometheus | https://github.com/helm/charts/pull/23692

https://github.com/helm/charts/issues/21103 updated

Let's leave this open until #1 is complete

Finished the last one: https://github.com/helm/charts/pull/23738

Pending prow test-infra issues from earlier. If it fails again I'll just force-merge it.

I have also updated https://github.com/helm/charts/issues/21103

Was this page helpful?
0 / 5 - 0 ratings