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
add consul plugin #530
add consul plugin #530
Conversation
inputs/consul/README.md
Outdated
|
||
```toml @sample.conf | ||
# Gather health check statuses from services registered in Consul | ||
[[inputs.consul]] |
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.
这个地方不是inputs.consul
|
||
## Example Output | ||
|
||
```text |
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.
如果指标样例有变化 也辛苦重新标注
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.
指标样例就是最新的
|
||
func (ins *Instance) Init() error { | ||
conf := api.DefaultConfig() | ||
|
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.
ins.Address 为空 返回 types.ErrInstancesEmpty。 username + password 或者 Token为空给出相应的提示
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.
consul 允许无密码和无token
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.
已处理
LGTM, thank you @yangliyl |
No description provided.