Skip to content

[Telemetry Operator] Exclusion of containers enables system namespace inclusion #15490

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

Closed
a-thaler opened this issue Sep 14, 2022 · 1 comment · Fixed by #15494
Closed

[Telemetry Operator] Exclusion of containers enables system namespace inclusion #15490

a-thaler opened this issue Sep 14, 2022 · 1 comment · Fixed by #15494
Assignees
Labels
area/logging Issues or PRs related to the logging module (deprecated) kind/bug Categorizes issue or PR as related to a bug. release blocker
Milestone

Comments

@a-thaler
Copy link
Contributor

Description
Assume a pipeline configuration like that:

apiVersion: telemetry.kyma-project.io/v1alpha1
kind: LogPipeline
metadata:
  name: cls-application-logs
spec:
  input:
    application:
      containers:
        exclude:
          - istio-proxy
...

then the expected behaviour according to documentation is that system namespaces are getting ignored (default behaviour for namespace selection stays untouched). In fact, that setting enables the streaming of logs for system namespaces.

@lindnerby
Copy link
Member

Fluent bit's rewrite_tag filter only evaluated rules until one matches, thus chaining rules for namespace and container selectors didn't work. We fixed this by piggybacking container selector rules in the rewrite_tag filter, if present and
for any namespace selectors we create an additional grep filter.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/logging Issues or PRs related to the logging module (deprecated) kind/bug Categorizes issue or PR as related to a bug. release blocker
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants