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

Icinga CheckCommand defaults-file not required #6

Open
trickert76 opened this issue Oct 18, 2021 · 0 comments
Open

Icinga CheckCommand defaults-file not required #6

trickert76 opened this issue Oct 18, 2021 · 0 comments

Comments

@trickert76
Copy link

There is an issue within the example Icinga check command in combination with the check itself.

The definition says, that proxysql_defaultfile is required. But when the defaultfile is given, the proxysql_user and proxysql_password is completly ignored. My default doesn't contain a user+pass.

I'm not sure, what is better here.

  • remove the required flag in the definition
  • or rewrite the Python script

In my case, I've just remove the flag.

And a minor issue. Maybe change the env python to env python3 because Python2 is EOL since nearly 2 years.

Also - in my case of latest Icinga2 on Debian-11, the check is already defined within the ITL, so it isn't necessary to redeclare the checkcommand.

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

No branches or pull requests

1 participant