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

Add include logs functionality #260

Merged
merged 3 commits into from
Nov 17, 2021

Conversation

chaitanyaphalak
Copy link
Contributor

@chaitanyaphalak chaitanyaphalak commented Nov 1, 2021

Set splunk.com/include annotation to true on pod and .Values.logsCollection.containers.useSplunkIncludeAnnotation flag to true to include its logs from ingested to your Splunk platform deployment.
All other logs will be ignored.

@chaitanyaphalak chaitanyaphalak requested review from a team as code owners November 1, 2021 17:48
Copy link
Contributor

@dmitryax dmitryax left a comment

Choose a reason for hiding this comment

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

Overall looks good.

Please delete the attribute once it's used in the resource/logs processor.

@chaitanyaphalak chaitanyaphalak force-pushed the add-include-logs-funtionality branch from b815799 to 65bbafe Compare November 1, 2021 23:38
@chaitanyaphalak chaitanyaphalak force-pushed the add-include-logs-funtionality branch 2 times, most recently from 0c0afbc to 0d524e8 Compare November 2, 2021 15:44
@chaitanyaphalak chaitanyaphalak force-pushed the add-include-logs-funtionality branch 3 times, most recently from bdcb2d0 to 9282502 Compare November 3, 2021 15:38
@chaitanyaphalak chaitanyaphalak force-pushed the add-include-logs-funtionality branch from b39d2af to 39fb832 Compare November 11, 2021 02:16
@chaitanyaphalak
Copy link
Contributor Author

chaitanyaphalak commented Nov 11, 2021

Hey @dmitryax , updated the PR, with your latest recommendations, lint-test is failing due to some fluentd error, I am not sure why?

@chaitanyaphalak chaitanyaphalak force-pushed the add-include-logs-funtionality branch 3 times, most recently from 27a8537 to e601539 Compare November 12, 2021 19:02
Copy link
Contributor

@dmitryax dmitryax left a comment

Choose a reason for hiding this comment

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

LGTM. did you run make render?

@chaitanyaphalak chaitanyaphalak force-pushed the add-include-logs-funtionality branch 2 times, most recently from 7b81515 to d02e21f Compare November 17, 2021 00:41
@chaitanyaphalak chaitanyaphalak force-pushed the add-include-logs-funtionality branch from 9b8feff to fe2a7e4 Compare November 17, 2021 17:23
Copy link
Contributor

@dmitryax dmitryax left a comment

Choose a reason for hiding this comment

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

LGTM

@dmitryax dmitryax merged commit ff4e7ff into signalfx:main Nov 17, 2021
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.

2 participants