Skip to content
This repository has been archived by the owner. It is now read-only.

[dev.icinga.com #2933] Improve the "open problems" grid #857

Closed
icinga-migration opened this issue Aug 2, 2012 · 8 comments
Closed

[dev.icinga.com #2933] Improve the "open problems" grid #857

icinga-migration opened this issue Aug 2, 2012 · 8 comments
Labels
Milestone

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented Aug 2, 2012

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

Created by tgelf on 2012-08-02 16:44:48 +00:00

Assignee: mhein
Status: Resolved (closed on 2012-08-03 10:58:09 +00:00)
Target Version: 1.8
Last Update: 2012-08-08 09:25:16 +00:00 (in Redmine)


I did a few more improvements. Please note that the patch in #2932 is required to make this one fit nicely. It does as follows:

  • Replace the host- and service-output columns with just one single check-output column. The former version was ugly even on Full-HD screens.
  • Hide the last host- and service-check columns per default, they are not of interest here
  • Add a last-state-change column. This way you can see how long your issues are lasting!
  • Default sort order is now last_state_change DESC
  • Removed two columns, one for the service and one for the host icon. WTF?!
  • Added a single column showing either the host or the service icon

Well, that's all for now.

Cheers,
Thomas Gelf

Attachments

Changesets

2012-08-03 10:53:37 +00:00 by mhein 4099731

Compact version of open problem view, thanks TG (fixes #2933)

2012-08-03 10:54:23 +00:00 by mhein f943576

Compact version of open problem view, thanks TG (fixes #2933)

2012-08-07 13:39:14 +00:00 by mhein 7e8d80c

Applied sort fix for OpenProblems view (fixes #2933)

2012-08-07 13:40:33 +00:00 by mhein 31ef980

Applied sort fix for OpenProblems view (fixes #2933)
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Aug 3, 2012

Updated by mfriedrich on 2012-08-03 10:57:39 +00:00

  • Status changed from New to Assigned
  • Assigned to set to mhein
  • Target Version set to 1.8
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Aug 3, 2012

Updated by mhein on 2012-08-03 10:58:09 +00:00

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

Applied in changeset f943576.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Aug 6, 2012

Updated by ossmon on 2012-08-06 07:15:12 +00:00

I've downloaded the actual icinga-web git master today (4c3a8...).
I can verify all the new features but the default sorting with last_state_change doesn't work.
When the cronk is loaded, there is no sorting.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Aug 6, 2012

Updated by tgelf on 2012-08-06 07:28:36 +00:00

Could you try if things change when you reset your profile in your preferences?

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Aug 6, 2012

Updated by ossmon on 2012-08-06 11:53:13 +00:00

i've resetted my profile in the preferences. No effect.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Aug 7, 2012

Updated by tgelf on 2012-08-07 13:27:34 +00:00

Hi ossmon!

app/modules/Api/config/views/host.xml, line 188 (by the end of the query defined for "TARGET_OPEN_PROBLEMS): could you try to add 'ORDER BY LAST_STATE_CHANGE DESC' before closing CDATA? Also please don't forget to bin/clearcache.sh and to reset your profile (probably not required, closing and re-opening the cronk shall suffice).

Cheers,
Thomas

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Aug 8, 2012

Updated by ossmon on 2012-08-08 08:45:12 +00:00

Hi Thomas,
with the ORDER Statement. It works !
Jacques

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Aug 8, 2012

Updated by tgelf on 2012-08-08 09:25:16 +00:00

Hi Jacques,

that's great! Icinga-Web 1.7.2 has already been published for download, you'll find the complete fix there.

Cheers,
Thomas

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
1 participant
You can’t perform that action at this time.