Citrix Instance State - Configure VMToolsState

[Constey (Constantin Lotz)]

Hi there,

im looking for a way to configure an ignore for the VMToolsState of the Citrix Instance State check. (check_mk-citrix_state)

Currently this option is missing in the console, or do I miss sth?

Kind regards,
Constantin

[aeckstein (Andre Eckstein)]

Hi Constantin,

yes, it seems that there are no paremeters configurable (checked for 2.3) :

def _parameter_valuespec_citrix_state():
              return Dictionary(
                  elements=[
                      (
                          "registrationstate",
                          Dictionary(
                              title=_("Interpretation of Registration States"),
                              elements=[
                                  ("Unregistered", MonitoringState(title=_("Unregistered"), default_value=2)),
                                  ("Initializing", MonitoringState(title=_("Initializing"), default_value=1)),
                                  ("Registered", MonitoringState(title=_("Registered"), default_value=0)),
                                  ("AgentError", MonitoringState(title=_("Agent Error"), default_value=2)),
                              ],
                              optional_keys=False,
                          ),
                      )
                  ],

[Constey (Constantin Lotz)]

Okay thanks, so i was not blind. I’ve just added a pull request for it

[c3rberus (MD)]

@Constey thanks for contributing the pull request, I found it at:

Update citrix_state.py - added vmtoolsstate config by Constey · Pull Request #689 · Checkmk/checkmk

Based on the bot comments “re-opening a PR which was force-rebased, is not possible”, you may want to re-submit the PR, no idea why it was closed on you.

Do you know what file needs to be modified for this to show up in WATO? Is this citrix_state.py inside /omd/sites/nagios/lib/python3/cmk/gui/plugins/wato/check_parameters path?

[Constey (Constantin Lotz)]

The modification is this one:

Comparing Checkmk:master...Constey:patch-1 · Checkmk/checkmk

feel free to do again a pull request, dont know why it was closed.

createt it again: Update citrix_state.py - added vmtoolsstate config by Constey · Pull Request #784 · Checkmk/checkmk · GitHub