Supress further event-log message if ifAdminStatus or ifOperStatus is down #745

Merged
merged 1 commit into from Apr 7, 2015

Projects

None yet

3 participants

@f0o
Member
f0o commented Apr 7, 2015

See #742

@laf laf merged commit aca531b into librenms:master Apr 7, 2015

1 check passed

Scrutinizer No new issues
Details
@paulgear
Member
paulgear commented Apr 8, 2015

Question: why do we need $brk? Can't we just "break" on line 271?

@laf
Member
laf commented Apr 8, 2015

I guess we can! @f0o any issues with that on your side?

@f0o
Member
f0o commented Apr 8, 2015

Oh that's true, kinda leftover before I stripped it

@f0o f0o referenced this pull request Apr 8, 2015
Merged

Simplify #758

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment