Fix Glue-Expansion on alerts #2522

Merged
merged 2 commits into from Dec 4, 2015

Projects

None yet

3 participants

@f0o
Member
f0o commented Nov 30, 2015

No description provided.

@f0o f0o Fix Glue-Expansion on alerts
b14d094
@f0o f0o added Bug Alerting labels Nov 30, 2015
@f0o f0o self-assigned this Nov 30, 2015
@f0o
Member
f0o commented Nov 30, 2015

Putting a Blocker here for last fixes

@f0o f0o Remove now obsoleted naive glue-checks
ae3f893
@f0o f0o removed their assignment Dec 1, 2015
@f0o
Member
f0o commented Dec 1, 2015

Releasing Blocker, Redundant glue is legitimate after all.

@f0o f0o closed this Dec 1, 2015
@f0o f0o reopened this Dec 1, 2015
@f0o
Member
f0o commented Dec 1, 2015

Please do extensive testing on this. We dont want alerting to explode.

@laf
Member
laf commented Dec 1, 2015

Testing.....

@laf
Member
laf commented Dec 2, 2015

👍 from me. Been running at home for 20+ hours. Alerts still triggering when they should. Using a group mapping as well.

@laf laf merged commit b47444a into librenms:master Dec 4, 2015

2 checks passed

Auto-Deploy Build finished. No test results found.
Details
Scrutinizer No new issues
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment