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.

Sign up for GitHub

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

Jump to bottom

50 analysis module first rule based analysis implemented #90

Merged
merged 14 commits into from
Nov 26, 2024
Merged

50 analysis module first rule based analysis implemented #90

merged 14 commits into from
Nov 26, 2024

Conversation

heskil
Copy link
Contributor

@heskil heskil commented Nov 26, 2024

  • added basic rule based analysis functionality
  • connected to backend alert api

Omega65536 and others added 12 commits November 22, 2024 23:50
@Omega65536
Start implementing the simple rule-based analyzer
ad34181 
Signed-off-by: Oliver Schnell <mail@oliver-schnell.de>
@Omega65536
Adhere to naming convention, fix tests, refactor SimpleRuleBasedAnalyzer
d39b997 
Signed-off-by: Oliver Schnell <mail@oliver-schnell.de>
@Omega65536
Update SimpleRuleBasedAnalyzer to send alerts to the backend
1577776 
Signed-off-by: Oliver Schnell <mail@oliver-schnell.de>
@Omega65536
Add basic test for the simpleRuleBasedAnalyzer
c7205eb 
Signed-off-by: Oliver Schnell <mail@oliver-schnell.de>
@Omega65536
Refactor and add more tests
2590222 
Signed-off-by: Oliver Schnell <mail@oliver-schnell.de>
@Omega65536
Raise exceptions when backend API call fails
d3af9d9 
Signed-off-by: Oliver Schnell <mail@oliver-schnell.de>
@Omega65536
Add limit option to the API to limit the number of triggered alerts, …
888bf8a 
…add more tests

Signed-off-by: Oliver Schnell <mail@oliver-schnell.de>
@heskil
added rule based analysis of diff backups
8904930 
Signed-off-by: Amelie Regl <mellyre42@gmail.com>
@heskil
added tests for diff analysis
44d882d 
Signed-off-by: Amelie Regl <mellyre42@gmail.com>
@heskil
removed debug prints
78e928a 
Signed-off-by: Amelie Regl <mellyre42@gmail.com>
@heskil
added basic rulebased analysis to inc backups
c087a4a 
Signed-off-by: Amelie Regl <mellyre42@gmail.com>
@heskil
small fix and removal of debug prints
b6bf530 
Signed-off-by: Amelie Regl <mellyre42@gmail.com>
@flo0852
Merge remote-tracking branch 'origin/dev' into 50-analysis-module-fir…
c37e6d6 
…st-rule-based-analysis-implemented
@flo0852
Workaround for floats
1e439ee 
Signed-off-by: Florian Oberndörfer <florian8751t@gmail.com>
flo0852
flo0852 approved these changes Nov 26, 2024
View reviewed changes
Copy link
Contributor

@flo0852 flo0852 left a comment

Choose a reason for hiding this comment

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

Looks really good. Some things I found, but we'll fix them next sprint

@heskil heskil merged commit 6b06808 into dev Nov 26, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@flo0852 flo0852 flo0852 approved these changes

@engelharddirk engelharddirk Awaiting requested review from engelharddirk

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@heskil @flo0852 @Omega65536