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

fix: prevent forceful exit on invalid command output #118

Conversation

abhishuraina
Copy link
Contributor

@abhishuraina abhishuraina commented Nov 12, 2024

Prevent integration to stop, on invalid command output. Also log error in case no matches are found, and log warning when more matches are found.

@abhishuraina abhishuraina requested a review from a team November 12, 2024 07:41
@abhishuraina abhishuraina force-pushed the NR-215396-bin-nri-nagios-exits-immediately-on-invalid-command-output branch from 6949665 to 637901d Compare November 12, 2024 10:01
@abhishuraina abhishuraina force-pushed the NR-215396-bin-nri-nagios-exits-immediately-on-invalid-command-output branch from 637901d to 99f00d4 Compare November 13, 2024 11:34
@abhishuraina abhishuraina requested a review from a team as a code owner November 13, 2024 11:34
@abhishuraina abhishuraina force-pushed the NR-215396-bin-nri-nagios-exits-immediately-on-invalid-command-output branch from 99f00d4 to c5b3a1a Compare November 14, 2024 06:56
src/main.go Outdated Show resolved Hide resolved
src/main.go Outdated Show resolved Hide resolved
src/main.go Outdated Show resolved Hide resolved
src/main.go Outdated Show resolved Hide resolved
@abhishuraina abhishuraina force-pushed the NR-215396-bin-nri-nagios-exits-immediately-on-invalid-command-output branch from c5b3a1a to a866f15 Compare November 14, 2024 10:47
@abhishuraina abhishuraina merged commit 08a5c6d into master Nov 18, 2024
13 checks passed
@abhishuraina abhishuraina deleted the NR-215396-bin-nri-nagios-exits-immediately-on-invalid-command-output branch November 18, 2024 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants