Alan West

Results 59 comments of Alan West

The receiver's readme states: > This receiver is beta. It has been tested on macOS and Linux with .NET v3.1. I haven't had a chance to try different versions of...

Hmm, good thought on upgrading the SDK, though I'm still hitting the issue after installing the latest `5.0.300`. I'm not sure what other factors may be at play here... when...

Ok so, bear with my silliness here... not sure this'll provide any clues, but I've been futzing around in nonsensical ways. I changed: https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/f8c550fe19262199463401ac5c3fdce44130a31b/receiver/dotnetdiagnosticsreceiver/network/reader.go#L77-L83 to print out each byte being...

@arminru I suspect this is different from #1006. When I join the link from the calendar for the .NET auto-instrumentation SIG, I do not get the "recording in progress" announcement....

> As a regular attendee of the .NET auto-instrumentation SIG meeting, I can confirm that for at least the past two weeks, I have been receiving the "recording in progress"...

Someone must have done something because I think this is the collector SIG from today https://youtu.be/jFjNEdSndhE. If someone can confirm, then I think this issue can be closed.

> I could live with "Shim" but I think "Adapter" could also work. I like `Adapter` or even `Bridge`. Wasn't a fan of `Shims` anyways, but chose it because of...

> The spec is using the word "shim" And apparently matches wikipedia's definition quite well https://en.wikipedia.org/wiki/Shim_(computing)

I've updated the PR description with a few of the changes discussed. 1) OpenTelemetry.PersistentStorage -> OpenTelemetry.PersistentStorage.FileSystem 2) OpenTelemetry.Shims.Serilog -> OpenTelemetry.Appenders.Serilog 3) OpenTelemetry.Shims.EventSource -> OpenTelemetry.Appenders.EventSource Instead of the word using the...

With https://github.com/open-telemetry/opentelemetry-dotnet-contrib/pull/1123, we have asserted a preference for pluralizing words in our package names. I've updated note 2 in this issue regarding pluralization. Let me know if this sounds reasonable...