Skip to content

Conversation

@dashpole
Copy link
Contributor

Description

Promote the receiver.prometheusreceiver.RemoveStartTimeAdjustment to beta, which disables metric start time adjustment by default.

Link to tracking issue

Fixes #43656

Testing

Tests were updated to test with the featue gate enabled and disabled.

@ridwanmsharif

This should wait until after #43912 to merge.

@dashpole dashpole requested review from a team and ArthurSens as code owners October 30, 2025 19:31
@github-actions github-actions bot added the receiver/prometheus Prometheus receiver label Oct 30, 2025
@dashpole
Copy link
Contributor Author

cc @eriksywu

@dashpole dashpole force-pushed the prometheus_no_adjustement branch 2 times, most recently from 4c6231f to aa7ddfb Compare October 30, 2025 20:10
@dashpole dashpole force-pushed the prometheus_no_adjustement branch from aa7ddfb to 4a752ba Compare October 30, 2025 20:20
Copy link
Member

@ArthurSens ArthurSens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just one nit

@dashpole dashpole added the ready to merge Code review completed; ready to merge by maintainers label Nov 5, 2025
@songy23 songy23 merged commit 90c5ffd into open-telemetry:main Nov 5, 2025
206 checks passed
@github-actions github-actions bot added this to the next release milestone Nov 5, 2025
jelly-afk pushed a commit to jelly-afk/opentelemetry-collector-contrib that referenced this pull request Nov 6, 2025
…lemetry#43916)

#### Description

Promote the receiver.prometheusreceiver.RemoveStartTimeAdjustment to
beta, which disables metric start time adjustment by default.

#### Link to tracking issue
Fixes
open-telemetry#43656

#### Testing

Tests were updated to test with the featue gate enabled and disabled.

@ridwanmsharif 

This should wait until after
open-telemetry#43912
to merge.

---------

Co-authored-by: Arthur Silva Sens <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Code review completed; ready to merge by maintainers receiver/prometheus Prometheus receiver

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[receiver/prometheus] Disable initial-point adjustment by default

4 participants