-
Notifications
You must be signed in to change notification settings - Fork 78
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
Use http helper #136
Use http helper #136
Conversation
ref #48 |
f9795dc
to
e1285f4
Compare
Signed-off-by: Yuta Iwama <ganmacs@gmail.com>
Signed-off-by: Yuta Iwama <ganmacs@gmail.com>
Signed-off-by: Yuta Iwama <ganmacs@gmail.com>
e1285f4
to
08ecf64
Compare
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.
Looks reasonable for me.
lib/fluent/plugin/in_prometheus.rb
Outdated
@@ -55,6 +50,8 @@ def configure(conf) | |||
nil | |||
end | |||
@num_workers = sysconf && sysconf.workers ? sysconf.workers : 1 | |||
@secure = @transport_config.protocol == :tls || (@ssl && @ssl['enable']) | |||
require 'openssl' if @secure |
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.
Put it on top of module is better.
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.
fixed 52f0cb7
lib/fluent/plugin/in_prometheus.rb
Outdated
end | ||
|
||
def send_request_to_each_worker | ||
bind == '0.0.0.0' ? '127.0.0.1' : @bind |
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.
bind =
is missing?
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.
right.. 90035b6
context 'when extra_conf is used' do | ||
let(:config) do | ||
%[ | ||
type prometheus |
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.
use @type
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.
fixed b054290
driver.run(timeout: 1) | ||
end | ||
|
||
context 'old parameter given' do |
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.
This text is very similar to parent text. What is the different point?
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.
changed descriptions 9edca6f
Signed-off-by: Yuta Iwama <ganmacs@gmail.com>
Signed-off-by: Yuta Iwama <ganmacs@gmail.com>
Signed-off-by: Yuta Iwama <ganmacs@gmail.com>
Signed-off-by: Yuta Iwama <ganmacs@gmail.com>
Signed-off-by: Yuta Iwama <ganmacs@gmail.com>
@repeatedly how about this? |
done by #148 |
This PR should be merged after the release which includes fluent/fluentd#2787.