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(k8sprocessor): add ignored pod names as config parameter #354

Merged
merged 5 commits into from
Dec 16, 2021

Conversation

pmalek-sumo
Copy link
Contributor

@pmalek-sumo pmalek-sumo commented Dec 2, 2021

@pmalek-sumo pmalek-sumo self-assigned this Dec 2, 2021
@github-actions github-actions bot added the go label Dec 2, 2021
@pmalek-sumo pmalek-sumo force-pushed the k8sprocessor-add-ignore-pod-names-as-parameter branch from ca7a45e to f1650d5 Compare December 2, 2021 12:12
@pmalek-sumo pmalek-sumo marked this pull request as ready for review December 2, 2021 12:27
@pmalek-sumo pmalek-sumo requested a review from a team as a code owner December 2, 2021 12:27
@pmalek-sumo pmalek-sumo linked an issue Dec 2, 2021 that may be closed by this pull request
@pmalek-sumo pmalek-sumo requested a review from pmm-sumo December 14, 2021 09:48
@pmalek-sumo pmalek-sumo force-pushed the k8sprocessor-add-ignore-pod-names-as-parameter branch from 832fc81 to ced2535 Compare December 16, 2021 16:33
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Dec 16, 2021
@@ -365,3 +365,34 @@ func WithDelimiter(delimiter string) Option {
return nil
}
}

var defaultExcludes = ExcludeConfig{
Pods: []ExcludePodConfig{
Copy link
Contributor

Choose a reason for hiding this comment

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

As discussed, we could keep the list empty and populate it in Helm chart only for tracing pipeline

@pmalek-sumo pmalek-sumo force-pushed the k8sprocessor-add-ignore-pod-names-as-parameter branch from d1db6ef to dd2b2d7 Compare December 16, 2021 17:32
@pmalek-sumo pmalek-sumo enabled auto-merge (squash) December 16, 2021 17:39
@pmalek-sumo pmalek-sumo merged commit 2dcbc25 into main Dec 16, 2021
@pmalek-sumo pmalek-sumo deleted the k8sprocessor-add-ignore-pod-names-as-parameter branch December 16, 2021 17:55
andrzej-stencel added a commit that referenced this pull request Feb 23, 2022
andrzej-stencel added a commit that referenced this pull request Feb 23, 2022
andrzej-stencel added a commit that referenced this pull request Feb 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation go
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Metadata for pods with specific names is not added
3 participants