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

Grafana integration #81

Open
wants to merge 64 commits into
base: master
Choose a base branch
from
Open

Grafana integration #81

wants to merge 64 commits into from

Conversation

Hebruwu
Copy link
Contributor

@Hebruwu Hebruwu commented Aug 8, 2022

No description provided.

Daniel Sabanov and others added 30 commits July 1, 2022 08:15
Add to_dict method to metrics.
2a4dce8
Add to_dict function to metrics.
dc2d84a
Merge branch 'master' into configure_metrics_with_yaml
b19e62b
Merge in master.
93cbbd2
Add comments to new methods.
99a4777
Remove leftover TODO
ca9f260
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
54b80fe 
for more information, see https://pre-commit.ci
Add from_dict to metrics and corresponding tests.
bebc01d
Add from_dict to metrics and corresponding tests.
b5572a2
Merge branch 'configure_metrics_with_yaml' of github.com:synthesized-…
40405b4 
…io/insight into configure_metrics_with_yaml
Remove redundant function.
7bb61c4
Add metrics_from_yaml and metrics_yaml_dump and corresponding tests.
7acb26c
Add missing yaml dependency.
ad3ad89
Add types-pyyaml to test requirements.
207f816
Add types-pyyaml to requirements for mypy purposes.
87856b6
Add types-pyyaml to requirements for mypy purposes.
d9cfcc7
Corrected typing error.
c62dab2
@simonhkswan
Add pyyaml stubs to precommit mypy + ignore pandas
123b824
Improve YAML related tests.
8ec27b9
Improve YAML related tests.
ce8d81c
Add adapter classes for metrics.
e15de0d
Add adapter classes for metrics.
b14bcd4
Merge with master
633c785
Add plotting functions
9a2b4ea
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
7f7cf9a 
for more information, see https://pre-commit.ci
Fixed metrics_usage tests.
43507bf
Fixed metrics_usage tests.
da9b766
Fixed metrics_usage tests.
3304c86
@simonhkswan
Update pyproject.toml
d1c8422
@simonhkswan
Update pyproject.toml
31cb4cd
Daniel Sabanov and others added 25 commits July 27, 2022 11:28
Add sqlalchemy schema
a0b91ec
Add the first tables to the database
8a8cd45
Merge with main
a7e4978
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
a974a63 
for more information, see https://pre-commit.ci
Update pyproject.toml
367860b
Update requirements.txt
d8d8c86
Update package requirements
f3cbe69
Merge branch 'database_addition' of github.com:synthesized-io/insight…
d6fae79 
… into database_addition
Remove postgres-data
951bf64
Update .gitignore
acdbccd
Update pyproject.toml
54261c6
Update pyproject.toml
e91b370
Merge remote-tracking branch 'origin' into database_addition
c4ad428
@Hebruwu
Update Dockerfile and requirements
47181db
@Hebruwu
Make metrics write their results into the database.
48381c0
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
4220e58 
for more information, see https://pre-commit.ci
@Hebruwu
Make the database fill in all of its fields.
1c14aaf
@Hebruwu
Make the database fill in all of its fields.
cb10d1e
@Hebruwu
Extract tests from seperate folder
ffe1217
@Hebruwu
fix circular import
7171e32
@Hebruwu
Add metric tests to a seperate folder
ddc9cb4
@Hebruwu
Introduce a version constant
f8aa808
@Hebruwu
Add tests for the databse.
25813c2
@Hebruwu
Make sure the database is cleaned after each run.
63ff050
@Hebruwu
Add grafana integration.
d5148fe
@Hebruwu Hebruwu linked an issue Aug 8, 2022 that may be closed by this pull request
Using Docker Compose, configure a database and Grafana dashboard for monitoring metrics #80
Open
@Hebruwu
Copy link
Contributor Author

Hebruwu commented Aug 8, 2022

The database branch needs to be merged in

@sonarcloud
Copy link

sonarcloud bot commented Aug 8, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

86.1% 86.1% Coverage
0.0% 0.0% Duplication

@codecov
Copy link

codecov bot commented Aug 8, 2022

Codecov Report

Merging #81 (d5148fe) into master (86567c1) will increase coverage by 1.3%.
The diff coverage is 83.4%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master     #81     +/-   ##
========================================
+ Coverage    79.0%   80.3%   +1.3%     
========================================
  Files           9      12      +3     
  Lines         687     816    +129     
  Branches      135     149     +14     
========================================
+ Hits          543     656    +113     
- Misses        113     123     +10     
- Partials       31      37      +6
Impacted Files Coverage Δ
src/insight/database/db_connection.py 0.0% <0.0%> (ø)
src/insight/metrics/base.py 82.8% <78.8%> (-3.7%) ⬇️
src/insight/metrics/metrics_usage.py 92.1% <83.3%> (+10.7%) ⬆️
src/insight/database/utils.py 88.0% <88.0%> (ø)
src/insight/database/schema.py 100.0% <100.0%> (ø)

@Hebruwu Hebruwu marked this pull request as ready for review August 8, 2022 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Using Docker Compose, configure a database and Grafana dashboard for monitoring metrics
2 participants
@Hebruwu @simonhkswan