Commit 788ff548 authored by Bob Van Landuyt's avatar Bob Van Landuyt

Group `parent not preloaded` errors in sentry

I still haven't found a moment to look into the exact cases that
causes this error, but we're not loosing any information this way.

In fact, it will be easier to find the right information as we only
need to look to one entry in Sentry.
parent 1bc2ac33
...@@ -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