-
Notifications
You must be signed in to change notification settings - Fork 183
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
Do not attach empty 'node' metadata #1268
Conversation
Do you know why 'node' metadata is empty? |
I'm not quite sure, but I have a clue which I will want to investigate. There's a lot of 404 responses from Kubernetes API server which look a bit funny, like this:
These responses are not something new, they occur in v1.2.1 as well. |
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.
We'll need to cherry-pick this to https://github.com/SumoLogic/sumologic-kubernetes-fluentd
fluent-plugin-enhance-k8s-metadata/test/plugin/test_filter_enhance_k8s_metadata.rb
Outdated
Show resolved
Hide resolved
This was actually submitted by @astencel-sumo 🤦🏻 (SumoLogic/sumologic-kubernetes-fluentd#54) Thanks! |
cae2436
to
a2b5659
Compare
a2b5659
to
d706053
Compare
Some metrics might not have a specific |
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.
👍
After attaching
node=nil
metadata to a metrics record, the other pluginprometheus_format
fails with "undefined method `gsub' for nil:NilClass":