Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Mar 13, 2025

What does this PR do?

Fixes the Elasticsearch exporter configuration in opentelemetry-kube-stack values. This was accidentally removed in #6444.

I'm going to file an issue to add tests that would've caught this as a follow-up. This is an urgent fix to get in before 9.0.0 RC2, which is why I'm not doing it in this PR.

Why is it important?

Currently, this configuration sends traces to the wrong index.

Checklist

  • I have read and understood the pull request guidelines of this project.
  • I have made corresponding change to the default configuration files
  • I have added an entry in ./changelog/fragments using the changelog tool

This is an automatic backport of pull request #7380 done by [Mergify](https://mergify.com).
(cherry picked from commit 5a922f2) # Conflicts: #	deploy/helm/edot-collector/kube-stack/values.yaml
@mergify mergify bot added backport conflicts There is a conflict in the backported pull request labels Mar 13, 2025
@mergify mergify bot requested a review from a team as a code owner March 13, 2025 13:37
@mergify mergify bot requested review from blakerouse and removed request for a team March 13, 2025 13:37
@mergify mergify bot requested a review from kaanyalti March 13, 2025 13:37
@mergify
Copy link
Contributor Author

mergify bot commented Mar 13, 2025

Cherry-pick of 5a922f2 has failed:

On branch mergify/bp/8.16/pr-7380 Your branch is up to date with 'origin/8.16'. You are currently cherry-picking commit 5a922f258. (fix conflicts and run "git cherry-pick --continue") (use "git cherry-pick --skip" to skip this patch) (use "git cherry-pick --abort" to cancel the cherry-pick operation) Changes to be committed:	new file: changelog/fragments/1741865161-fix-otel-kube-stack-config.yaml Unmerged paths: (use "git add <file>..." to mark resolution)	both modified: deploy/helm/edot-collector/kube-stack/values.yaml 

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@swiatekm
Copy link
Contributor

Not necessary on this branch.

@swiatekm swiatekm closed this Mar 13, 2025
@mergify mergify bot deleted the mergify/bp/8.16/pr-7380 branch March 13, 2025 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport conflicts There is a conflict in the backported pull request

3 participants