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

fix: pod_association #1254

Merged
merged 1 commit into from
Feb 20, 2025
Merged

fix: pod_association #1254

merged 1 commit into from
Feb 20, 2025

Conversation

kinolaev
Copy link
Contributor

I'm sorry, I made a mistake in #1239, there are must be three pod_association instead of three source inside one pod_association. All source blocks has to match for the pod to be associated with a signal but we want it to be associated when one of the conditions is met (as in Opentelemetry Collector chart).

@kinolaev kinolaev force-pushed the fix-pod-association branch from 0e3d172 to 01fd469 Compare February 19, 2025 11:37
@kinolaev
Copy link
Contributor Author

Hello @petewall and @rlankfo, sorry for pinging you but I believe it is very important fix, because my previous PR (part of release 2.0.11) could break k8sattributes processor for many users

Copy link
Member

@rlankfo rlankfo left a comment

Choose a reason for hiding this comment

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

thanks for catching this!

@petewall petewall merged commit e1d0d5a into grafana:main Feb 20, 2025
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants