Update Elasticsearch chart dependency to 8.5.1 #97
Conversation
The old version uses an outdated apiVersion for kind: PodDisruptionBudget, which breaks this chart for Kubernetes 1.21 and ongoing.
|
Hi, ElasticSearch 8.5.1 is not licensed by the Apache 2.0 anymore. That is why we hold the upgrade. |
|
It also breaks the CI |
Hi. We don't ship ES server/client codes here, we don't directly depends on the ES server (8.5.1 is the Helm Chart version), and, ES is not mandatory required (because we now support using a demo postgresql storage too). Is this still a problem? |
|
If it is an optional config, we should be fine. |
|
As the older Elasticsearch Helm template does not run on Kubernetes <= 1.21, I could not deploy Skywalking in our cluster at all. If the postgres backend is not recommended for production, I cannot use that either. The update to chart version 8.5.1 was a tryout of mine to get Skywalking running. I would appreciate it very much if you could provide a Skywalking version that can be installed on Kubernetes 1.25. |
|
At least, you could use ElasticSearch official helm to deploy it first, then use our helm to connect the existence ElasticSearch. |
|
@kezhenxu94 Could you take a look at the ElasticSearch deployment in our helm to upgrade? |
Thank you for the hint, I will give this a try. |
|
Closing in favor of #98 |
The old version uses an outdated apiVersion for kind: PodDisruptionBudget, which breaks this chart for Kubernetes 1.21 and ongoing.
The following error emerges when using the old Elastic Helm chart:
As you can see, the newer version of the Elastic Helm chart uses some logic to pass the correct apiVersion: https://github.com/elastic/helm-charts/blob/v8.5.1/elasticsearch/templates/poddisruptionbudget.yaml
Kind regards
Franz