Skip to content

Conversation

@glensc
Copy link
Contributor

@glensc glensc commented Mar 3, 2016

@glensc glensc changed the title understang ping6 output from iputils package [check_ping] understang ping6 output from iputils package Mar 3, 2016
@waja waja added the bugfix label Jan 1, 2017
@waja waja added this to the 2.3 milestone Jan 1, 2017
@waja
Copy link
Member

waja commented Jan 4, 2017

@glensc do you think you can rebase this against master as we fixed travis there?

@glensc glensc force-pushed the pld/nagios-plugins-ping.patch branch from ada50e3 to 05ac8a9 Compare January 4, 2017 20:25
@glensc
Copy link
Contributor Author

glensc commented Jan 4, 2017

done!

Copy link
Contributor

@sni sni left a comment

Choose a reason for hiding this comment

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

Looks good

@riskersen
Copy link
Contributor

riskersen commented Jan 10, 2017

Could you please implement some tests for the new output strings?

@waja waja modified the milestones: 2.3, 2.4 Dec 15, 2020
@RincewindsHat RincewindsHat merged commit 85c8dcd into monitoring-plugins:master Jan 14, 2022
@glensc glensc deleted the pld/nagios-plugins-ping.patch branch October 21, 2022 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants