search

graphite-project / carbon

Carbon is one of the components of Graphite, and is responsible for receiving metrics over the network and writing them down to disk using a storage backend.
http://graphite.readthedocs.org/
Apache License 2.0
1.51k stars 490 forks source link

[1.1.x] WHISPER_FALLOCATE_CREATE documentation improved | fix flake8 errors | Add optional normalization when relaying tagged metrics | lint fix | Update default storage schema in storage-schemas.conf.example #830

Closed deniszh closed 5 years ago

deniszh commented 5 years ago

Backports the following commits to 1.1.x:

codecov-io commented 5 years ago

Codecov Report

Merging #830 into 1.1.x will increase coverage by 0.53%. The diff coverage is 61.9%.

Impacted file tree graph

@@            Coverage Diff             @@
##            1.1.x     #830      +/-   ##
==========================================
+ Coverage   49.62%   50.15%   +0.53%     
==========================================
  Files          37       37              
  Lines        3434     3443       +9     
  Branches      494      495       +1     
==========================================
+ Hits         1704     1727      +23     
+ Misses       1621     1604      -17     
- Partials      109      112       +3
Impacted Files Coverage Δ
lib/carbon/conf.py 40.72% <ø> (ø) :arrow_up:
lib/carbon/pipeline.py 64% <0%> (ø) :arrow_up:
lib/carbon/routers.py 68.75% <0%> (ø) :arrow_up:
lib/carbon/cache.py 80.92% <100%> (+0.25%) :arrow_up:
lib/carbon/protocols.py 73.56% <100%> (+0.23%) :arrow_up:
lib/carbon/client.py 55.29% <66.66%> (+0.53%) :arrow_up:
lib/carbon/util.py 48.84% <0%> (+5.38%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 62a1721...4460d9a. Read the comment docs.

piotr1212 commented 5 years ago

Don't know if https://github.com/graphite-project/carbon/pull/825 needs to be included as it didn't fix the issue.

deniszh commented 5 years ago

829 contains rollback of #825, so, I think yes - I do not want cherry pick individual commits for backporting.