Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

Adding deprecation notice to repo #24417

Merged
merged 1 commit into from
Nov 13, 2020
Merged

Conversation

mattfarina
Copy link
Contributor

No description provided.

@helm-bot helm-bot added Contribution Allowed If the contributor has signed the DCO or the CNCF CLA (prior to the move to a DCO). size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 13, 2020
Copy link
Member

@scottrigby scottrigby left a comment

Choose a reason for hiding this comment

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

👍

@scottrigby
Copy link
Member

See #23944

@mattfarina mattfarina merged commit df36ceb into helm:master Nov 13, 2020
@mattfarina mattfarina deleted the obselete-notice branch November 13, 2020 22:48
@sfernandez
Copy link

sfernandez commented Nov 23, 2020

i'm astonished with the little time of migration do you deliver to applications using this chart.
a 6 month (6 MONTH!) time-to-migrate-or-die with a global pandemic in place it's not sufficient.

The most annoying is the part where the chart not only it's deprecated, but it is removed from the repository. I could expect this kind of lifetime for an angular library. For an infrastructural service like a loadbalancer i will expect a broader lifetime, retrocompatibility a migration guide for the "new thing",etc..

Thank you guys for the systems i have to quick fix now.

@scottrigby
Copy link
Member

scottrigby commented Dec 22, 2020

@sfernandez I hope since this comment you were able to resolve your systems. The charts repo maintainers and the entire Helm team have worked hard to ensure end users will have to make only minimal changes to keep their systems running:

  • There was a 1 year timeline, corresponding to match the Helm 2 deprecation announcement and timeline. The initial notice was added to this repo's README to clarify this in November 2019
  • There was a period of time where we were unsure where charts package history would live. As it turns out, we collaborated with GitHub who generously offered to sponsor hosting these indefinitely
  • I'm sure there are things we could have done better. However, if you want to see the step-by-step action plan to ensure the best experience possible for end users, including how to ensure charts will continue to be available to running systems with only minimal changes (the repo URL), see [stable/incubator] relocate package history #23850, Charts Repo Deprecation Plan and Timeline #23944, and the linked issues/PRs
  • For a summary of the ongoing initiative to relocate charts to new repos, see: https://github.com/helm/charts/issues/21103#issuecomment-732992002. Chart relocation is not complete because maintaining charts has always been a community effort. Deprecated charts which have not yet relocated can always be adopted and un-deprecated. If you're interested in helping that effort, that issue is the entry point for – and place to track – participation and status.

To summarize that last comment link, Helm community charts development has not ended, it is simply bigger than ever before and has required moving to new, distributed hosting, discovery, and tooling.

For additional history and information, these blog posts are good places to start:

@sfernandez
Copy link

Hi @scottrigby ,

1 year announcement it is still insufficient to migrate an infrastructural service, also not providing a fallback/or an upgrade manual in the same notice. This worsens because of the COVID situation, in this context you should had given more time to people.

I will still use Helm because it's an amazing tool. But things could be done better.

Thank you for your time answering and your invitation to participate, i will have a look.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Contribution Allowed If the contributor has signed the DCO or the CNCF CLA (prior to the move to a DCO). size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants