open-telemetry / open-telemetry/weaver

JQ helpers: support for includedNamespaces

Open
#900 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Rust
Stars
486
Forks
108
Avg merge
3d 4h
Merged PRs (30d)
40

Description

We allow to exclude namespaces via an option, but sometimes it's desirable to generate a small and known subset of signals.

JQ filter that implements it is not trivial (nothing in JQ is), so it might be a good idea to support includedNamespaces and support filtering in this way as well.

Here's the filter

    filter: >
      semconv_grouped_metrics({"exclude_deprecated": true})
      | map(.metrics)
      | flatten
      | .[]
      | select (($included_namespaces | length == 0) or (.root_namespace as $ns | $included_namespaces | index($ns)))

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

The issue names no file or test. Start by locating the existing namespace-exclusion option and its JQ filter, then compare it with the provided includedNamespaces filter. Done means the option selects the requested root namespaces, preserves the empty-list behavior shown, and is covered by relevant tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
tooling
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.