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

[dev.icinga.com #13037] Add ability to increase check_timeout for Services #546

Closed
icinga-migration opened this issue Nov 3, 2016 · 7 comments
Assignees
Milestone

Comments

@icinga-migration
Copy link

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

Created by mwaldmueller on 2016-11-03 13:08:19 +00:00

Assignee: (none)
Status: New
Target Version: 1.3.1
Last Update: 2016-11-30 08:46:54 +00:00 (in Redmine)


Add ability to increase check_timeout for Services to overwrite the default CheckCommand timeout of 60 seconds.

@icinga-migration
Copy link
Author

Updated by tgelf on 2016-11-03 13:13:49 +00:00

  • Target Version set to 1.3.0

The attribute is only available on commands, not on services. But this could easily be solved by using a custom field, defined at a command level. Just: the timeout property doesn't allow for vars right now in the Director.

@icinga-migration
Copy link
Author

Updated by mwaldmueller on 2016-11-03 13:15:13 +00:00

It's also available for Services: http://docs.icinga.org/icinga2/latest/doc/module/icinga2/chapter/object-types#objecttype-service

@icinga-migration
Copy link
Author

Updated by tgelf on 2016-11-03 13:17:23 +00:00

mwaldmueller wrote:

It's also available for Services: http://docs.icinga.org/icinga2/latest/doc/module/icinga2/chapter/object-types#objecttype-service

I had a look there, checked again - still cannot see it

@icinga-migration
Copy link
Author

Updated by mwaldmueller on 2016-11-03 13:19:56 +00:00

Please search for "check_timeout"...

The problem is that I don't want to modify the CheckCommands from the ITL, so I it's better to set the timeout for the Service.

@icinga-migration
Copy link
Author

Updated by tgelf on 2016-11-03 15:38:05 +00:00

Sorry, had trouble with the search starting at the wrong place - you're right!

@icinga-migration
Copy link
Author

Updated by tgelf on 2016-11-30 08:46:54 +00:00

  • Target Version changed from 1.3.0 to 1.3.1

@widhalmt
Copy link
Member

+1 thanks for dealing with this. I hit this,too.

@Thomas-Gelf Thomas-Gelf self-assigned this Jul 13, 2017
Thomas-Gelf added a commit that referenced this issue Jul 13, 2017
Thomas-Gelf added a commit that referenced this issue Jul 14, 2017
Thomas-Gelf added a commit that referenced this issue Jul 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants