Fluentd frontend #22
Open
Fluentd frontend #22
Conversation
This file contains hidden or 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
|
Can you remove or reset the cta_admin_dr_ls.py file? I think we have that where we need it right now and that's causing this not to merge. The fluentd Kubernetes files are good. |
scarletnorberg
force-pushed
the
fluentd_frontend
branch
from
44010d2 to
d5033db
Compare
ericvaandering
requested changes
Feb 29, 2024
| instance "#{ENV['CTA_INSTANCE']}" ###"#{ENV['ENSTORE_CONFIG_PORT']}" | ||
| </record> | ||
| </filter> | ||
| <filter showqueues> |
There was a problem hiding this comment.
Have you tried what is here: https://docs.fluentd.org/configuration/config-file#wildcards-expansions-and-other-tips
In other words, what if you named your sources cta_admin.tapepool, cta_admin.tapeall, etc
The way I read it you could have ONE filter <filter cta_admin.*> to add the instance and whatever other values you want.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Adding in some of the fluentd stuff
Also a mistake in the cloudprober code