Skip to content

Add the third tracing state#1402

Merged
st0012 merged 2 commits intomasterfrom
support-third-tracing-state
Apr 23, 2021
Merged

Add the third tracing state#1402
st0012 merged 2 commits intomasterfrom
support-third-tracing-state

Conversation

@st0012
Copy link
Copy Markdown
Contributor

@st0012 st0012 commented Apr 16, 2021

  • rate == 0 - Tracing enabled. Rejects all locally created transactions but respects sentry-trace.
  • 1 > rate > 0 - Tracing enabled. Samples locally created transactions with the rate and respects sentry-trace.
  • rate < 0 or rate > 1 - Tracine disabled.

@st0012 st0012 added this to the 4.4.0 milestone Apr 16, 2021
@st0012 st0012 self-assigned this Apr 16, 2021
@st0012 st0012 requested review from jan-auer and rhcarvalho April 16, 2021 13:32
@st0012 st0012 force-pushed the support-third-tracing-state branch from 893f3a6 to 66d4d74 Compare April 16, 2021 13:33
@st0012 st0012 force-pushed the support-third-tracing-state branch from 9a2bf52 to ecde56c Compare April 16, 2021 15:17
@codecov-io
Copy link
Copy Markdown

codecov-io commented Apr 16, 2021

Codecov Report

Merging #1402 (3af03ab) into master (f82a2c3) will increase coverage by 0.59%.
The diff coverage is 100.00%.

❗ Current head 3af03ab differs from pull request most recent head ecde56c. Consider uploading reports for the commit ecde56c to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1402      +/-   ##
==========================================
+ Coverage   98.19%   98.79%   +0.59%     
==========================================
  Files         214      119      -95     
  Lines        9862     6045    -3817     
==========================================
- Hits         9684     5972    -3712     
+ Misses        178       73     -105     
Impacted Files Coverage Δ
sentry-ruby/lib/sentry/configuration.rb 97.39% <100.00%> (ø)
sentry-ruby/spec/sentry/configuration_spec.rb 98.62% <100.00%> (+0.02%) ⬆️
sentry-ruby/spec/sentry/transaction_spec.rb 100.00% <100.00%> (ø)
sentry-rails/app/jobs/sentry/send_event_job.rb 72.22% <0.00%> (-5.56%) ⬇️
sentry-raven/lib/raven/processor/cookies.rb
sentry-raven/spec/raven/client_state_spec.rb
...try-raven/lib/raven/interfaces/single_exception.rb
...ntry-raven/spec/raven/utils/context_filter_spec.rb
sentry-raven/spec/raven/logger_spec.rb
sentry-raven/lib/raven/integrations/rails.rb
... and 89 more

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 f82a2c3...ecde56c. Read the comment docs.

@st0012 st0012 requested review from HazAT and rhcarvalho April 20, 2021 09:57
Copy link
Copy Markdown
Contributor

@rhcarvalho rhcarvalho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few suggestions, generally ltgm

st0012 added 2 commits April 23, 2021 22:45
- rate == 0 - Tracing enabled. Rejects all locally created transactions but
  respects sentry-trace.
- 1 > rate > 0 - Tracing enabled. Samples locally created transactions
  with the rate and respects sentry-trace.
- rate < 0 or rate > 1 - Tracine disabled.
@st0012 st0012 force-pushed the support-third-tracing-state branch from 6a7ad1d to 57c586c Compare April 23, 2021 14:45
@st0012 st0012 merged commit 036a661 into master Apr 23, 2021
@st0012 st0012 deleted the support-third-tracing-state branch April 23, 2021 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants