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: disable the metadata pipeline for OTC log collector by default #2084

Merged
merged 1 commit into from
Feb 3, 2022

Conversation

swiatekm
Copy link

@swiatekm swiatekm commented Feb 3, 2022

Description

This pipeline was enabled by default by mistake, as the default configuration doesn't need it, and the k8s_tagger within puts some additional load on the API server. I'm commenting it out as we can't use conditional logic directly in values.yaml.


Checklist

Remove items which don't apply to your PR.

  • Changelog updated
Testing performed
  • Redeploy fluentd and fluentd-events pods
  • Confirm events, logs, and metrics are coming in

@swiatekm swiatekm force-pushed the fix/otc-log-collector-pipeline branch from f50c142 to 33c3dc8 Compare February 3, 2022 13:16
@github-actions github-actions bot added the documentation documentation label Feb 3, 2022
@swiatekm swiatekm force-pushed the fix/otc-log-collector-pipeline branch from 33c3dc8 to 7f374d4 Compare February 3, 2022 13:17
@swiatekm swiatekm marked this pull request as ready for review February 3, 2022 13:17
@swiatekm swiatekm requested a review from a team as a code owner February 3, 2022 13:17
@swiatekm swiatekm force-pushed the fix/otc-log-collector-pipeline branch from 7f374d4 to 1ec78fb Compare February 3, 2022 13:58
@swiatekm swiatekm enabled auto-merge (rebase) February 3, 2022 13:59
@swiatekm swiatekm merged commit 6cc2763 into main Feb 3, 2022
@swiatekm swiatekm deleted the fix/otc-log-collector-pipeline branch February 3, 2022 14:02
@perk-sumo perk-sumo added this to the v2.5 milestone Feb 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants