]> granicus.if.org Git - icinga2/commitdiff
Modify windows check_update command to allow specifying thresholds. 6480/head
authorAndrew Jaffie <ajaffie@gmail.com>
Tue, 24 Jul 2018 20:00:47 +0000 (16:00 -0400)
committerMichael Insel <michael@insel.email>
Tue, 23 Oct 2018 09:28:43 +0000 (11:28 +0200)
refs #4720

doc/10-icinga-template-library.md
itl/command-plugins-windows.conf
plugins/check_update.cpp

index a080bb2750d4c2f98f2c8fac6f8ccd3f75176405..71b9a3a556bc9fe5b29e43914cccbe2da3853571 100644 (file)
@@ -1653,14 +1653,14 @@ Custom attributes:
 
 Name                | Description
 :-------------------|:------------
-update\_win\_warn   | **Optional**. If set, returns warning when important updates are available.
-update\_win\_crit   | **Optional**. If set, return critical when important updates that require a reboot are available.
+update\_win\_warn   | **Optional**. The warning threshold.
+update\_win\_crit   | **Optional**. The critical threshold.
 update\_win\_reboot | **Optional**. Set to treat 'may need update' as 'definitely needs update'. Please Note that this is true for almost every update and is therefore not recommended.
+ignore\_reboot      | **Optional**. Set to disable behavior of returning critical if any updates require a reboot.
 
 
-In contrast to most other plugins, the values of check_update's custom attributes do not set thresholds, but just enable/disable the behavior described in the table above.  
-It can be enabled/disabled for example by setting them to "true" or "false", "1" or "0" would also work.  
-Thresholds will always be "1".
+If a warning threshold is set but not a critical threshold, the critical threshold will be set to one greater than the set warning threshold.
+Unless the `ignore_reboot` flag is set, if any updates require a reboot the plugin will return critical.
 
 > **Note**
 >
index 1e48dc6b0f9526669908cab8316e50a5521b204c..c8dd1a6b0248e99907b3f7e1306c34b3b02e1746 100644 (file)
@@ -271,17 +271,21 @@ object CheckCommand "update-windows" {
 
        arguments = {
                "-w" = {
-                       set_if = "$update_win_warn$"
-                       description = "Warn if there are important updates available"
+                       value = "$update_win_warn$"
+                       description = "Number of updates to trigger a warning"
                }
                "-c" = {
-                       set_if = "$update_win_crit$"
-                       description = "Critical if there are important updates that require a reboot"
+                       value = "$update_win_crit$"
+                       description = "Number of updates to trigger a critical"
                }
                "--possible-reboot" = {
                        set_if = "$update_win_reboot$"
                        description = "Treat 'may need update' as 'definitely needs update'"
                }
+               "--no-reboot-critical" = {
+                       set_if = "$ignore_reboot$"
+                       description = "Do not automatically return critical if an update requiring reboot is present."
+               }
        }
 
        timeout = 5m
index 3543400c00e910c3e1f4a632eb9c0b61d7aefcd4..64621882f850039b020803e142059f00348ee384 100644 (file)
@@ -33,11 +33,11 @@ namespace po = boost::program_options;
 
 struct printInfoStruct
 {
-       bool warn{false};
-       bool crit{false};
+       int warn{0};
+       int crit{0};
        LONG numUpdates{0};
-       bool important{false};
-       bool reboot{false};
+       bool ignoreReboot{false};
+       int reboot{0};
        bool careForCanRequest{false};
 };
 
@@ -55,9 +55,10 @@ static int parseArguments(int ac, WCHAR **av, po::variables_map& vm, printInfoSt
                ("help,h", "Print help message and exit")
                ("version,V", "Print version and exit")
                ("debug,d", "Verbose/Debug output")
-               ("warning,w", "Warn if there are important updates available")
-               ("critical,c", "Critical if there are important updates that require a reboot")
+               ("warning,w", po::value<int>(), "Number of updates to trigger a warning.")
+               ("critical,c", po::value<int>(), "Number of updates to trigger a critical.")
                ("possible-reboot", "Treat \"update may need reboot\" as \"update needs reboot\"")
+               ("no-reboot-critical", "Do not automatically return critical if an update requiring reboot is present.")
                ;
 
        po::wcommand_line_parser parser(ac, av);
@@ -87,23 +88,20 @@ static int parseArguments(int ac, WCHAR **av, po::variables_map& vm, printInfoSt
                        L"\nAfter some time, it will then output a string like this one:\n\n"
                        L"\tUPDATE WARNING 8 | updates=8;1;1;0\n\n"
                        L"\"UPDATE\" being the type of the check, \"WARNING\" the returned status\n"
-                       L"and \"8\" is the number of important updates updates.\n"
+                       L"and \"8\" is the number of important updates.\n"
                        L"The performance data is found behind the \"|\", in order:\n"
                        L"returned value, warning threshold, critical threshold, minimal value and,\n"
-                       L"if applicable, the maximal value. Performance data will only be displayed when\n"
-                       L"you set at least one threshold\n\n"
+                       L"if applicable, the maximal value.\n\n"
                        L"An update counts as important when it is part of the Security- or\n"
                        L"CriticalUpdates group.\n"
                        L"Consult the msdn on WSUS Classification GUIDs for more information.\n"
                        L"%s' exit codes denote the following:\n"
                        L" 0\tOK,\n\tNo Thresholds were broken or the programs check part was not executed\n"
                        L" 1\tWARNING,\n\tThe warning, but not the critical threshold was broken\n"
-                       L" 2\tCRITICAL,\n\tThe critical threshold was broken\n"
+                       L" 2\tCRITICAL,\n\tThe critical threshold was broken or an update required reboot.\n"
                        L" 3\tUNKNOWN, \n\tThe program experienced an internal or input error\n\n"
-                       L"%s works different from other plugins in that you do not set thresholds\n"
-                       L"but only activate them. Using \"-w\" triggers warning state if there are not\n"
-                       L"installed and non-optional updates. \"-c\" triggers critical if there are\n"
-                       L"non-optional updates that require a reboot.\n"
+                       L"If a warning threshold is set but not a critical threshold, the critical\n"
+                       L"threshold will be set to one greater than the set warning threshold.\n\n"
                        L"The \"possible-reboot\" option is not recommended since this true for nearly\n"
                        L"every update."
                        , progName, progName);
@@ -113,10 +111,14 @@ static int parseArguments(int ac, WCHAR **av, po::variables_map& vm, printInfoSt
                std::cout << "Version: " << VERSION << '\n';
                return 0;
        }
-
-       printInfo.warn = vm.count("warning") > 0;
-       printInfo.crit = vm.count("critical") > 0;
+       if(vm.count("warning"))
+               printInfo.warn = vm["warning"].as<int>();
+       if (vm.count("critical"))
+               printInfo.crit = vm["critical"].as<int>();
+       else if (vm.count("warning"))
+               printInfo.crit = printInfo.warn + 1;
        printInfo.careForCanRequest = vm.count("possible-reboot") > 0;
+       printInfo.ignoreReboot = vm.count("no-reboot-critical") > 0;
 
        l_Debug = vm.count("debug") > 0;
 
@@ -131,10 +133,10 @@ static int printOutput(const printInfoStruct& printInfo)
        state state = OK;
        std::wstring output = L"UPDATE ";
 
-       if (printInfo.important)
+       if (printInfo.numUpdates >= printInfo.warn && printInfo.warn)
                state = WARNING;
 
-       if (printInfo.reboot)
+       if ((printInfo.reboot && !printInfo.ignoreReboot) || (printInfo.numUpdates >= printInfo.crit && printInfo.crit))
                state = CRITICAL;
 
        switch (state) {
@@ -148,8 +150,13 @@ static int printOutput(const printInfoStruct& printInfo)
                output.append(L"CRITICAL ");
                break;
        }
-
-       std::wcout << output << printInfo.numUpdates << L" | 'update'=" << printInfo.numUpdates << L";"
+       output.append(std::to_wstring(printInfo.numUpdates));
+       if (printInfo.reboot) {
+               output.append(L"; ");
+               output.append(std::to_wstring(printInfo.reboot));
+               output.append(L" NEED REBOOT ");
+       }
+       std::wcout << output << L" | 'update'=" << printInfo.numUpdates << L";"
                << printInfo.warn << L";" << printInfo.crit << L";0;" << '\n';
 
        return state;
@@ -214,15 +221,16 @@ static int check_update(printInfoStruct& printInfo)
                pUpdate->get_InstallationBehavior(&pIbehav);
                pIbehav->get_RebootBehavior(&updateReboot);
                if (updateReboot == irbAlwaysRequiresReboot) {
-                       printInfo.reboot = true;
+                       printInfo.reboot++;
                        if (l_Debug)
                                std::wcout << L"It requires reboot" << '\n';
                        continue;
                }
-               if (printInfo.careForCanRequest && updateReboot == irbCanRequestReboot)
+               if (printInfo.careForCanRequest && updateReboot == irbCanRequestReboot) {
                        if (l_Debug)
                                std::wcout << L"It requires reboot" << '\n';
-               printInfo.reboot = true;
+                       printInfo.reboot++;
+               }
        }
 
        if (l_Debug)