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

Start time metric adjuster should deal with cumulative resets #37717

Closed
ridwanmsharif opened this issue Feb 5, 2025 · 1 comment · Fixed by #37718
Closed

Start time metric adjuster should deal with cumulative resets #37717

ridwanmsharif opened this issue Feb 5, 2025 · 1 comment · Fixed by #37718
Labels
bug Something isn't working needs triage New item requiring triage receiver/prometheus Prometheus receiver

Comments

@ridwanmsharif
Copy link
Member

Component(s)

receiver/prometheus

What happened?

When the Start time metric adjuster is used - all points used the same start timestamp. Even after a reset, which makes no
sense for a counter which is not supposed to go down.

Instead when a reset is detected, the adjuster should use the reset timestamp for all next points for the metric.

Collector version

all

Environment information

Environment

OS: (e.g., "Ubuntu 20.04")
Compiler(if manually compiled): (e.g., "go 14.2")

OpenTelemetry Collector configuration

Log output

Additional context

No response

@ridwanmsharif ridwanmsharif added bug Something isn't working needs triage New item requiring triage labels Feb 5, 2025
@github-actions github-actions bot added the receiver/prometheus Prometheus receiver label Feb 5, 2025
Copy link
Contributor

github-actions bot commented Feb 5, 2025

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

ridwanmsharif added a commit to ridwanmsharif/opentelemetry-collector-contrib that referenced this issue Feb 5, 2025
…djuster

Fixes open-telemetry#37717

Prior to this change, when the start time metric adjuster was used all
points used the same start timestamp. Even after a reset, which makes no
sense for a counter which is not supposed to go down.

Instead this change makes it so that when a reset is detected, the the
reset points timestamp is used as the next start time.

Signed-off-by: Ridwan Sharif <[email protected]>
ridwanmsharif added a commit to ridwanmsharif/opentelemetry-collector-contrib that referenced this issue Feb 5, 2025
…djuster

Fixes open-telemetry#37717

Prior to this change, when the start time metric adjuster was used all
points used the same start timestamp. Even after a reset, which makes no
sense for a counter which is not supposed to go down.

Instead this change makes it so that when a reset is detected, the the
reset points timestamp is used as the next start time.

Signed-off-by: Ridwan Sharif <[email protected]>
khushijain21 pushed a commit to khushijain21/opentelemetry-collector-contrib that referenced this issue Feb 14, 2025
…djuster (open-telemetry#37718)

Fixes
open-telemetry#37717

Prior to this change, when the start time metric adjuster was used all
points used the same start timestamp. Even after a reset, which makes no
sense for a counter which is not supposed to go down.

Instead this change makes it so that when a reset is detected, the the
reset points timestamp is used as the next start time.

Signed-off-by: Ridwan Sharif <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs triage New item requiring triage receiver/prometheus Prometheus receiver
Projects
None yet
1 participant