Commit c1ad6645 authored by Sean McGivern's avatar Sean McGivern

Merge branch 'bvl-fix-sentry-error-grouping' into 'master'

Group `parent not preloaded` errors in sentry

See merge request gitlab-org/gitlab-ce!31424
parents d594217e 788ff548
...@@ -42,7 +42,7 @@ module GroupDescendant ...@@ -42,7 +42,7 @@ module GroupDescendant
parent = child.parent parent = child.parent
exception = ArgumentError.new <<~MSG exception = ArgumentError.new <<~MSG
parent: [GroupDescendant: #{parent.inspect}] was not preloaded for [#{child.inspect}]") Parent was not preloaded for child when rendering group hierarchy.
This error is not user facing, but causes a +1 query. This error is not user facing, but causes a +1 query.
MSG MSG
extras = { extras = {
...@@ -50,7 +50,7 @@ module GroupDescendant ...@@ -50,7 +50,7 @@ module GroupDescendant
child: child.inspect, child: child.inspect,
preloaded: preloaded.map(&:full_path) preloaded: preloaded.map(&:full_path)
} }
issue_url = 'https://gitlab.com/gitlab-org/gitlab-ce/issues/40785' issue_url = 'https://gitlab.com/gitlab-org/gitlab-ce/issues/49404'
Gitlab::Sentry.track_exception(exception, issue_url: issue_url, extra: extras) Gitlab::Sentry.track_exception(exception, issue_url: issue_url, extra: extras)
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment