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

Add nrpe v3 parameter in ITL #9296

Merged
merged 4 commits into from
Jun 22, 2022
Merged

Add nrpe v3 parameter in ITL #9296

merged 4 commits into from
Jun 22, 2022

Conversation

Napsty
Copy link
Contributor

@Napsty Napsty commented Mar 25, 2022

This adds the -3 parameter to the nrpe check command.
This parameter allows to transmit much more data through NRPE than NRPEv2.

From the check_nrpe 4.x help output:

 -3, --v3-packets-only        Only use version 3 packets, not version 4

@cla-bot cla-bot bot added the cla/signed label Mar 25, 2022
Copy link
Member

@Al2Klimov Al2Klimov left a comment

Choose a reason for hiding this comment

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

And the documentation?

@Al2Klimov Al2Klimov added this to the 2.14.0 milestone Mar 28, 2022
@Al2Klimov Al2Klimov requested a review from yhabteab March 28, 2022 14:36
@Napsty
Copy link
Contributor Author

Napsty commented Mar 30, 2022

@Al2Klimov I adjusted the descriptions as the original descriptions might lead to confusion when using NRPEv4.

@Al2Klimov Al2Klimov requested a review from yhabteab June 14, 2022 07:58
Copy link
Member

@yhabteab yhabteab left a comment

Choose a reason for hiding this comment

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

LGTM!

@Al2Klimov Al2Klimov merged commit ff6cdc2 into Icinga:master Jun 22, 2022
yhabteab pushed a commit that referenced this pull request Sep 5, 2022
Add nrpe v3 parameter in ITL
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.

None yet

3 participants