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.

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

add otel collector service telemetry metrics #364

Merged
merged 5 commits into from
Aug 18, 2023

Commits on Aug 17, 2023

  1. add otel collector service telemetry metrics

    Configure a prometheus endpoint which serves metrics about the
    otel-collector itself. Also adds a prom_scraper_config so promscraper
    scrapes the metrics.
    
    Signed-off-by: Rebecca Roberts <[email protected]>
    mkocher committed Aug 17, 2023
    Configuration menu
    Copy the full SHA
    29ac92b View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    0815d9d View commit details
    Browse the repository at this point in the history

Commits on Aug 18, 2023

  1. fix: selectively enable prom scraping of otel collectors

    When OTel Collector jobs are disabled, don't output any prom scraper
    configuration so that the prom scraper skips over these jobs. This
    avoids continual error messages logged by the prom scraper when it can't
    scrape a destination.
    ctlong authored and acrmp committed Aug 18, 2023
    Configuration menu
    Copy the full SHA
    160698d View commit details
    Browse the repository at this point in the history
  2. fix(otel-collector-windows): align default port with otel-collector

    Signed-off-by: Andrew Crump <[email protected]>
    ctlong authored and acrmp committed Aug 18, 2023
    Configuration menu
    Copy the full SHA
    8285ad1 View commit details
    Browse the repository at this point in the history
  3. refactor: remove service prefix from telemetry properties

    Since we already dropped exact parity with OTel Collector settings with
    `ingress.grpc.tls.*`, we think it's preferable to remove the `service.`
    prefix from telemetry properties in the OTel Collector jobs.
    
    Signed-off-by: Carson Long <[email protected]>
    acrmp committed Aug 18, 2023
    Configuration menu
    Copy the full SHA
    0c0ed48 View commit details
    Browse the repository at this point in the history