Skip to content
This repository has been archived by the owner on Jan 15, 2019. It is now read-only.

[dev.icinga.com #3988] Host Status Cronk : Wrong number of Warning or Critical services #1113

Closed
icinga-migration opened this issue Apr 23, 2013 · 7 comments
Milestone

Comments

@icinga-migration
Copy link

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

Created by ossmon on 2013-04-23 10:09:36 +00:00

Assignee: jmosshammer
Status: Resolved (closed on 2013-04-26 09:58:18 +00:00)
Target Version: 1.9
Last Update: 2013-04-26 11:34:15 +00:00 (in Redmine)

Icinga Version: 1.8.4
Icinga Web Version: 1.9.0beta
IDO Version: 1.8.4
OS Version: RHEL 6.1
DB Type: MySQL
DB Version: 5.1.61
Browser Version: Firefox 11

In the host status cronk, the number of critical or warning services is always one
although the real number is greater than one when you click on the host and see all the services.

WrongNumberofCWServices.JPG

Attachments

Changesets

2013-04-26 09:30:23 +00:00 by jmosshammer 6984575

Fix grouping in LegacyApi not working

The Problem here was that disableAutoIdentifierFields must be
triggered to prevent our custom Doctrine implementation from
 adding the primary key (which crashes groupings)

 fixes #3988
@icinga-migration
Copy link
Author

Updated by jmosshammer on 2013-04-25 13:36:55 +00:00

  • Assigned to set to jmosshammer
  • Target Version set to 1.9

True, can reproduce this issue

@icinga-migration
Copy link
Author

Updated by ossmon on 2013-04-25 13:58:27 +00:00

There is also the problem when the user has custom variable restriction.
See #3578

@icinga-migration
Copy link
Author

Updated by jmosshammer on 2013-04-26 09:33:51 +00:00

Hey,

this should be fixed in the next branch, see 6984575

@icinga-migration
Copy link
Author

Updated by ossmon on 2013-04-26 09:42:52 +00:00

Hi Jannis,
i would test it, but i need also the patches of Markus (Tag feature/credentials-3715).
Can you make a merge in the next branch ?

@icinga-migration
Copy link
Author

Updated by jmosshammer on 2013-04-26 09:58:18 +00:00

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

Applied in changeset 6984575.

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2013-04-26 10:00:41 +00:00

@ossmon

Markus added more fixes and merged them to 'next' for your kind tests. thanks in advance.

https://git.icinga.org/?p=icinga-web.git;a=shortlog;h=refs/heads/next

@icinga-migration
Copy link
Author

Updated by ossmon on 2013-04-26 11:34:15 +00:00

It works fine now !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant