[dev.icinga.com #4187] Notification overview

[icinga-migration]

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

Created by elippmann on 2013-05-15 15:00:51 +00:00

Assignee: (none) Status: Resolved (closed on 2013-08-16 15:44:21 +00:00) Target Version: 1.0-7 Last Update: 2013-08-19 08:43:21 +00:00 (in Redmine)

---

As a developer I am required to provide at least all the information about notifications that the classic webinterface of icinga 1.x does. Further I have to define sort columns and possible filters on top. Reference is https://wiki.icinga.org/display/cranberry/%27Classic%27+Views. I do not have to copy the column format of the classic ui but can combine information when it's reasonable.

Changesets

2013-08-16 14:23:14 +00:00 by jmeyer 3c2122515afa95ad7e808b0f9f0602efc107d8c9

Basic structure for the notification overview

refs #4187

2013-08-16 14:23:14 +00:00 by jmeyer f89d641b0676a36c45c197094632610de945157b

Add notification queries and display notification details

The notification queries to fetch all required details are now implemented
and are used to display data in the overview. Still missing is the host/service
detail popup and some quicklinks. (Contact, Notification command)

refs #4187

2013-08-16 14:23:14 +00:00 by jmeyer baaf5b81d9ef49580574a17cfc7518b2beb0e58d

Add mouse-over effect and host/service detail popup

refs #4187

2013-08-16 14:23:15 +00:00 by jmosshammer 3c56f5c53aff142d601a6ce8c7b55647a208ce6f

Remove qUrl, use ListController::query, proper default sorting

refs #4187

2013-08-16 14:23:15 +00:00 by elippmann f4b8b369a8aa826b050c2db26151a88395c393b3

StatusQuery: Use pipes as concat

PostgreSQL does not support CONCAT()

refs #4187

2013-08-16 14:23:15 +00:00 by elippmann e5f3a063fb49d74709c2aec1fc3a3f13f5582be1

NotificationQuery: Fix PostgreSQL icompatibility

refs #4187

2013-08-16 14:32:31 +00:00 by elippmann 83fb198cd28aefcff6ca8cf522ffc2ea6240c0d2

CS: Fix violations in MonitoringProperties, NotificationView and StatusQuery

refs #4187

2013-08-16 15:14:55 +00:00 by jmosshammer 26402f1c0a5b805c28ac49bf6ea6766375062b3f

Merge branch 'feature/notification-overview-4187'

resolves #4187

[icinga-migration]

Updated by mhein on 2013-07-29 12:01:42 +00:00

• Target Version set to 140

[icinga-migration]

Updated by mhein on 2013-07-29 12:53:22 +00:00

• Estimated Hours set to 8

[icinga-migration]

Updated by mhein on 2013-07-29 15:23:44 +00:00

• Parent Id set to 4076

[icinga-migration]

Updated by mhein on 2013-07-29 15:24:30 +00:00

• Parent Id deleted 4076

[icinga-migration]

Updated by elippmann on 2013-08-07 13:22:16 +00:00

• Target Version changed from 140 to 1.0-7

[icinga-migration]

Updated by elippmann on 2013-08-07 13:29:56 +00:00

• Status changed from New to Assigned
• Assigned to set to mjentsch

[icinga-migration]

Updated by elippmann on 2013-08-07 13:36:39 +00:00

• Assigned to changed from mjentsch to jmeyer

[icinga-migration]

Updated by icinga-kanban on 2013-08-09 09:26:26 +00:00

Build !#510 triggered by commit 156e12a passed successfully.

Branch: origin/feature/notification-overview-4187 Author: Johannes Meyer

[icinga-migration]

Updated by icinga-kanban on 2013-08-09 13:49:29 +00:00

Build !#511 triggered by commit f5969e7 passed successfully.

Branch: origin/feature/notification-overview-4187 Author: Johannes Meyer

[icinga-migration]

Updated by icinga-kanban on 2013-08-09 14:06:54 +00:00

Build !#512 triggered by commit 8602181 passed successfully.

Branch: origin/feature/notification-overview-4187 Author: Johannes Meyer

[icinga-migration]

Updated by jmeyer on 2013-08-09 14:25:06 +00:00

• Status changed from Assigned to 8
• Assigned to deleted jmeyer
• Done % changed from 0 to 100

Sort direction cannot be changed currently as this seems to be ignored by the db backend in some way. (It is permanently ASC.) This seems to also affect other views so I just ignore this as well..

The quicklinks mentioned in the second revision are still missing, but I do not know whether we actually have these views available.

I also did not have the time to write any tests, so coverage might be a reason to reject this feature but I leave this as a exercise for someone else. :D

[icinga-migration]

Updated by jmosshammer on 2013-08-12 09:11:46 +00:00

• Status changed from 8 to Assigned
• Assigned to set to jmosshammer

Looks good, few issues (solved in branch)

• Sort '-1' does not make sense, asc and desc works
• Big switch/case blocks in views are not very clean, moved to helper
• Removed qUrl references
• Used $this->query instead of direct select on backend in ListController, as this one handles parameter handling etc.

[icinga-migration]

Updated by jmosshammer on 2013-08-12 09:12:20 +00:00

• Status changed from Assigned to 8

[icinga-migration]

Updated by elippmann on 2013-08-12 13:21:23 +00:00

• Assigned to changed from jmosshammer to elippmann

[icinga-migration]

Updated by elippmann on 2013-08-12 13:40:52 +00:00

• Assigned to changed from elippmann to jmosshammer
• Done % changed from 100 to 50

Hey guys,

with pgsql as backend this view does not work unfortunately :(

[icinga-migration]

Updated by elippmann on 2013-08-16 14:16:22 +00:00

• Status changed from 8 to Assigned
• Assigned to changed from jmosshammer to elippmann
• Done % changed from 50 to 100

[icinga-migration]

Updated by icinga-kanban on 2013-08-16 14:19:44 +00:00

Build !#555 triggered by the commits ddac860, 76e8b19 passed successfully.

Branch: origin/feature/notification-overview-4187 Author: Eric Lippmann

[icinga-migration]

Updated by elippmann on 2013-08-16 14:31:09 +00:00

• Status changed from Assigned to 8
• Assigned to deleted elippmann

[icinga-migration]

Updated by jmosshammer on 2013-08-16 15:44:22 +00:00

• Status changed from 8 to Resolved