Skip to content

Feature/detalizing logs#600

Merged
litleleprikon merged 20 commits into
moira-alert:masterfrom
androndo:feature/detalizing_logs
Feb 25, 2021
Merged

Feature/detalizing logs#600
litleleprikon merged 20 commits into
moira-alert:masterfrom
androndo:feature/detalizing_logs

Conversation

@androndo

@androndo androndo commented Feb 1, 2021

Copy link
Copy Markdown
Contributor

PR Summary

Detalizing logs:

  • changed old fields from message text (like [trigger_id], etc) to structured fileds
  • api: added http-fields
  • notifier: added configuring log level by subscription and contact
  • checker: added configuring log level by trigger
  • docs changes here
    Relates Improve logging #441

@androndo
androndo requested a review from a team as a code owner February 1, 2021 08:52
@coveralls

coveralls commented Feb 3, 2021

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.05%) to 74.683% when pulling 912de62 on androndo:feature/detalizing_logs into 94f1dda on moira-alert:master.

Comment thread api/middleware/logger.go
Comment thread checker/check.go Outdated
Comment thread checker/check.go Outdated
Comment thread checker/check.go Outdated
Comment thread checker/check.go Outdated
Comment thread notifier/scheduler.go Outdated
Comment thread notifier/scheduler.go Outdated
Comment thread notifier/scheduler.go Outdated
Comment thread notifier/scheduler.go Outdated
Comment thread notifier/scheduler.go Outdated
@androndo
androndo force-pushed the feature/detalizing_logs branch 3 times, most recently from 06c0fba to 2157a4b Compare February 12, 2021 14:29
litleleprikon
litleleprikon previously approved these changes Feb 23, 2021
@androndo
androndo force-pushed the feature/detalizing_logs branch from 5d6fc29 to 912de62 Compare February 25, 2021 12:35
@litleleprikon
litleleprikon merged commit 7e7af22 into moira-alert:master Feb 25, 2021
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.

3 participants