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

Update README for inputs.ping with correct cmd for native ping on Linux #8787

Merged
merged 3 commits into from
Feb 8, 2021

Conversation

sspaink
Copy link
Contributor

@sspaink sspaink commented Feb 2, 2021

Required for all PRs:

  • Associated README.md updated.

Fix: #8778

Update README to clarify the sysctl command you need to run to enable using native ping on Linux.

Copy link
Contributor

@telegraf-tiger telegraf-tiger bot left a comment

Choose a reason for hiding this comment

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

🤝 ✅ CLA has been signed. Thank you!

@srebhan srebhan self-assigned this Feb 3, 2021
@helenosheaa helenosheaa added the fix pr to fix corresponding bug label Feb 4, 2021
Copy link
Member

@srebhan srebhan left a comment

Choose a reason for hiding this comment

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

Perfect thx!

@srebhan srebhan added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label Feb 8, 2021
@sspaink sspaink merged commit 90392e1 into influxdata:master Feb 8, 2021
ssoroka pushed a commit that referenced this pull request Feb 17, 2021
…ux (#8787)

* Update readme  to enable native ping

* Provide more information on the numbers being set

* reference for ping_group_range already exists

(cherry picked from commit 90392e1)
arstercz pushed a commit to arstercz/telegraf that referenced this pull request Mar 5, 2023
…ux (influxdata#8787)

* Update readme  to enable native ping

* Provide more information on the numbers being set

* reference for ping_group_range already exists
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix pr to fix corresponding bug ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

inputs.ping failures after updating from 1.17.0 to 1.17.2
3 participants