Add AWS ECS/Fargate to container-info Parsing #2914
Comments
github-actions
bot
added
the
agent-nodejs
trentm
changed the title
|
When implementing this, we might also want to add testing based on the shared "json-specs/cgroup_parsing.json" added in elastic/apm#686 |
|
That shared spec file is being sync'd in in #2949 |
astorm
added a commit
to elastic/apm-nodejs-http-client
that referenced
this issue
Oct 18, 2022
* feat: Pull container-info functionality into local module elastic/apm-agent-nodejs#2914
|
Re-opened. This is only resolved when we update to a new release of 'elastic-apm-http-client' with the above commit. |
trentm
added a commit
that referenced
this issue
Oct 27, 2022
…nfig re-fetch interval Update elastic-apm-http-client to get these changes. This also drops using 'container-info' directly in this repo's tests. Given that the impl in that module and in the http-client might diverge over time, it isn't necessarily valid to use 'container-info' here. Closes: #2914 Closes: #2941
trentm
added a commit
that referenced
this issue
Oct 27, 2022
…nfig re-fetch interval (#3008) Update elastic-apm-http-client to get these changes. This also drops using 'container-info' directly in this repo's tests. Given that the impl in that module and in the http-client might diverge over time, it isn't necessarily valid to use 'container-info' here. Closes: #2914 Closes: #2941
fpm-peter
pushed a commit
to fpm-git/apm-agent-nodejs
that referenced
this issue
Aug 20, 2024
…nfig re-fetch interval (elastic#3008) Update elastic-apm-http-client to get these changes. This also drops using 'container-info' directly in this repo's tests. Given that the impl in that module and in the http-client might diverge over time, it isn't necessarily valid to use 'container-info' here. Closes: elastic#2914 Closes: elastic#2941
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is your feature request related to a problem? Please describe.
We've identified a new pattern for pod UID strings. See spec PR.
Describe the solution you'd like
Update client parsing to include this new pattern. Also, consider pulling
container-infomodule into either the client or agent project.The text was updated successfully, but these errors were encountered: