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

feat: Upgrade OTel Collector to version 0.120.0 (contrib 0.120.1) #1873

Merged
merged 28 commits into from
Mar 3, 2025

Conversation

hisarbalik
Copy link
Contributor

@hisarbalik hisarbalik commented Feb 25, 2025

Description

Changes proposed in this pull request (what was done and why):

  • Bump OTel image version to 0.120.0
  • Bump OTel contrib version to 0.120.1
  • Update transform processor statements with new flat expressions
  • Fix issue, prometheus namespace filter not working for metrics otelcol_*
  • Add new annotation telemetry.kyma-project.io/internal-metrics-compatibility-mode to the telemetry resource to enabled backward compatibility for OTel internal metrics
  • Add benchmark test result for OTel version 0.120.0
  • Add e2e test with compatibility mode enabled for metric pipeline self-monitor healthy path

Changes refer to particular issues, PRs or documents:

Traceability

  • The PR is linked to a GitHub issue.
  • The follow-up issues (if any) are linked in the Related Issues section.
  • If the change is user-facing, the documentation has been adjusted.
  • If a CRD is changed, the corresponding Busola ConfigMap has been adjusted.
  • The feature is unit-tested.
  • The feature is e2e-tested.

@hisarbalik hisarbalik added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. kind/chore Categorizes issue or PR as related to a chore. area/metrics MetricPipeline area/traces TracePipeline area/manager Manager or module changes labels Feb 25, 2025
@hisarbalik hisarbalik requested a review from a team as a code owner February 25, 2025 07:12
@kyma-bot kyma-bot added cla: yes Indicates the PR's author has signed the CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 25, 2025
@hisarbalik hisarbalik added this to the 1.35.0 milestone Feb 25, 2025
@kyma-bot kyma-bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 25, 2025
@hisarbalik hisarbalik changed the title chore: Bump OTel image version to 0.120.0 chore: Bump OTel image version to 0.120.0 (contrib 0.120.1) Feb 25, 2025
Copy link
Collaborator

@a-thaler a-thaler left a comment

Choose a reason for hiding this comment

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

I cannot find the changes for the self-monitor, compensating the missing _total suffix

@a-thaler a-thaler changed the title chore: Bump OTel image version to 0.120.0 (contrib 0.120.1) feat: Upgrade OTel Collector to version 0.120.0 (contrib 0.120.1) Feb 27, 2025
@a-thaler a-thaler added kind/feature Categorizes issue or PR as related to a new feature. and removed kind/chore Categorizes issue or PR as related to a chore. area/manager Manager or module changes labels Feb 27, 2025
@hisarbalik hisarbalik removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 28, 2025
@a-thaler a-thaler dismissed their stale review February 28, 2025 15:22

rakesh takes over

@hisarbalik hisarbalik added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 28, 2025
@hisarbalik hisarbalik removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 3, 2025
@hisarbalik hisarbalik requested a review from NHingerl March 3, 2025 11:52
@kyma-bot kyma-bot added the lgtm Looks good to me! label Mar 3, 2025
@hisarbalik hisarbalik merged commit 8822ef8 into kyma-project:main Mar 3, 2025
54 checks passed
@hisarbalik hisarbalik deleted the bump-otel-version-to-0.120.0 branch March 3, 2025 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Documentation changes area/metrics MetricPipeline area/traces TracePipeline cla: yes Indicates the PR's author has signed the CLA. kind/feature Categorizes issue or PR as related to a new feature. lgtm Looks good to me! size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants