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.

Sign up for GitHub

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

Jump to bottom

IDO isn't updated on check_period removal #5037

Closed
Al2Klimov opened this issue Feb 24, 2017 · 3 comments
Closed

IDO isn't updated on check_period removal #5037

Al2Klimov opened this issue Feb 24, 2017 · 3 comments
Labels
area/db-ido Database output bug Something isn't working

Comments

@Al2Klimov
Copy link
Member

Icinga/icingaweb2#2697
@Al2Klimov
Copy link
Member Author

For the record: the config object hash changes so this is not the cause.

@dnsmichi dnsmichi added bug Something isn't working area/db-ido Database output labels Feb 28, 2017
@TheFlyingCorpse
Copy link
Contributor

TheFlyingCorpse commented Mar 7, 2017 •
edited
Loading

Tested:

  • New timeperiod used as expected.
  • Changed timeperiod used as expected.
  • From defined to no timeperiod, bug reproduced.

Looking at the "Checkable" for check_period it tries to return a raw result when it is defined.
I guess since it is not configured it will never try to return the data from this (GetCheckPeriod()), when what we want back for IDO is a reference to either 0 or NULL (the schema is default 0).

  • EventCommand has the same issue (it appears to do it the same way as TimePeriod).

@dnsmichi dnsmichi mentioned this issue Jan 18, 2018
Closed
6 tasks
@dnsmichi
Copy link
Contributor

Parent ticket for solving these problems is #6012.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/db-ido Database output bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@TheFlyingCorpse @dnsmichi @Al2Klimov