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

[dev.icinga.com #8829] Figure out why command validators are not triggered #2807

Closed
icinga-migration opened this Issue Mar 20, 2015 · 3 comments

Comments

Projects
None yet
1 participant
@icinga-migration
Member

icinga-migration commented Mar 20, 2015

This issue has been migrated from Redmine: https://dev.icinga.com/issues/8829

Created by mfriedrich on 2015-03-20 12:43:35 +00:00

Assignee: gbeutner
Status: Resolved (closed on 2015-03-20 14:55:03 +00:00)
Target Version: 2.3.3
Last Update: 2015-03-20 14:55:53 +00:00 (in Redmine)

Icinga Version: v2.3.0-74-g8282725
Backport?: Already backported
Include in Changelog: 1

They are implemented inside command.cpp but are not executed.

icinga2 - The Icinga 2 network monitoring daemon (version: v2.3.0-74-g8282725; debug)

Copyright (c) 2012-2015 Icinga Development Team (https://www.icinga.org)
License GPLv2+: GNU GPL version 2 or later 
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Application information:
  Installation root: /usr
  Sysconf directory: /etc
  Run directory: /var/run
  Local state directory: /var
  Package data directory: /usr/share/icinga2
  State path: /var/lib/icinga2/icinga2.state
  Objects path: /var/cache/icinga2/icinga2.debug
  Vars path: /var/cache/icinga2/icinga2.vars
  PID path: /var/run/icinga2/icinga2.pid

System information:
  Operating system: Linux
  Operating system version: 3.18.8-201.fc21.x86_64
  Architecture: x86_64
  Distribution: "Fedora release 21 (Twenty One)"

Example with failing values but going through:

object CheckCommand "http_foo" {
        import "plugin-check-command"
        import "ipv4-or-ipv6"

        //command = [ PluginDir + "/check_http" ]
        command = PluginDir + "/check_http"

        arguments = {
                "-H" = "$http_vhost$"
                "-I" = "$http_address$"
                "-u" = "$http_uri$"
                "-p" = "$http_port"
                "-S" = {
                        set_if = "$http_ssl"
                }
                "--sni" = {
                        set_if = "$http_sni$"
                }
        }
}


object Host "foo" {
        check_command = "http_foo"
}

Changesets

2015-03-20 14:49:55 +00:00 by (unknown) e8cee8d

Allow using more than one %validator rule for the same type

fixes #8829

2015-03-20 14:55:13 +00:00 by (unknown) 86a1bbf

Allow using more than one %validator rule for the same type

fixes #8829
@icinga-migration

This comment has been minimized.

Show comment
Hide comment
@icinga-migration

icinga-migration Mar 20, 2015

Member

Updated by gbeutner on 2015-03-20 14:48:00 +00:00

  • Assigned to changed from mfriedrich to gbeutner
Member

icinga-migration commented Mar 20, 2015

Updated by gbeutner on 2015-03-20 14:48:00 +00:00

  • Assigned to changed from mfriedrich to gbeutner
@icinga-migration

This comment has been minimized.

Show comment
Hide comment
@icinga-migration

icinga-migration Mar 20, 2015

Member

Updated by Anonymous on 2015-03-20 14:55:03 +00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 0 to 100

Applied in changeset e8cee8d.

Member

icinga-migration commented Mar 20, 2015

Updated by Anonymous on 2015-03-20 14:55:03 +00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 0 to 100

Applied in changeset e8cee8d.

@icinga-migration

This comment has been minimized.

Show comment
Hide comment
@icinga-migration

icinga-migration Mar 20, 2015

Member

Updated by gbeutner on 2015-03-20 14:55:53 +00:00

  • Target Version changed from 2.4.0 to 2.3.3
  • Backport? changed from TBD to Yes
Member

icinga-migration commented Mar 20, 2015

Updated by gbeutner on 2015-03-20 14:55:53 +00:00

  • Target Version changed from 2.4.0 to 2.3.3
  • Backport? changed from TBD to Yes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment