-
Notifications
You must be signed in to change notification settings - Fork 518
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
Managing the Helm Charts for APM Server #7129
Comments
@kseniia-kolpakova could we get some support from the automation team for moving the helm charts to the APM Server and adapting the existing e2e tests for APM Server helm charts (if they need to be adapted). |
Closing this, as suggested in https://github.com/elastic/observability-robots/issues/1002#issuecomment-1029315151 the helm charts have not been GA; and apm-server should be part of elastic-agent helm charts moving forward. |
👋🏻 @simitt, as discussed last week, we'll create a PR to remove apm-server from the main branch on https://github.com/elastic/helm-charts. Do you have suggestions for the communication we'll have to do explaining why we are removing it when we'll create the public PR? cc @nkammah |
@chrisdistasio wanted to follow up with you regarding the wording. |
That's a good question. There is no Elastic Agent chart released on our Helm repository ATM. I know that there is a work in progress in elastic/beats#22572 but I don't know the status, cc @ph @kuisathaverat). |
i see these two PRs as well. elastic/beats#28102, elastic/beats#28103. i'm a bit uncomfortable with removal/deprecation as it's unclear to me that we have a solid or agreed-upon replacement, but the fact that it is beta creates some space. @jmlrt do you have a drop dead date for language/public pr? |
The deadline to remove the APM chart or transfer its ownership to the APM team is the date of the first 8.x Helm chart release which is now planned for 8.3.0 (https://github.com/elastic/dev/issues/1930#issuecomment-1088514502). However, the sooner is the better because if you decide to not remove the chart, there may be some work required to make it compatible with 8.x APM Server versions. |
more details to follow
The text was updated successfully, but these errors were encountered: