Skip to content

check_process: add count of process(es) to the top syntax#371

Open
inqrphl wants to merge 2 commits intomainfrom
fix-check-process-syntax
Open

check_process: add count of process(es) to the top syntax#371
inqrphl wants to merge 2 commits intomainfrom
fix-check-process-syntax

Conversation

@inqrphl
Copy link
Copy Markdown
Contributor

@inqrphl inqrphl commented Apr 28, 2026

ok-synax is changed as well -> processes to process(es) for optional plularity

ensures that that even when check result is warning/critical it will include the count:

before: ./snclient run check_process "warning=count gt 500" WARNING - |'count'=9307;500;0;0

after: ./snclient run check_process "critical=count gt 500" CRITICAL - 9309 process(es) running |'count'=9309;0;500;0

Ahmet Oeztuerk added 2 commits April 28, 2026 23:53
ok-synax is changed as well -> processes to process(es) for optional plularity

ensures that that even when check result is warning/critical it will include the count:

before:
./snclient run check_process "warning=count gt 500"
WARNING -  |'count'=9307;500;0;0

after:
./snclient run check_process "critical=count gt 500"
CRITICAL - 9309 process(es) running  |'count'=9309;0;500;0
@inqrphl inqrphl force-pushed the fix-check-process-syntax branch from 8a0b061 to 7f8c70b Compare April 28, 2026 22:42
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.

1 participant