github-actions[bot]
commented
2 months ago Test coverage for da59ac6d4cdab3617d84491f6dbfcebd37fc3f38
Name Stmts Miss Branch BrPart Cover Missing
-------------------------------------------------------------------------------------------
paas_app_charmer/__init__.py 29 14 0 0 52% 13-14, 19-20, 26-27, 33-34, 40-41, 47-48, 54-55
paas_app_charmer/_gunicorn/__init__.py 0 0 0 0 100%
paas_app_charmer/_gunicorn/charm.py 15 0 0 0 100%
paas_app_charmer/_gunicorn/webserver.py 83 4 16 1 95% 175, 187-193
paas_app_charmer/_gunicorn/workload_config.py 8 0 0 0 100%
paas_app_charmer/_gunicorn/wsgi_app.py 16 0 0 0 100%
paas_app_charmer/app.py 123 0 50 2 99% 95->exit, 145->151
paas_app_charmer/charm.py 212 25 46 4 88% 32-33, 40-41, 190-191, 193-194, 215->exit, 227-231, 284-286, 339-340, 345, 350, 355, 365, 370, 375, 380, 385, 410
paas_app_charmer/charm_state.py 107 2 20 2 97% 184, 268
paas_app_charmer/charm_utils.py 23 0 0 0 100%
paas_app_charmer/database_migration.py 35 0 2 0 100%
paas_app_charmer/databases.py 25 2 11 1 92% 89-90
paas_app_charmer/django/__init__.py 2 0 0 0 100%
paas_app_charmer/django/charm.py 34 4 6 2 85% 44, 81, 96-97
paas_app_charmer/exceptions.py 5 0 0 0 100%
paas_app_charmer/fastapi/__init__.py 2 0 0 0 100%
paas_app_charmer/fastapi/charm.py 29 0 0 0 100%
paas_app_charmer/flask/__init__.py 2 0 0 0 100%
paas_app_charmer/flask/charm.py 24 0 0 0 100%
paas_app_charmer/go/__init__.py 2 0 0 0 100%
paas_app_charmer/go/charm.py 26 0 0 0 100%
paas_app_charmer/observability.py 20 0 6 1 96% 41->45
paas_app_charmer/secret_storage.py 50 3 16 5 88% 51, 55->54, 56->58, 86, 105
paas_app_charmer/utils.py 21 4 14 2 71% 30, 32, 52-53
-------------------------------------------------------------------------------------------
TOTAL 893 58 187 20 92%Static code analysis report
Run started:2024-09-02 04:54:05.818702
Test results:
No issues identified.
Code scanned:
Total lines of code: 1956
Total lines skipped (#nosec): 1
Total potential issues skipped due to specifically being disabled (e.g., #nosec BXXX): 0
Run metrics:
Total issues (by severity):
Undefined: 0
Low: 0
Medium: 0
High: 0
Total issues (by confidence):
Undefined: 0
Low: 0
Medium: 0
High: 0
Files skipped (0):
Add pebble log forwarding support for PaaS app charms.
Pebble log forwarding is a new feature introduced in Juju 3.4, where Pebble can directly send the application logs (standard outputs) to Loki, instead of relying on Promtail and log files. In this pull request, enable the Pebble log forwarding feature for eligible applications. For an application to be eligible, it must be deployed in a model with a Juju agent version 3.4 or higher, and it must bundle with the v1 version of the
loki_push_apilibrary. Under other conditions, the old Promtail-based log upload method will be used.