search

elastic / cloud-on-k8s

Elastic Cloud on Kubernetes
Other
2.52k stars 686 forks source link

moves the annotation eck.k8s.elastic.co/license to values.yaml #7912

Closed michbeck100 closed 1 week ago

michbeck100 commented 2 weeks ago

Moves the annotation eck.k8s.elastic.co/license to values.yaml to make it configurable. This fixes #6261.

thbkrkr commented 2 weeks ago

Hi, what value do you want to configure for this annotation and why?

michbeck100 commented 2 weeks ago

I would like to override this annotation with

eck.k8s.elastic.co/license: enterprise
thbkrkr commented 1 week ago

Could you explain why do you need this? This annotation is to allow to request a certain level of license to be authorized to use certain ECK custom resources. It should no longer be necessary since ECK 2.12.0 because all of the ECK's custom resources, including when using Helm charts, are distributed under the basic license.

michbeck100 commented 1 week ago

din't know that, sorry. Then this MR is no longer needed. Closing