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

[dev.icinga.com #5324] Allow exclude (!) to work for regexp matches for hosts #1406

Closed
icinga-migration opened this issue Dec 13, 2013 · 3 comments
Labels
bug
Milestone

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented Dec 13, 2013

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

Created by liedekef on 2013-12-13 11:06:46 +00:00

Assignee: (none)
Status: Resolved (closed on 2014-01-03 17:10:10 +00:00)
Target Version: 1.11
Last Update: 2014-01-03 17:10:10 +00:00 (in Redmine)

Icinga Version: 1.10.2
OS Version: CentOS 6.4

Based on #3720 , I just found a small issue (just got around to testing the new release 1.10-2).
It seems a small oversight was being done for hosts and "!" for regex, see the unified diff in attach. I also corrected some forgotten use_regexp=false statements, but it shouldn't interfere.
The unified diff should apply cleanly against the latest source.

Attachments

Changesets

2014-01-03 15:17:53 +00:00 by (unknown) f4c6e85

Allow exclude (!) to work for regexp matches for hosts (Franky Van Liedekerke)

Refs #5324

2014-01-03 15:22:25 +00:00 by (unknown) 7650fc6

Merge branch 'fix/regexp-exclude-hosts-5324' into next

Fixes #5324

2014-01-03 17:09:00 +00:00 by (unknown) cd1d7a0

Update Changelog.

Refs #5272
Refs #5324
Refs #5412
Refs #5406
@icinga-migration
Copy link
Member Author

@icinga-migration icinga-migration commented Dec 13, 2013

Updated by mfriedrich on 2013-12-13 12:36:06 +00:00

  • Target Version set to 1.10.3
@icinga-migration
Copy link
Member Author

@icinga-migration icinga-migration commented Jan 3, 2014

Updated by mfriedrich on 2014-01-03 15:16:48 +00:00

  • Target Version changed from 1.10.3 to 1.11
@icinga-migration
Copy link
Member Author

@icinga-migration icinga-migration commented Jan 3, 2014

Updated by Anonymous on 2014-01-03 17:10:10 +00:00

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

Applied in changeset 7650fc6.

@icinga-migration icinga-migration added this to the 1.11 milestone Jan 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.