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

feat(dashboard): add possibility to display CRITICAL log lvl in dashboard #1337

Merged
merged 5 commits into from
Sep 17, 2024

Conversation

sgoral-splunk
Copy link
Contributor

Issue number: ADDON-74234

Summary

Added possibility to display messages logged with CRITICAL level in the error section in the monitoring dashboard.

Changes

Global config -> dashboard has a new setting: error_panel_log_lvl, which allows the user to define what log level should be displayed in the errors section of the monitoring panel

User experience

The user can define error log lvl in the global config. By default only ERROR

Checklist

If your change doesn't seem to apply, please leave them unchecked.

@sgoral-splunk sgoral-splunk marked this pull request as ready for review September 11, 2024 14:41
@sgoral-splunk sgoral-splunk requested review from a team as code owners September 11, 2024 14:41
Copy link
Contributor

@soleksy-splunk soleksy-splunk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix pipeline :D

@sgoral-splunk sgoral-splunk force-pushed the feat/critical_loglvl_error_dashboard branch from b970d8a to e62966b Compare September 17, 2024 13:58
@artemrys artemrys enabled auto-merge (squash) September 17, 2024 14:33
@artemrys artemrys merged commit d937986 into develop Sep 17, 2024
76 checks passed
@artemrys artemrys deleted the feat/critical_loglvl_error_dashboard branch September 17, 2024 15:58
@github-actions github-actions bot locked and limited conversation to collaborators Sep 17, 2024
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.

4 participants