Component(s)
processor/k8sattributes
Describe the issue you're reporting
Description
In PR #48398, which fixes memory leak issues with associations by container.id, we implemented the fix under the assumption that PodIdentifiers are unique and maps to a single pod, as discussed in this thread.
However, looking at the implementation of config.Validate(), there doesn't seem to be any check currently enforcing or verifying this uniqueness.
I am opening this issue following this PR comment to address this missing validation.
Tip
React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding +1 or me too, to help us triage it. Learn more here.
Component(s)
processor/k8sattributes
Describe the issue you're reporting
Description
In PR #48398, which fixes memory leak issues with associations by
container.id, we implemented the fix under the assumption thatPodIdentifiersare unique and maps to a single pod, as discussed in this thread.However, looking at the implementation of
config.Validate(), there doesn't seem to be any check currently enforcing or verifying this uniqueness.I am opening this issue following this PR comment to address this missing validation.
Tip
React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding
+1orme too, to help us triage it. Learn more here.