Skip to content

Feature/add rule based detector to getting started#210

Merged
thorinaboenke merged 10 commits into
developmentfrom
feature/add_rule_based_detector_to_getting_started
Jul 13, 2026
Merged

Feature/add rule based detector to getting started#210
thorinaboenke merged 10 commits into
developmentfrom
feature/add_rule_based_detector_to_getting_started

Conversation

@thorinaboenke

@thorinaboenke thorinaboenke commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Task

#201

Description

  • Add a RuleDetector (detector-rule), running alongside NewValueDetector in the default docker-compose stack
  • parser sends parsed logs to both.
  • Ship it with R003 - CheckForExceptions, R004 - ErrorLevelFound enabled; R001 - TemplateNotFound, is added while doing the getting started tutorial.
  • alerts land in output-rule.%Y%m%d via a second / pair in the existing fluentout container (no second Fluentd container needed
  • Add one wildcard template to container/config/templates.txt so normal Nginx access-log lines parse successfully, keeping R001 quiet during normal traffic.
  • Update getting_started.md tutorial and docker-compose.md docs and demonstrating a deliberately unparseable log line

How Has This Been Tested?

manually executed the getting started tutorial

Checklist

  • This Pull-Request goes to the development branch.
  • I have successfully run prek locally.
  • I have added tests to cover my changes.
  • I have linked the issue-id to the task-description.
  • I have performed a self-review of my own code.

@thorinaboenke
thorinaboenke marked this pull request as draft July 13, 2026 12:59
@thorinaboenke
thorinaboenke marked this pull request as ready for review July 13, 2026 13:36
@thorinaboenke
thorinaboenke merged commit 0c3eeaf into development Jul 13, 2026
1 check passed
@thorinaboenke
thorinaboenke deleted the feature/add_rule_based_detector_to_getting_started branch July 13, 2026 13:45
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