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

Exclude aggregate interface 'all' thas has missing error/drop fields #2385

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hpaluch-pil
Copy link

Hello!
When I tried stock "net_alert" examples for Telegraf I found spurious errors in my /var/log/kapacitor/kapacitor.log:

lvl=error msg="cannot perform derivative" service=kapacitor task_master=main task=net_alert_stream
node=derivative3 err="field is the wrong type" field=err_out type=<nil>

After a bit of debug it turned out that it is caused by Telegraf inserting data for virtual interface all that is
missing some fields, including these error and drop fields required by these Tasks.

So excluding interface all from results seems to fix this problem.

Please let me know what you think about this issue and proposed solution.

Required for all non-trivial PRs
  • Rebased/mergable
  • Tests pass
  • CHANGELOG.md updated
  • Sign CLA (if not already signed)

@docmerlin
Copy link
Contributor

@hpaluch-pil Hey, man, this looks great, but I can't accept it without you signing the CLA :-( Without that, our lawyers won't let us accept code from outside PRs.

@docmerlin docmerlin self-requested a review September 4, 2020 19:18
Copy link
Contributor

@docmerlin docmerlin left a comment

Choose a reason for hiding this comment

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

@hpaluch-pil hey could you sign the CLA :-( Without that, our lawyers won't let us accept code from outside PRs.

@hpaluch-pil
Copy link
Author

Hi @docmerlin !
I can sign CLA myself. But I don't dare to push my employer to sign CCLA, because it may cause substantial legal obligations and/or risks.

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