This repository has been archived by the owner on Feb 22, 2022. It is now read-only.
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.
Charts still appear in aggregator hubs if a deprecated version is not published before deleting source code #23128
Closed
Description
Problem
There are some charts still listed on the helm/artifact hub(s) that no longer exist in the incubator/stable repos source code.
Cause
I believe this is because the chart was moved to stable back in 2017 rather than deprecated in the incubator repo. For example, you can see incubator/cockroachdb
is listed in the hub(s) now, because the PR that moved/removed the chart did so before publishing a version where the chart was marked deprecated
(#344).
Proposed solution
- Search for all charts that were deleted before being properly deprecated, and update list below accordingly:
$ helm repo add stable https://kubernetes-charts.storage.googleapis.com $ helm repo add incubator https://kubernetes-charts-incubator.storage.googleapis.com $ helm search repo | awk '{ print $1 }' | xargs ls -d
- Re-add each deleted chart with a README pointing to the new location and mark it deprecated, so the old versions will no longer appear listed in the hub(s) (for reasoning, see https://helm.sh/docs/topics/charts/#deprecating-charts)
- For each missing chart, use helm 2 to create a generic v1 chart that will pass CI
$ file=incubator/cockroachdb $ helm2 create $file
- Find the last version before the missing chart was removed (example, "incubator/cockroachdb"), so we can bump the version
$ helm search repo $file | awk '{ print $2 }' 0.1.1
- Make required deprecation changes (per https://github.com/helm/charts/blob/master/PROCESSES.md#deprecating-a-chart)
- Open a PR and link to this tracking issue
- Once merged, wait 30 mins and ensure the chart is no longer listed in the hub(s)
- For each missing chart, use helm 2 to create a generic v1 chart that will pass CI
Status
- incubator/cockroachdb: [incubator/cockroachdb] deprecate missing chart #23129
- incubator/consul: [incubator/consul] deprecate missing chart #23220
- incubator/docker-registry: [incubator/docker-registry] deprecate missing chart #23222
- incubator/drone: [incubator/drone] deprecate missing chart #23223
- incubator/gocd: [incubator/gocd] deprecate missing chart #23224
- incubator/grafana: [incubator/grafana] deprecate missing chart #23225
- incubator/kube-ops-view: [incubator/kube-ops-view] deprecate missing chart #23226
- incubator/kube-spot-termination-notice-handler: [incubator/kube-spot-termination-notice-handler] deprecate missing chart #23227
- incubator/mongodb: [incubator/mongodb] deprecate missing chart #23229
- incubator/mongodb-replicaset: [incubator/mongodb-replicaset] deprecate missing chart #23230
- incubator/neo4j: [incubator/neo4j] deprecate missing chart #23231
- incubator/oauth-proxy: [incubator/oauth-proxy] deprecate missing chart #23234
- incubator/prometheus: [incubator/prometheus] deprecate missing chart #23235
- incubator/spark: [incubator/spark] deprecate missing chart #23236
- stable/falco: [stable/falco] deprecate missing chart #23237
- stable/sugarcrm: This was removed by [stable/sugarcrm ] - Remove deprecated chart #5670 after publishing a deprecated version, so it's not necessary to re-add here.
Metadata
Assignees
Labels
No labels