Skip to content

Removing the stable repository#6095

Merged
mattfarina merged 1 commit intohelm:dev-v3from
mattfarina:remove-stable
Jul 29, 2019
Merged

Removing the stable repository#6095
mattfarina merged 1 commit intohelm:dev-v3from
mattfarina:remove-stable

Conversation

@mattfarina
Copy link
Collaborator

The stable repository provides a quick on-boarding with a set of
community curated charts. Two problems with the community stable
repository has lead to its need to be removed.

  1. The URL is hard coded to a Google Cloud bucket under Google's
    control. This was setup when Helm was part of Kubernetes and
    Kubernetes was a Google project. The bucket cannot be
    transferred to another non-Google controlled project. And,
    the bucket is not accessible in some parts of the world
    (e.g., China).
  2. The number of charts in the stable repository has grown
    generally unmaintainable. The repository maintainers cannot
    manage the number of PRs coming it cauing delays in response
    or no response and PRs are automatically closed. This is
    a poor experience.

The alternative is the Helm Hub that provides a central point of
search for many Helm repositories. Different people and organizations
can maintain their own charts. A central server is not needed as
Helm is setup to be distributed.

cc @helm/charts-maintainers

The stable repository provides a quick onboarding with a set of
community curated charts. Two problems with the community stable
repository has lead to its need to be removed.

1. The URL is hard coded to a Google Cloud bucket under Google's
   control. This was setup when Helm was part of Kubernetes and
   Kubernetes was a Google project. The bucket cannot be
   transfered to another non-Google controlled project. And,
   the bucket is not accessible in some parts of the world
   (e.g., China).
2. The number of charts in the stable repository has grown
   generally unmaintainable. The repository maintainers cannot
   manage the number of PRs coming it cauing delays in response
   or no response and PRs are automatically closed. This is
   a poor experience.

The alternatice is the Helm Hub that provides a central point of
search for many Helm repositories. Different people and organizations
can maintain their own charts. A central server is not needed as
Helm is setup to be distributed.

Signed-off-by: Matt Farina <[email protected]>
@helm-bot helm-bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jul 26, 2019
@mattfarina mattfarina added the v3.x Issues and Pull Requests related to the major version v3 label Jul 26, 2019
@maorfr
Copy link
Member

maorfr commented Jul 26, 2019

This looks good to me. Do we already have a deprecation plan for the charts repository? Is there a doc maybe?

Copy link
Member

@bacongobbler bacongobbler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@bacongobbler
Copy link
Member

The v3 docs may need an update as well since the stable repository is a commonly used example. Would you be willing to update helm-www as well, or would you mind filing a ticket for that?

@mattfarina
Copy link
Collaborator Author

@maorfr When helm v2 is deprecated at the earliest. That timeline is up for discussing in the next Helm Dev Call.

@bacongobbler I noticed that. I wanted one PR to land at a time. I have a few in flight. I'll get to that unless someone else wants to.

Copy link
Contributor

@hickeyma hickeyma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks.

@norrs
Copy link

norrs commented Aug 20, 2019

What happens with the existing charts that A LOT of users depends on? Will they be migrated to Helm Hub? Where is Helm Hub? ( https://hub.helm.sh/ I assume? )

Please make follow up issues for chart owners to migrate their charts before you remove anything please, to avoid chaos and angry helm users.

cc @helm/charts-maintainers

@prydonius
Copy link
Member

@norrs this just removes stable as a default repository, it can still be added manually with helm repo add stable https://kubernetes-charts.storage.googleapis.com

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M Denotes a PR that changes 30-99 lines, ignoring generated files. v3.x Issues and Pull Requests related to the major version v3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants