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

[BUG] Alert export errors prevent proper processing of events in eventengine #2225

Closed
lunkwill42 opened this issue Dec 3, 2020 · 0 comments · Fixed by #2226
Closed

[BUG] Alert export errors prevent proper processing of events in eventengine #2225

lunkwill42 opened this issue Dec 3, 2020 · 0 comments · Fixed by #2226
Assignees
Labels
Milestone

Comments

@lunkwill42
Copy link
Member

lunkwill42 commented Dec 3, 2020

Describe the bug
We are seeing errors in eventengine.log that indicate various errors in the alert export subsystem (mostly problems with actually serializing the alerts to JSON for output to the stream). It seems these errors stop further processing of the event in question by eventengine, meaning events aren't processed properly.

This only appears to apply if an export script is actually configured in eventengine.conf.

Expected behavior
Export of alerts should only be considered a side-effect of the processing of an event. Errors in this side effect should never prevent the event engine from proceeding with the internal processing of events.

Event engine should log these errors (so they can be tracked down), but ultimately ignore them and move on.

Environment (please complete the following information):

  • OS on the NAV server: Debian 10
  • NAV version installed: 5.1.0
  • Method of installation: from Debian package
@lunkwill42 lunkwill42 added the bug label Dec 3, 2020
@lunkwill42 lunkwill42 added this to the 5.1.1 milestone Dec 3, 2020
@lunkwill42 lunkwill42 self-assigned this Dec 3, 2020
@lunkwill42 lunkwill42 linked a pull request Dec 3, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant