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

NoMethodError when using in_monitor_agent with a unbuffered plugin #1364

Closed
shuji-koike opened this issue Dec 12, 2016 · 2 comments
Closed
Labels
bug Something isn't working v0.14

Comments

@shuji-koike
Copy link
Contributor

shuji-koike commented Dec 12, 2016

When using in_monitor_agent with out_null plugin a unbuffered plugin (which @buffer is nil), NoMethodError will occur each time http://0.0.0.0:24220/api/plugins is accessed.

Minimal config to reproduce the problem.

<source>
  @type monitor_agent
  bind 0.0.0.0
  port 24220
</source>

<match **>
  @type null
</match>

start fluentd and, kick monitor_agent.

curl http://0.0.0.0:24220/api/plugins

NoMethodError occurs.

2016-12-12 15:16:34 +0900 [warn]: plugin/in_monitor_agent.rb:341:rescue in block in get_monitor_info: unexpected error in monitoring plugins key="buffer_queue_length" plugin=Fluent::Plugin::NullOutput error_class=NoMethodError error="undefined method `queue' for nil:NilClass"
2016-12-12 15:16:34 +0900 [warn]: plugin/in_monitor_agent.rb:341:rescue in block in get_monitor_info: unexpected error in monitoring plugins key="buffer_total_queued_size" plugin=Fluent::Plugin::NullOutput error_class=NoMethodError error="undefined method `stage_size' for nil:NilClass"
@shuji-koike
Copy link
Contributor Author

shuji-koike commented Dec 16, 2016

Turns out that, not only out_null but some plugins witch @buffer is nil will cause the same problem.
I will update my issue title and comment.

@shuji-koike shuji-koike changed the title NoMethodError when using in_monitor_agent with out_null plugin NoMethodError when using in_monitor_agent with a unbuffered plugin Dec 16, 2016
@repeatedly
Copy link
Member

Patch is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working v0.14
Projects
None yet
Development

No branches or pull requests

3 participants