search

scalyr / scalyr-agent-2

The source code for Scalyr Agent 2, the daemon process Scalyr customers run on their servers to collect metrics and logs.
Apache License 2.0
70 stars 59 forks source link

DockerLog class for docker_log and raw_logs to improve readability and robustness of the code #1242

Closed alesnovak-s1 closed 8 months ago

github-actions[bot] commented 8 months ago

Test Results

     20 files  ±0       20 suites  ±0   31m 56s :stopwatch: +21s 1 477 tests ±0  1 457 :heavy_check_mark: ±0    20 :zzz: ±0  0 :x: ±0  7 060 runs  ±0  6 835 :heavy_check_mark: ±0  225 :zzz: ±0  0 :x: ±0 

Results for commit 9072aa3c. ± Comparison against base commit ae9f3d64.

codecov[bot] commented 8 months ago

Codecov Report

Attention: 13 lines in your changes are missing coverage. Please review.

Comparison is base (ae9f3d6) 81.56% compared to head (9072aa3) 77.93%.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/scalyr/scalyr-agent-2/pull/1242/graphs/tree.svg?width=650&height=150&src=pr&token=KrvyP8n7be&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scalyr)](https://app.codecov.io/gh/scalyr/scalyr-agent-2/pull/1242?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scalyr) ```diff @@ Coverage Diff @@ ## master #1242 +/- ## ========================================== - Coverage 81.56% 77.93% -3.64% ========================================== Files 176 176 Lines 42565 42583 +18 Branches 4749 4749 ========================================== - Hits 34718 33184 -1534 - Misses 6617 8344 +1727 + Partials 1230 1055 -175 ``` | [Files](https://app.codecov.io/gh/scalyr/scalyr-agent-2/pull/1242?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scalyr) | Coverage Δ | | |---|---|---| | [...calyr\_agent/builtin\_monitors/kubernetes\_monitor.py](https://app.codecov.io/gh/scalyr/scalyr-agent-2/pull/1242?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scalyr#diff-c2NhbHlyX2FnZW50L2J1aWx0aW5fbW9uaXRvcnMva3ViZXJuZXRlc19tb25pdG9yLnB5) | `45.55% <48.00%> (-22.07%)` | :arrow_down: | ... and [22 files with indirect coverage changes](https://app.codecov.io/gh/scalyr/scalyr-agent-2/pull/1242/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scalyr)