graphite-project / graphite-web

A highly scalable real-time graphing system
http://graphite.readthedocs.org/
Apache License 2.0
5.89k stars 1.26k forks source link

Avoid infinite loop on summarize() #2784

Closed jonnangle closed 1 year ago

jonnangle commented 1 year ago

Fixes #2783

codecov-commenter commented 1 year ago

Codecov Report

Base: 76.30% // Head: 76.29% // Decreases project coverage by -0.01% :warning:

Coverage data is based on head (9ed43ac) compared to base (d6bda76). Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #2784 +/- ## ========================================== - Coverage 76.30% 76.29% -0.02% ========================================== Files 88 88 Lines 9623 9625 +2 Branches 2053 2054 +1 ========================================== Hits 7343 7343 - Misses 2014 2015 +1 - Partials 266 267 +1 ``` | [Impacted Files](https://codecov.io/gh/graphite-project/graphite-web/pull/2784?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=graphite-project) | Coverage Δ | | |---|---|---| | [webapp/graphite/render/functions.py](https://codecov.io/gh/graphite-project/graphite-web/pull/2784/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=graphite-project#diff-d2ViYXBwL2dyYXBoaXRlL3JlbmRlci9mdW5jdGlvbnMucHk=) | `95.03% <0.00%> (-0.09%)` | :arrow_down: | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=graphite-project). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=graphite-project)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

deniszh commented 1 year ago

Looks good, @jonnangle , thanks! Merging.

deniszh commented 1 year ago

💚 All backports created successfully

Status Branch Result
1.1.x

Questions ?

Please refer to the Backport tool documentation