timdaman / check_docker

Nagios plugin to check docker containers
GNU General Public License v3.0
152 stars 60 forks source link

Forced combination of no_ok and no_performance #83

Open knorx opened 2 years ago

knorx commented 2 years ago

There are two flags for compressing the amount of ok messages and disabling performance data. However, currently if you provide no_ok you automatically set no_performance as well:

   global no_performance
   no_performance = args.no_ok

I this a typo? If yes, I suggest to correct it and set args.no_performance. If it is not a typo I would anyways suggest to change it. Everyone who wants to combine the effects can provide both flags. The use case I have, where I want to see compressed OK's, but use performance data anyways, is currently not possible.