Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove helm chart #15904

Merged
merged 1 commit into from
Feb 21, 2024
Merged

Remove helm chart #15904

merged 1 commit into from
Feb 21, 2024

Conversation

gianm
Copy link
Contributor

@gianm gianm commented Feb 14, 2024

The helm chart was originally moved here in #11163 from https://github.com/helm/charts/tree/master/incubator/druid after the helm/charts repository was deprecated. However, it has been excluded from releases since then, due to uncertainty around whether we need IP clearance. We have not had volunteers willing to sort this out, so this patch removes the code.

It can be re-added if a volunteer is available to sort out the IP clearance process (or to analyze the provenance of the code enough to determine that IP clearance isn't necessary).

See thread at: https://lists.apache.org/thread/ygyzt23m06vc775nq5dsm349rf0j47dg

The helm chart was originally moved here in apache#11163 from
https://github.com/helm/charts/tree/master/incubator/druid after the
helm/charts repository was deprecated. However, it has been excluded
from releases since then, due to uncertainty around whether we need
IP clearance. We have not had volunteers willing to sort this out,
so this patch removes the code.

It can be re-added if a volunteer is available to sort out the
IP clearance process.

See thread at: https://lists.apache.org/thread/ygyzt23m06vc775nq5dsm349rf0j47dg
@gianm
Copy link
Contributor Author

gianm commented Feb 14, 2024

FYI to folks who have contributed to this tree in the past year: @jwitko, @asdf2014, @zemin-piao, @jaegwonseo, @abhishekagarwal87. The linked thread has some details about why this tree hasn't been included in a release yet and why I'm proposing it be removed now.

The IP clearance process can be difficult and time-consuming to navigate, and this tree has so far been a casualty of that. Perhaps one of you would be interested in pursuing that (or helping determine that it isn't necessary) which would enable us to re-add the tree. Or perhaps you'd be interested in maintaining this code outside of ASF repos, and we can link to it.

Sorry to be the bearer of such news, if you weren't already aware of this issue.

Copy link
Member

@asdf2014 asdf2014 left a comment

Choose a reason for hiding this comment

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

👍 SGTM, I just created a code repository to maintain these codes. Here is the address of the new repository: https://github.com/asdf2014/druid-helm. Later, I can help update docs related to the Helm Chart and notify the contributors of the relevant PRs to migrate to this new repository.

@maver1ck
Copy link
Contributor

maver1ck commented Jun 25, 2024

Hi @gianm and @asdf2014
I wasn't using Druid for a while but I was original creator of helm/charts Druid helm chart.
How can I help you here ? Do you need any support ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Helm Chart https://github.com/apache/druid/tree/master/helm/druid
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants