Merge upstream changes #130

npazosmendez · 2022-12-28T19:12:40Z

No description provided.

Avoid of alloc for func args copy

Render marshalSJON/Raw/CSV preallocated bytes buffer

…y-functions Update compatibility docs to include holtWintersConfidenceArea

Fix for panic on empty interval string

…ests Fix consolidateFunc setting and add test for consolidateBy

Add test for timeFunction

Add test for cumulative function

Add test for timeStack function

…-test Add test for setXFilesFactor function

Logging improvements

Fix parser panic on = at the end

response/backend cache short duration tune

* Add preliminary support for mTLS (all required TLS options) * Remove CarbonSearch support * Fix Idle Connection Timeout for HTTP Client (shouldn't reuse read timeout as it was before) * Move all common part for protocols to helper pkg

NOTE: This should work, however it was not properly tested (except for classic config without TLS) This change should allow to specify basic parameters for TLS listeners as well as allow to enable mTLS on listener

* Add tests to verify that basic functionality works as expected * Fix some problems that was discovered while testing

Refactor HTTP client and listeners

If set to `true` it will try to combine multiple taregts into single multiFetch request At it's current stage it might need more work, especially around handling fallbackSeries, but it already should be a huge improvement Fixes #631, #509, #63

There were some options that were missing in manual copy and as all work was performed on a copy that resulted in them being reverted to default value. Use go-deepcopy to correctly do recursive copy.

Since Go 1.16 ioutil is deprecated, migrating render_handler to io

Multifetch

Update changelog with all the changes that happened since previous release

* Update all vendored deps to latest stable versions * Fix difference for IronDB (new version of library have minor differences in how you can instanciate a client)

let's docker/metadata to handle this

check groupByNode aggregation function

functions: check applyByNode field overflow

CLAassistant · 2022-12-28T19:12:48Z

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 4 committers have signed the CLA.

✅ carrieedwards
❌ msaf1980
❌ Civil
❌ deniszh
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

msaf1980 and others added 30 commits October 20, 2022 17:52

avoid of alloc for func args copy

4103e65

Merge pull request #728 from msaf1980/reduce_alloc_args

e38ed01

Avoid of alloc for func args copy

render: Marshal benchmarks

c96a5dd

render: optimize Marshal perfomance

bf0ffdc

Merge pull request #729 from msaf1980/render_marshal

396f5b0

Render marshalSJON/Raw/CSV preallocated bytes buffer

Update compatibility docs to include holtWintersConfidenceArea

596aff3

Merge pull request #730 from go-graphite/cedwards/update-compatibilit…

7d3b255

…y-functions Update compatibility docs to include holtWintersConfidenceArea

check for empty interval string

3dfe040

Merge pull request #731 from msaf1980/fix_panic_on_emty_interval

50b983a

Fix for panic on empty interval string

Add name for consolidateBy results to match Graphite web

1ce30e8

Add tests for consolidateBy function

6d77210

Update TestEvalExprModifiedOrigin to check ConsolidationFunc values

d84b0fa

Remove unnecessary consolidationFunc settings and update cairo test

8ec57ba

Update e2e test

501ca38

Fix e2e test

076b37f

Merge pull request #733 from go-graphite/cedwards/add-consolidateBy-t…

966a0b0

…ests Fix consolidateFunc setting and add test for consolidateBy

Update name and tags for cumulative to match Graphite web

a5d1c84

Add test for cumulative function

13f522d

Add test for timeFunction

f351d01

Merge pull request #735 from go-graphite/cedwards/add-timeFunction-test

5daf0ba

Add test for timeFunction

Merge pull request #734 from go-graphite/cedwards/add-cumulative-test

34f2365

Add test for cumulative function

Fix name of results in timeStack to match Graphite web

c1e6d10

Add test for timeStack

8fcfe03

Merge pull request #736 from go-graphite/cedwards/add-timestack-test

b07075b

Add test for timeStack function

Add xFilesFactor tag and prevent modifying input for SetXFilesFactor

45fd536

Add test for SetXFilesFactor

dc479f3

Merge pull request #737 from go-graphite/cedwards/add-setXFilesFactor…

10df3fe

…-test Add test for setXFilesFactor function

Fix handling of NaN values in stdev function

258b3d0

Add test for stdev function

81029cb

Merge branch 'main' into cedwards/add-stdev-test

1801123

msaf1980 and others added 26 commits November 21, 2022 11:18

parser: fix panic on = at the end

257385d

Merge pull request #739 from msaf1980/logging

c422204

Logging improvements

Merge branch 'main' into fix_parser_panic

4e25f3b

Merge pull request #740 from msaf1980/fix_parser_panic

006be60

Fix parser panic on = at the end

response/backend cache short duration tune

5c90fff

Update configuration.md

04e6318

Merge pull request #741 from msaf1980/cache_tune

0bb2584

response/backend cache short duration tune

Refactor HTTP client

0611cb8

* Add preliminary support for mTLS (all required TLS options) * Remove CarbonSearch support * Fix Idle Connection Timeout for HTTP Client (shouldn't reuse read timeout as it was before) * Move all common part for protocols to helper pkg

Update some of vendored modules

e61cfd3

Add preliminary support of specifying TLS settings for listeners

b960cb1

NOTE: This should work, however it was not properly tested (except for classic config without TLS) This change should allow to specify basic parameters for TLS listeners as well as allow to enable mTLS on listener

Continue work on listeners

fda6d5b

* Add tests to verify that basic functionality works as expected * Fix some problems that was discovered while testing

Merge pull request #742 from go-graphite/httpclient

2484afd

Refactor HTTP client and listeners

Fix problem with config sanitization

94a6abf

There were some options that were missing in manual copy and as all work was performed on a copy that resulted in them being reverted to default value. Use go-deepcopy to correctly do recursive copy.

Remove deprecated ioutil package from render_handler

673b6ce

Since Go 1.16 ioutil is deprecated, migrating render_handler to io

Remove some of the deprecated functions

8f5ee35

Merge pull request #743 from go-graphite/multifetch

5d1d832

Multifetch

Update changelog

715e93c

Update changelog with all the changes that happened since previous release

Update vendored deps

5d2251e

* Update all vendored deps to latest stable versions * Fix difference for IronDB (new version of library have minor differences in how you can instanciate a client)

Update changelog for v0.16.0

50cb0c4

Fix docker upload image workflow

5a92ce1

Set latest to auto

f165db7

let's docker/metadata to handle this

functions: check groupByNode aggregation function

4df4412

Merge pull request #744 from msaf1980/fix_groupByNode

8cdc72f

check groupByNode aggregation function

functions: check applyByNode field overflow

abd2bd0

Merge pull request #745 from msaf1980/fix_applyByNode

d04db28

functions: check applyByNode field overflow

npazosmendez requested a review from carrieedwards December 28, 2022 19:40

carrieedwards approved these changes Dec 28, 2022

View reviewed changes

npazosmendez merged commit 29222e6 into grafana:main Dec 28, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Merge upstream changes #130

Merge upstream changes #130

npazosmendez commented Dec 28, 2022

CLAassistant commented Dec 28, 2022

Merge upstream changes #130

Merge upstream changes #130

Conversation

npazosmendez commented Dec 28, 2022

CLAassistant commented Dec 28, 2022