Paulo Janotti
Paulo Janotti
One observation @djaglowski, there shouldn't be a mix of named and unnamed entries in the same event. That is related to the code used to generate the actual event. So...
Sorry, for the delay... @strawgate > @pjanotti by generated names do you mean something like param1, param2, param3? Isn't that option 2? Yes, correct - just using the place holder...
@djaglowski I do prefer that we always have a map, this should simplify any ottl processing. @strawgate any preferences?
(was about to hit enter @BinaryFissionGames :) saying the same) The Event API already handles remote operations, see [EvtOpenSession](https://learn.microsoft.com/en-us/windows/win32/api/winevt/nf-winevt-evtopensession) and [EVT_RPC_LOGIN](https://learn.microsoft.com/en-us/windows/win32/api/winevt/ns-winevt-evt_rpc_login). So no need to add a new dependency. The...
Thanks @JonathanWamsley!
Please add the `Run Windows` label.
Fixing one of the `Run Windows` failures via #33835
#33837 skips another failing test
Test failure is https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/33438
`correctness-traces` failures seems unrelated https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/9766816496/job/26960642383#step:7:908 ``` 2024/07/02 19:37:14 | Sent: 3,907 traces (6/sec) | Received: 3,907 items (6/sec) panic: test timed out after 10m0s running tests: TestTracingGoldenData (10m0s) TestTracingGoldenData/opencensus-otlp (9m30s)...