Victor Martinez

Results 227 comments of Victor Martinez

> @v1v (or others), That change triggered a build error in each test step, as expected. For example: https://apm-ci.elastic.co/blue/organizations/jenkins/apm-agent-nodejs%2Fapm-agent-nodejs-mbp/detail/PR-2950/2/pipeline Much better now, the `filebeat` exports the docker context accordingly: -...

I've been thinking about enriching the docker context for the healthcheck, and found we could use `metricbeat` (https://www.elastic.co/guide/en/beats/metricbeat/current/metricbeat-metricset-docker-healthcheck.html), so I'd propose to create a new library step in the `apm-pipeline-library`...

`Metricbeat` is now reporting the docker healthy status for all the containers in the current context -> see [this](https://apm-ci.elastic.co/job/apm-agent-nodejs/job/apm-agent-nodejs-mbp/job/PR-2950/12/artifact/docker-10.0-release-metricbeat.log-20220927.ndjson/*view*/) If needed we could even send those details to Elasticsearch. Let...

> Is that a problem? Not at all. We can even move away from the existing file based and send the data to ES if that will help to get...

@trentm , I think this PR is now ready, I reverted the changes required for testing this new behaviour of storing the docker context using the metricbeat and filebeat.

I've just realised the `from` transformation is not working as expected, so I'm debugging what's going on, sorry for the spam!

@elastic/observablt-robots , I'd like to gather some feedback from you, this is just a PoC, just wanna validate if this way of running things in K8s could help us to...

I guess the default docker image requires some further configuration as done in the current dockerfile in the tests folder.

As far as I see `time_ns` is supported from `3.7` as per https://docs.python.org/3/library/time.html#time.time_ns While `3.7` is the lowers supported python version as per -> https://devguide.python.org/versions/#supported-versions Though, there might be people...

I've just created https://github.com/ansible-collections/community.general/pull/5342 with a similar proposal as discussed here