Skip to content

Add monitoring solution using Grafana#59

Merged
lamr02n merged 219 commits intomainfrom
add-grafana
Feb 20, 2025
Merged

Add monitoring solution using Grafana#59
lamr02n merged 219 commits intomainfrom
add-grafana

Conversation

@lamr02n
Copy link
Collaborator

@lamr02n lamr02n commented Feb 18, 2025

This PR contains a large part of the newly added monitoring solution, including the Grafana dashboards. Resolves #7

@lamr02n lamr02n added the enhancement New feature or request label Feb 18, 2025
@lamr02n lamr02n added this to the First heiDGAF version milestone Feb 18, 2025
@lamr02n lamr02n self-assigned this Feb 18, 2025
@lamr02n
Copy link
Collaborator Author

lamr02n commented Feb 18, 2025

Note that not all tests have been updated yet

@stefanDeveloper
Copy link
Owner

@lamr02n Tests are still failing, could you investigate for our merge?

Copy link
Owner

@stefanDeveloper stefanDeveloper left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions
Copy link

github-actions bot commented Feb 19, 2025

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
1538 1530 99% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
src/base/data_classes/clickhouse_connectors.py 100% 🟢
src/base/kafka_handler.py 100% 🟢
src/base/logline_handler.py 100% 🟢
src/base/utils.py 100% 🟢
src/detector/detector.py 99% 🟢
src/inspector/inspector.py 98% 🟢
src/logcollector/batch_handler.py 100% 🟢
src/logcollector/collector.py 100% 🟢
src/logserver/server.py 100% 🟢
src/monitoring/clickhouse_connector.py 100% 🟢
src/monitoring/monitoring_agent.py 96% 🟢
src/prefilter/prefilter.py 100% 🟢
TOTAL 99% 🟢

updated for commit: 8d643ae by action🐍

@codecov
Copy link

codecov bot commented Feb 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.15%. Comparing base (1940225) to head (8d643ae).
Report is 220 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #59      +/-   ##
==========================================
+ Coverage   99.12%   99.15%   +0.03%     
==========================================
  Files          16       16              
  Lines        1481     1538      +57     
  Branches      135      147      +12     
==========================================
+ Hits         1468     1525      +57     
  Misses          8        8              
  Partials        5        5              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lamr02n
Copy link
Collaborator Author

lamr02n commented Feb 19, 2025

@stefanDeveloper All tests are green, we're good to go!

@lamr02n lamr02n merged commit 68de1d7 into main Feb 20, 2025
18 checks passed
@lamr02n lamr02n deleted the add-grafana branch February 20, 2025 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add performance tests

2 participants