Tracing: Fix sampler defaults #5887

matej-g · 2022-11-10T14:24:27Z

Signed-off-by: Matej Gera matejgera@gmail.com

I added CHANGELOG entry for this change.
Change is not relevant to the end user.

Changes

Fixes #5872

Makes sure we use rate limiting sampler as default, as was the case pre-migration
Correct the of parent-based sampler and option - we should always use parent based sampler to respect the span parent, if it is being sampled. Optionally let user configure parent based sampler.

Verification

Added unit test

matej-g · 2022-11-10T14:25:39Z

cc @metonymic-smokey

yeya24

Thanks for the quick fix!

yeya24

Shall we get this into the release branch and then merge back to main?

matej-g · 2022-11-14T20:39:29Z

@yeya24, yes, that's probably better idea if we want a patch release. I'll point it to release 29 branch then.

Signed-off-by: Matej Gera <matejgera@gmail.com>

Signed-off-by: Matej Gera <matejgera@gmail.com> Signed-off-by: Matej Gera <matejgera@gmail.com>

yeya24

Looks good to me. One question only.

yeya24 · 2022-11-15T06:43:46Z

.circleci/config.yml

@@ -3,6 +3,7 @@ version: 2.1

 orbs:
  go: circleci/go@1.7.1
+  git-shallow-clone: guitarrapc/git-shallow-clone@2.4.0


Any reason to add this btw? @matej-g

The CI was failing because of https://cloud-native.slack.com/archives/CL25937SP/p1666777294865379, it's just a cherry-pick from the main but should not affect the release itself.

* Fix sampler defaults Signed-off-by: Matej Gera <matejgera@gmail.com> * Add CHANGELOG Signed-off-by: Matej Gera <matejgera@gmail.com> * Replace checkout with git-shallow-clone (#5829) Signed-off-by: Matej Gera <matejgera@gmail.com> Signed-off-by: Matej Gera <matejgera@gmail.com> Signed-off-by: Matej Gera <matejgera@gmail.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com>

* compact: remove cancel on SyncMetas errors (#5923) in a favour of 86b4039 SyncMetas will retry if it's retriable. Also, the cleanPartialMarked calls are surrounded by runutil.Repeat() will be repeated, the ones not and are not retriable will throw an interrupt to run.Group() by returning err and Group will call cancel() as it's configured for its interrupt func. Signed-off-by: Seena Fallah <seenafallah@gmail.com> Signed-off-by: Seena Fallah <seenafallah@gmail.com> * Cut v0.30.0-rc.0 (#5992) * Cut v0.30.0-rc.0 Signed-off-by: bwplotka <bwplotka@gmail.com> * mdox fix. Signed-off-by: bwplotka <bwplotka@gmail.com> Signed-off-by: bwplotka <bwplotka@gmail.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> * Cut 0.30.0 (#6011) Signed-off-by: bwplotka <bwplotka@gmail.com> Signed-off-by: bwplotka <bwplotka@gmail.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> * *: cut 0.30.1 (#6017) * fix duplicate metrics registration in redis client (#6009) * fix duplicate metrics registration in redis client Signed-off-by: Kama Huang <kamatogo13@gmail.com> * fixed test Signed-off-by: Kama Huang <kamatogo13@gmail.com> Signed-off-by: Kama Huang <kamatogo13@gmail.com> * *: cut 0.30.1 Add CHANGELOG entry. Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> Signed-off-by: Kama Huang <kamatogo13@gmail.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> Co-authored-by: Kama Huang <121007071+kama910@users.noreply.github.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> * Tracing: Fix sampler defaults (#5887) * Fix sampler defaults Signed-off-by: Matej Gera <matejgera@gmail.com> * Add CHANGELOG Signed-off-by: Matej Gera <matejgera@gmail.com> * Replace checkout with git-shallow-clone (#5829) Signed-off-by: Matej Gera <matejgera@gmail.com> Signed-off-by: Matej Gera <matejgera@gmail.com> Signed-off-by: Matej Gera <matejgera@gmail.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> * CHANGELOG: fix Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> Signed-off-by: Seena Fallah <seenafallah@gmail.com> Signed-off-by: bwplotka <bwplotka@gmail.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> Signed-off-by: Kama Huang <kamatogo13@gmail.com> Signed-off-by: Matej Gera <matejgera@gmail.com> Co-authored-by: Seena Fallah <seenafallah@gmail.com> Co-authored-by: Kama Huang <121007071+kama910@users.noreply.github.com>

Added re-try mechanism for store inital sync, where if the initial sync fails, it tries to do the initial sync again every 5 seconds for 15 seconds duration (total 3 re-tries for initial sync of store). Signed-off-by: Kartik-Garg <kartik.garg@infracloud.io> Store: Make initial sync more robust Added re-try mechanism for store inital sync, where if the initial sync fails, it tries to do the initial sync again every 5 seconds for 15 seconds duration (total 3 re-tries for initial sync of store). Signed-off-by: Kartik-Garg <kartik.garg@infracloud.io> Merge release 0.30 into main (thanos-io#6041) * compact: remove cancel on SyncMetas errors (thanos-io#5923) in a favour of 86b4039 SyncMetas will retry if it's retriable. Also, the cleanPartialMarked calls are surrounded by runutil.Repeat() will be repeated, the ones not and are not retriable will throw an interrupt to run.Group() by returning err and Group will call cancel() as it's configured for its interrupt func. Signed-off-by: Seena Fallah <seenafallah@gmail.com> Signed-off-by: Seena Fallah <seenafallah@gmail.com> * Cut v0.30.0-rc.0 (thanos-io#5992) * Cut v0.30.0-rc.0 Signed-off-by: bwplotka <bwplotka@gmail.com> * mdox fix. Signed-off-by: bwplotka <bwplotka@gmail.com> Signed-off-by: bwplotka <bwplotka@gmail.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> * Cut 0.30.0 (thanos-io#6011) Signed-off-by: bwplotka <bwplotka@gmail.com> Signed-off-by: bwplotka <bwplotka@gmail.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> * *: cut 0.30.1 (thanos-io#6017) * fix duplicate metrics registration in redis client (thanos-io#6009) * fix duplicate metrics registration in redis client Signed-off-by: Kama Huang <kamatogo13@gmail.com> * fixed test Signed-off-by: Kama Huang <kamatogo13@gmail.com> Signed-off-by: Kama Huang <kamatogo13@gmail.com> * *: cut 0.30.1 Add CHANGELOG entry. Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> Signed-off-by: Kama Huang <kamatogo13@gmail.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> Co-authored-by: Kama Huang <121007071+kama910@users.noreply.github.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> * Tracing: Fix sampler defaults (thanos-io#5887) * Fix sampler defaults Signed-off-by: Matej Gera <matejgera@gmail.com> * Add CHANGELOG Signed-off-by: Matej Gera <matejgera@gmail.com> * Replace checkout with git-shallow-clone (thanos-io#5829) Signed-off-by: Matej Gera <matejgera@gmail.com> Signed-off-by: Matej Gera <matejgera@gmail.com> Signed-off-by: Matej Gera <matejgera@gmail.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> * CHANGELOG: fix Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> Signed-off-by: Seena Fallah <seenafallah@gmail.com> Signed-off-by: bwplotka <bwplotka@gmail.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> Signed-off-by: Kama Huang <kamatogo13@gmail.com> Signed-off-by: Matej Gera <matejgera@gmail.com> Co-authored-by: Seena Fallah <seenafallah@gmail.com> Co-authored-by: Kama Huang <121007071+kama910@users.noreply.github.com>

* compact: remove cancel on SyncMetas errors (thanos-io#5923) in a favour of 86b4039 SyncMetas will retry if it's retriable. Also, the cleanPartialMarked calls are surrounded by runutil.Repeat() will be repeated, the ones not and are not retriable will throw an interrupt to run.Group() by returning err and Group will call cancel() as it's configured for its interrupt func. Signed-off-by: Seena Fallah <seenafallah@gmail.com> Signed-off-by: Seena Fallah <seenafallah@gmail.com> * Cut v0.30.0-rc.0 (thanos-io#5992) * Cut v0.30.0-rc.0 Signed-off-by: bwplotka <bwplotka@gmail.com> * mdox fix. Signed-off-by: bwplotka <bwplotka@gmail.com> Signed-off-by: bwplotka <bwplotka@gmail.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> * Cut 0.30.0 (thanos-io#6011) Signed-off-by: bwplotka <bwplotka@gmail.com> Signed-off-by: bwplotka <bwplotka@gmail.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> * *: cut 0.30.1 (thanos-io#6017) * fix duplicate metrics registration in redis client (thanos-io#6009) * fix duplicate metrics registration in redis client Signed-off-by: Kama Huang <kamatogo13@gmail.com> * fixed test Signed-off-by: Kama Huang <kamatogo13@gmail.com> Signed-off-by: Kama Huang <kamatogo13@gmail.com> * *: cut 0.30.1 Add CHANGELOG entry. Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> Signed-off-by: Kama Huang <kamatogo13@gmail.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> Co-authored-by: Kama Huang <121007071+kama910@users.noreply.github.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> * Tracing: Fix sampler defaults (thanos-io#5887) * Fix sampler defaults Signed-off-by: Matej Gera <matejgera@gmail.com> * Add CHANGELOG Signed-off-by: Matej Gera <matejgera@gmail.com> * Replace checkout with git-shallow-clone (thanos-io#5829) Signed-off-by: Matej Gera <matejgera@gmail.com> Signed-off-by: Matej Gera <matejgera@gmail.com> Signed-off-by: Matej Gera <matejgera@gmail.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> * CHANGELOG: fix Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> Signed-off-by: Seena Fallah <seenafallah@gmail.com> Signed-off-by: bwplotka <bwplotka@gmail.com> Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com> Signed-off-by: Kama Huang <kamatogo13@gmail.com> Signed-off-by: Matej Gera <matejgera@gmail.com> Co-authored-by: Seena Fallah <seenafallah@gmail.com> Co-authored-by: Kama Huang <121007071+kama910@users.noreply.github.com>

pull-request-size bot added the size/M label Nov 10, 2022

matej-g mentioned this pull request Nov 10, 2022

query: v0.29.0. Opentracing config create gopanic SIGSEGV #5872

Closed

yeya24 previously approved these changes Nov 10, 2022

View reviewed changes

matej-g dismissed yeya24’s stale review via 6baa20e November 11, 2022 08:56

yeya24 reviewed Nov 12, 2022

View reviewed changes

matej-g force-pushed the fix-sampler-defaults branch from 6e89d4b to 7980c2e Compare November 14, 2022 21:20

pull-request-size bot added size/L and removed size/M labels Nov 14, 2022

matej-g changed the base branch from main to release-0.29 November 14, 2022 21:21

matej-g added 2 commits November 14, 2022 22:24

Fix sampler defaults

10ef0d1

Signed-off-by: Matej Gera <matejgera@gmail.com>

Add CHANGELOG

d142ac7

Signed-off-by: Matej Gera <matejgera@gmail.com>

matej-g force-pushed the fix-sampler-defaults branch from 7980c2e to d142ac7 Compare November 14, 2022 21:24

Replace checkout with git-shallow-clone (thanos-io#5829)

a489b31

Signed-off-by: Matej Gera <matejgera@gmail.com> Signed-off-by: Matej Gera <matejgera@gmail.com>

yeya24 approved these changes Nov 15, 2022

View reviewed changes

yeya24 merged commit df047ef into thanos-io:release-0.29 Nov 15, 2022

matej-g mentioned this pull request Jan 10, 2023

nil pointer dereference in main-2022-10-02-d533abf8 forward #6021

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Tracing: Fix sampler defaults #5887

Tracing: Fix sampler defaults #5887

matej-g commented Nov 10, 2022

matej-g commented Nov 10, 2022

yeya24 left a comment

yeya24 left a comment

matej-g commented Nov 14, 2022

yeya24 left a comment

yeya24 Nov 15, 2022

matej-g Nov 15, 2022

Tracing: Fix sampler defaults #5887

Tracing: Fix sampler defaults #5887

Conversation

matej-g commented Nov 10, 2022

Changes

Verification

matej-g commented Nov 10, 2022

yeya24 left a comment

Choose a reason for hiding this comment

yeya24 left a comment

Choose a reason for hiding this comment

matej-g commented Nov 14, 2022

yeya24 left a comment

Choose a reason for hiding this comment

yeya24 Nov 15, 2022

Choose a reason for hiding this comment

matej-g Nov 15, 2022

Choose a reason for hiding this comment