feat(performance): Sync activerecord and net-http span names

[sl0thentr0py]

The span prefixes are needed as documented here for the discover slow ops dashboards to be populated successfully.

Note that this won't break any existing user queries since we currently don't allow filtering/searching by span ops.

Fixes #1674

[AbhiPrasad]

See the current accepted span operation prefixes: https://github.com/getsentry/sentry/blob/809b7fe54c6f06cc1e4c503cf83ded896472a011/src/sentry/projectoptions/defaults.py#L74

[st0012]

Thanks for the fix! Can you also add a changelog entry for it?

[st0012]

@sl0thentr0py sorry I missed one thing: perhaps we can also add template. prefix to ActionView's tracers?

[codecov-commenter]

Codecov Report

Merging #1681 (e3244bf) into master (9889bbe) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1681      +/-   ##
==========================================
+ Coverage   98.37%   98.39%   +0.01%     
==========================================
  Files         136      136              
  Lines        7830     7833       +3     
==========================================
+ Hits         7703     7707       +4     
+ Misses        127      126       -1     

Impacted Files	Coverage Δ
...lib/sentry/rails/tracing/action_view_subscriber.rb	100.00% <100.00%> (ø)
...b/sentry/rails/tracing/active_record_subscriber.rb	100.00% <100.00%> (ø)
sentry-rails/spec/sentry/rails/activejob_spec.rb	99.37% <100.00%> (ø)
...entry/rails/tracing/action_view_subscriber_spec.rb	100.00% <100.00%> (ø)
...try/rails/tracing/active_record_subscriber_spec.rb	100.00% <100.00%> (ø)
sentry-rails/spec/sentry/rails/tracing_spec.rb	99.16% <100.00%> (ø)
sentry-ruby/lib/sentry/net/http.rb	100.00% <100.00%> (ø)
sentry-ruby/spec/sentry/net/http_spec.rb	100.00% <100.00%> (ø)
sentry-ruby/lib/sentry/breadcrumb.rb	100.00% <0.00%> (+3.70%)	⬆️

---

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 9889bbe...e3244bf. Read the comment docs.

[sl0thentr0py]

Yea I added those then removed it because I wasn't able to make it clean. The db one works with prefixes, if we go with a prefix for views it'll look like template.render_template.action_view.

Nicer would be template.render.action_view? In that case I'll go with a mapping hash instead of a prefix. What do you think?

[sl0thentr0py]

nvm I just added the prefix, it's not the nicest looking with repetition but it's fine for now..

[st0012]

Since it's to support a feature, I think the duplication is fine.

[st0012]

Thanks for the feature! I'll put this to 5.1.0 release, which will be released soon after 5.0

[github-actions]

This pull request has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!

---

"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀