Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[processor/tailsampling] Include componentID as prefix in metrics 'policy' #34192

Merged
merged 8 commits into from
Aug 27, 2024
Merged

[processor/tailsampling] Include componentID as prefix in metrics 'policy' #34192

merged 8 commits into from
Aug 27, 2024

Commits on Jul 22, 2024

  1. Include componentID as prefix in metrics 'policy' 

    This change includes the componentID as a dot prefix to the metrics
'policy' dimension when generating metrics for the processor. The change
ensures that similarly named policys in the tail sampling processor that
belong to different components also has a unique value.

Resolves: open-telemetry#34099
    @EOjeah
    EOjeah committed Jul 22, 2024
    Configuration menu
    Copy the full SHA
    47e2f00 View commit details
    Browse the repository at this point in the history

Commits on Aug 22, 2024

  1. Include Test to varify policy dimension value 

    Undo previous rename of telemtry to telemetryBuilder as it's not related to the attached issue. In order for the component ID to be passed from settings, the newTraceProcessor definition had to be chanaged to accept processor.Settings and not just the TelemetrySettings
    @EOjeah
    EOjeah committed Aug 22, 2024
    Configuration menu
    Copy the full SHA
    ff0872d View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' of github.com:EOjeah/opentelemetry-collector-cont… 

    …rib into bug/unique-policy-name-tail-sampling-processor
    @EOjeah
    EOjeah committed Aug 22, 2024
    Configuration menu
    Copy the full SHA
    ef09f24 View commit details
    Browse the repository at this point in the history

  3. Merge branch 'main' of github.com:EOjeah/opentelemetry-collector-cont… 

    …rib into bug/unique-policy-name-tail-sampling-processor
    @EOjeah
    EOjeah committed Aug 22, 2024
    Configuration menu
    Copy the full SHA
    d90b57f View commit details
    Browse the repository at this point in the history

  4. Include change log entry

    @EOjeah
    EOjeah committed Aug 22, 2024
    Configuration menu
    Copy the full SHA
    518ccbd View commit details
    Browse the repository at this point in the history

  5. Merge branch 'main' of github.com:EOjeah/opentelemetry-collector-cont… 

    …rib into bug/unique-policy-name-tail-sampling-processor
    @EOjeah
    EOjeah committed Aug 22, 2024
    Configuration menu
    Copy the full SHA
    5406267 View commit details
    Browse the repository at this point in the history

Commits on Aug 23, 2024

  1. Merge branch 'main' of github.com:EOjeah/opentelemetry-collector-cont… 

    …rib into bug/unique-policy-name-tail-sampling-processor
    @EOjeah
    EOjeah committed Aug 23, 2024
    Configuration menu
    Copy the full SHA
    078511c View commit details
    Browse the repository at this point in the history

Commits on Aug 27, 2024

  1. Merge branch 'main' of github.com:EOjeah/opentelemetry-collector-cont… 

    …rib into bug/unique-policy-name-tail-sampling-processor
    @EOjeah
    EOjeah committed Aug 27, 2024
    Configuration menu
    Copy the full SHA
    ecba7dc View commit details
    Browse the repository at this point in the history