Robert Pająk
Robert Pająk
Our PowerShell module starts to have not so simple logic. It may be good to test some parts e.g. using https://pester.dev/
We can create a PowerShell module manifest for the https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation/blob/main/OpenTelemetry.DotNet.Auto.psm1 so that we can: 1. Add combability info (`Destkop` edition and `5.1` version?) 2. Add license and author info 3....
After a quick look at 1. Your `clang` version looks old. I would suggest using at least `7.0`. 2. What is your version of `gcc`? I suggest at least `8.3`...
The release artifacts should contain dependencies' license and copywrite notices. This looks like the way suggested by CNCF legals: https://infra.apache.org/licensing-howto.html (https://github.com/open-telemetry/community/issues/1280#issuecomment-1607950483) Reference: https://github.com/open-telemetry/community/issues/1280 In OTel .NET AutoInstrumentation we are bundling...
> I think we should also update the existing examples to use the new instrumentation. > > I propose to: > > 1. Merge this PR > 2. Create an...
Implement https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/configuration/file-configuration.md#parse for YAML.
### Problem Statement OTel maintainers and approvers are on low capacity and the `otelmongo` module has no codeowners. ### Proposed Solution Move the ownership to MongoDB team. First deprecate and...
Should this be `TraceOption`? I know we have the `SpanExporter`, but I wonder if this needs to match that or the corollary to metric: trace. _Originally posted by @MrAlias in...
### Description Issue detected by `gosec` ``` golangci-lint ./zpages templates.go:70:10: G203: The used method does not auto-escape HTML. This can potentially lead to 'Cross-site Scripting' vulnerabilities, in case the attacker...
From https://github.com/open-telemetry/opentelemetry-specification/blob/563958cb2bd8529990f19fdce7a5f3643bf63091/specification/configuration/sdk-environment-variables.md?plain=1#L222: > The SDK MAY accept a comma-separated list to enable setting multiple exporters. This would be useful, but doesn't really fit with how the package is currently designed....