https://github.com/open-telemetry/weaver/blob/152c88630b1379a403519d3468b38d7cbc4c1097/defaults/jq/semconv.jq#L123 It looks like this relies on the `id` having a particular form that is not guanteed. For example: https://github.com/open-telemetry/semantic-conventions/blob/06536459c81f8d1d83d53bd04ab6638cb1315c42/model/cpu/deprecated.yaml#L2-L18 Similar to `semconv_attributes` it looks like this should be update to `map(select(has("deprecated") | not))`: https://github.com/open-telemetry/weaver/blob/152c88630b1379a403519d3468b38d7cbc4c1097/defaults/jq/semconv.jq#L80C9-L80C45 This fix resolved the lack of filtering in [this](https://github.com/open-telemetry/opentelemetry-go/pull/7032/commits/d63632a85b8fb87103fc395df99adef71d415fcf) usecase.
weaver/defaults/jq/semconv.jq
Line 123 in 152c886
It looks like this relies on the
idhaving a particular form that is not guanteed. For example:https://github.com/open-telemetry/semantic-conventions/blob/06536459c81f8d1d83d53bd04ab6638cb1315c42/model/cpu/deprecated.yaml#L2-L18
Similar to
semconv_attributesit looks like this should be update tomap(select(has("deprecated") | not)):https://github.com/open-telemetry/weaver/blob/152c88630b1379a403519d3468b38d7cbc4c1097/defaults/jq/semconv.jq#L80C9-L80C45
This fix resolved the lack of filtering in this usecase.