Skip to content

Removes Druid chart deprecation message and flag#11897

Merged
asdf2014 merged 1 commit intoapache:masterfrom
isandeep41:rem_helm_deprecation_notice
Nov 13, 2021
Merged

Removes Druid chart deprecation message and flag#11897
asdf2014 merged 1 commit intoapache:masterfrom
isandeep41:rem_helm_deprecation_notice

Conversation

@isandeep41
Copy link
Contributor

The Druid Chart was deprecated in helm/charts repo

⚠️ Repo Archive Notice
As of Nov 13, 2020, charts in this repo will no longer be updated. For more information, see the Helm Charts Deprecation and Archive Notice, and Update.

Deprecation message and flag can be removed as Druid chart will be supported in Druid repo.


This PR has:

  • been self-reviewed.
  • added documentation for new or modified features or behaviors.
  • added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • added or updated version, license, or notice information in licenses.yaml
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • added integration tests.
  • been tested in a test Druid cluster.

@a2l007 a2l007 added the Helm Chart https://github.com/apache/druid/tree/master/helm/druid label Nov 9, 2021
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.

LGTM 👍

@asdf2014 asdf2014 merged commit 400e90d into apache:master Nov 13, 2021
@abhishekagarwal87 abhishekagarwal87 added this to the 0.23.0 milestone May 11, 2022
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