Andrew Wilkins

Results 473 comments of Andrew Wilkins

Related: https://github.com/open-telemetry/opentelemetry-collector/issues/13778 I haven't personally run into a need for this (yet), but I can certainly understand the need and value. I think the way to solve this at the...

> field_descriptions These aren't really descriptions so much as names/identifiers. Anyway, I suggest aligning with https://docs.aws.amazon.com/vpc/latest/userguide/flow-log-records.html and https://docs.aws.amazon.com/cli/latest/reference/ec2/create-flow-logs.html, and call this config `format`. > Detection of VPC logs coming through...

> I will keep this open for their feedback and once we have agreement, let's either close this one or #38861 #38861 is more of a meta issue, so I...

@Kavindu-Dodan I think that works for decoding AWS service logs, but I'm not sure that it will be ideal for parsing custom application logs exported from CloudWatch in #44562. If...

> If encoding is specialized only for CW, then they must expect CW message JSON which they must parse to extract messages This seems like a pretty tall order for...

> We can come back to this if you still see above configurations make it complex. My view is, we have now pushed complication to implementation (ex:- Detecting trigger payload...

> This comes from Lambda framework and is a standard. The REPORT section comes with memory consumption & Runtime information. Got it. This is probably better handled by https://github.com/open-telemetry/opentelemetry-lambda, but...

To move this forward, can we first explore how custom logs will be handled through CloudWatch? We can start by focusing on a relatively simple case: a Lambda application that...

> If there's any further refinements, we can move that to #44562. OK, let's take the discussion there. The outcome affects how we implement this one though, so please hold...

Reopening as https://github.com/elastic/beats/pull/40755 has been reverted. CI started failing after that was merged; it doesn't look likely that it's specific to the change, but all the same Beats CI needed...