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 #5906] notification filters V2 #1393

Closed
icinga-migration opened this issue Apr 1, 2014 · 6 comments
Closed

[dev.icinga.com #5906] notification filters V2 #1393

icinga-migration opened this issue Apr 1, 2014 · 6 comments

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented Apr 1, 2014

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

Created by mfriedrich on 2014-04-01 15:05:59 +00:00

Assignee: gbeutner
Status: Resolved (closed on 2014-04-09 08:25:47 +00:00)
Target Version: 0.0.10
Last Update: 2014-04-09 08:25:47 +00:00 (in Redmine)


Drop 'notification_' prefix and '_filter' suffix. Remove or operator with constants, use array with constants.

types = [ Recovery, Flapping ] 
states = [ Ok, Warning ]

User: notification_types & notification_states.

  • make notification/user attributes an array
  • store the raw values, calculate value in OnConfigLoaded
  • warning on unknown filters
  • validate different states for hosts and services

Changesets

2014-04-07 12:00:18 +00:00 by gbeutner cea15e1

Implement new state and type filters.

Fixes #5906

2014-04-09 08:25:23 +00:00 by gbeutner 606834e

Rename the notification attributes.

Fixes #5906

Relations:

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 2, 2014

Updated by mfriedrich on 2014-04-02 12:14:31 +00:00

  • Status changed from New to Assigned
  • Assigned to set to gbeutner
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 2, 2014

Updated by gbeutner on 2014-04-02 19:56:59 +00:00

  • Assigned to changed from gbeutner to mfriedrich
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 6, 2014

Updated by mfriedrich on 2014-04-06 12:40:41 +00:00

  • Description updated
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 7, 2014

Updated by gbeutner on 2014-04-07 12:02:29 +00:00

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

Applied in changeset cea15e1.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 7, 2014

Updated by mfriedrich on 2014-04-07 21:35:03 +00:00

  • Status changed from Resolved to Assigned
  • Assigned to changed from mfriedrich to gbeutner
  • Done % changed from 100 to 90

renaming to 'types' and 'states' is missing.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Apr 9, 2014

Updated by gbeutner on 2014-04-09 08:25:47 +00:00

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

Applied in changeset 606834e.

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