Skip to content
This repository has been archived by the owner on Oct 6, 2023. It is now read-only.

fix(flapping) start flapping on !OK state #523

Closed
wants to merge 1 commit into from

Conversation

UrBnW
Copy link
Contributor

@UrBnW UrBnW commented Jun 25, 2021

Hi,

Description

it's possible for flapping algorithm to flag hosts and services as starting to flap when they switch from a non-OK to OK state.
Which, from a user point of view, is rather strange, as a FLAPPINGSTART notification will be sent, with a OK state.
Would make more sense to receive a FLAPPINGSTART notification with a non-OK state.
Let's then make a host or service starting to flap only when it switches to a non-OK state.
This gives it a very last chance not to be flagged as flapping, avoiding as such unneeded notifications.

Type of change

  • Patch fixing an issue (non-breaking change)
  • New functionality (non-breaking change)
  • Breaking change (patch or feature) that might cause side effects breaking part of the Software
  • Updating documentation (missing information, typo...)

Target serie

  • 19.10.x
  • 20.04.x
  • 20.10.x
  • 21.04.x
  • 21.10.x (master)

How this pull request can be tested ?

Enable flapping detection, make a host or service to flap, and be sure it is flagged as flapping only when it switches to a non-OK state.

Many thanks 👍

@sonarqube-decoration
Copy link

SonarQube Quality Gate:
Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@omercier
Copy link
Contributor

Hi,
Thanks a lot for your contribution!
This PR will be processed by our dev team for our 21.10 release.
Regards,
Olivier

@sonarqube-decoration
Copy link

SonarQube Quality Gate:
Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@rem31
Copy link
Contributor

rem31 commented Sep 29, 2021

this pr is fixed by it #552

@rem31 rem31 closed this Sep 29, 2021
@UrBnW
Copy link
Contributor Author

UrBnW commented Sep 29, 2021

Hi @rem31, really good news, perfect, many thanks 👍

@UrBnW UrBnW deleted the flapstartnonok branch September 29, 2021 07:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants