Trask Stalnaker
Trask Stalnaker
> This OTEP doesn't include any technical detail about the reversal of a decision that was agonized over for months. @dyladan can you add this as a code review comment...
> Is there a prototype for this? https://github.com/open-telemetry/opentelemetry-python/pull/4587
@open-telemetry/android-maintainers @open-telemetry/cpp-maintainers @open-telemetry/dotnet-maintainers @open-telemetry/erlang-maintainers @open-telemetry/go-maintainers @open-telemetry/java-maintainers @open-telemetry/javascript-maintainers @open-telemetry/php-maintainers @open-telemetry/python-maintainers @open-telemetry/ruby-maintainers @open-telemetry/rust-maintainers @open-telemetry/swift-maintainers @open-telemetry/collector-maintainers @open-telemetry/collector-contrib-maintainers please review if you have a chance, thanks!!
> Should we require SDKs to provide clear guidance on when the complex attributes can be used and when they shouldn't be? Maybe as part of comments or documentation in...
> I am against this proposal as I do not think that handling `event.name` for `Enabled` is more important than for other attributes. I'm not sure I agree, but even...
removing `triage:followup` since the automation will automatically add it back 2 weeks after additional comments / ref links
> adding `LoggerConfig#min_severity` and `LoggerConfig#trace_based` properties are no brainers. I wonder if anyone would disagree? If not, then adding those two properties (along with extending [#4381](https://github.com/open-telemetry/opentelemetry-specification/pull/4381) to reference them) seems...
> If the community decides to that bringing proper pipeline support to SDKs is important, I do think its important to solve it holistically, and look at how the concepts...
@mariomac @marctc is the issue due to "Require branches to be up to date before merging"? If so, it's ok to remove that setting (see https://github.com/open-telemetry/community/blob/main/docs/how-to-configure-new-repository.md#branch-protection-rule-main). Or, if you want...
> Definitely removing the _Require branches to be up to date before merging_ setting would help 💯 done! can you see if this is enough to unblock work, since otherwise...