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.

Sign up for GitHub

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

Jump to bottom

Remove dependency on system binary from ping input plugin #6050

Merged
merged 17 commits into from
Jul 11, 2019
Merged

Remove dependency on system binary from ping input plugin #6050

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
ad024c6
WIP: add native ping
AlirieGray Apr 29, 2019
2d173e2
Attempt to record accurate ping time
glinton May 6, 2019
f097abb
Conditionally use system ping
glinton May 7, 2019
f2ec8e4
WIP
glinton May 13, 2019
14e7794
Update go-ping
glinton Jun 12, 2019
f319653
Enforce timeout
glinton Jun 12, 2019
ace14ac
Merge branch 'master' into feat/native-ping-new
glinton Jun 26, 2019
0272ec1
Use updated library interface
glinton Jun 26, 2019
42198df
Update dependency
glinton Jun 27, 2019
2317bbb
Update method logic and add test
glinton Jun 27, 2019
b734cab
fmt
glinton Jun 27, 2019
6563913
Update readme
glinton Jul 2, 2019
701c744
Merge branch 'master' into feat/native-ping-new
glinton Jul 2, 2019
05b0e4a
Fix typos in readme
glinton Jul 2, 2019
4a1dddf
Address feedback
glinton Jul 3, 2019
ecdf5f2
Set ttl only on supported platforms
glinton Jul 3, 2019
a5808b4
fmt tests
glinton Jul 3, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix typos in readme
  • Loading branch information
@glinton
glinton committed Jul 2, 2019
commit 05b0e4a679035f993ed54f3c31da9a67be8f0d8b
4 changes: 2 additions & 2 deletions plugins/inputs/ping/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,13 +77,13 @@ LimitNOFILE=4096
It is preferred that this plugin listen on privileged ICMP sockets. To do so, telegraf can either be run as the root user or the root user can add the capability to access raw sockets to telegraf by running the following commant:

```
setcap cap_net_raw=eip /home/test/telegraf
setcap cap_net_raw=eip /path/to/telegraf
```

Another option (doesn't work as well or in all circumstances) is to listen on unprivileged raw sockets (non-Windows only). The system group of the user running telegraf must be allowed to create ICMP Echo sockets. [See man pages icmp(7) for `ping_group_range`](http://man7.org/linux/man-pages/man7/icmp.7.html). On Linux hosts, run the following to give a group the proper permissions:

```
sudo sysctl -w net.ipv4.ping_group_range="GROUPID GROUPID"
sudo sysctl -w net.ipv4.ping_group_range="GROUP_ID_LOW GROUP_ID_HIGH"
```


Expand Down