-
Notifications
You must be signed in to change notification settings - Fork 10
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): Add Fluent-bit to K8tls #233
Merged
anurag-rajawat
merged 2 commits into
5GSEC:main
from
JonesJefferson:feat/issue-218-add-fluent-bit-to-k8tls
Aug 14, 2024
Merged
(feat): Add Fluent-bit to K8tls #233
anurag-rajawat
merged 2 commits into
5GSEC:main
from
JonesJefferson:feat/issue-218-add-fluent-bit-to-k8tls
Aug 14, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
df2ba29
to
127ae54
Compare
127ae54
to
be0003b
Compare
Signed-off-by: Jones Jefferson <[email protected]>
be0003b
to
844868e
Compare
shivaccuknox
reviewed
Aug 9, 2024
90986c8
to
48e0e05
Compare
Signed-off-by: Jones Jefferson <[email protected]>
48e0e05
to
c67b00c
Compare
anurag-rajawat
approved these changes
Aug 14, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, rebase and we're good to merge it.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Changes to helm of nimbus-k8tls and numbus-k8tls adapter to support sending k8tls reports using fluent-bit. It will be sent to stdout of fluent-bit by default. If elasticsearch is enabled in helm values file, then it will be sent to elasticsearch.
Fixes #223
Does this PR introduce a breaking change?
Checklist
<type>: <description>
conventionAdditional information for reviewer
Mention if this PR is part of any design or a continuation of previous PRs
Fluent bit logs
Working example
https://www.loom.com/share/e99311c4757444f4aafe22ea372cbf62?sid=5113fd0b-ba9e-4b49-9580-8aec4abc51c3