remove support for `ELASTIC_APM_KUBERNETES_*` envvars

Open trentm opened this issue 3 years ago • 0 comments

We should consider deprecating and removing the old environment variables in the next major.

from the description of https://github.com/elastic/apm-agent-nodejs/pull/1041

Way back when https://github.com/elastic/apm/issues/21 was implemented for the Node.js APM agent, the envvar configuration support accidentally used the ELASTIC_APM_ prefix for the new Kubernetes environment variable support:

KUBERNETES_NODE_NAME | system.kubernetes.node.name KUBERNETES_POD_NAME | system.kubernetes.pod.name KUBERNETES_NAMESPACE | system.kubernetes.namespace KUBERNETES_POD_UID | system.kubernetes.pod.uid

Then in #1041 support was added for the unprefixed envvar names, as originally intended. At the time the prefixed-envvars were kept for backward compat.

Apr 27 '22 16:04 trentm