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

Ctor 941 cache double check #237

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Conversation

tanguyvda
Copy link
Contributor

Description

if for some reason we can't find data for a host or service in the broker cache, this PR adds the possibility to also check in the database.

this will create "a best effort" cache that is mostly here to avoid dropping events because we can't find either the host name or the service description in the broker cache.

obviously this feature is disabled by default. Because:

  • broker cache should be a reliable source of information
  • it requires a luasql dependency (available on debian but not on EL type of OS)

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

Target serie

  • 22.10.x
  • 23.04.x
  • 23.10.x
  • 24.04.x
  • master

How this pull request can be tested ?

hard to test because it requires a failed broker cache

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant