Lee E Hinman

@jmlrt updated with https://github.com/elastic/helm-charts/pull/1688 while I was testing my changes :-) Closing since his work does the update and fixes some 8.x issues.

I've used the approach where you try to open X times with a Y delay between then in cases like this. pseudo code: ``` func myOpen (name string) (*Channel, error)...

@faec this might be more trouble than it is worth, but what do you think about adding a benchmark test for this? I was thinking 2 scenarios might be interesting....

> @leehinman I was expecting to write more tests ("all existing tests pass" isn't as reassuring as I want it to be). :-) yeah. >Some scenarios you mentioned might be...

The change looks "OK" to me, but metricbeat tests are failing with a rabbitmq test.

@kaanyalti were you able to look into the rabbitmq failures?

For Option1 do we have to provide a conditional? I think both inputs could be enabled at the same time, it would just have to be non-fatal for the source...

Do any of the fields change when you ingest events with agentbeat? I'm noticing some name changes in monitoring, input, and module names and I'm curious if that changes the...

Closing future work is tracked in https://github.com/elastic/ingest-dev/issues/6243

Looking at the post I think this is expected behavior. The 900 open files are because filebeat is keeping the file open until the `[close_inactive](https://www.elastic.co/docs/reference/beats/filebeat/filebeat-input-log#filebeat-input-log-close-inactive) timer (default 5 min) expires...