Age | Commit message (Collapse) | Author |
|
|
|
|
|
Use of square brackets can cause kramdown warnings, as well
as angle brackets, so they should be removed or put in backticks
|
|
|
|
|
|
Fix links as discovered by the lint in the docs project
|
|
These were disabled in production mode, but that also broke the rest of
the performance bar. As they were only enabled in development mode, we
can just remove them for now.
|
|
1. The output isn't great. It can be hard to find hotspots and, even
when you do find them, to find why those are hotspots.
2. It uses some jQuery-specific frontend code which we can remove now
that we don't have this any more.
3. It's only possible to profile the initial request, not any subsequent
AJAX requests.
|
|
|
|
Extend the support for "X-Profile-Token: <token>" to have an additional
header that defines type of profile we are looking for, like:
X-Profile-Mode: execution
X-Profile-Mode: memory
|
|
Removing extra blank lines in docs that serve
no purpose, as well as some --- lines that also
are not needed.
|
|
First MR in a series updating all redirected links
in CE documentation to the destination URLs
|
|
|
|
|
|
|
|
|
|
|
|
Markdown renderers find it easier to determine
where lists start and end when lists are surrounded
by new lines.
For consistency, also ensure entries in the list
are aligned when they span multipls lines.
|
|
|
|
Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
|
|
|
|
|
|
refresh the path to performance bar in admin settings with the up-to-date value
See merge request gitlab-org/gitlab-ce!23316
|
|
|
|
|
|
|
|
The pngquant tool was used https://pngquant.org, and particularly, the
following command:
/usr/bin/pngquant -f --skip-if-larger --ext .png --speed 1 image.png
Before: 47584K
After : 34924K
|
|
But first, rewrite the performance bar in Vue:
1. Remove the peek-host gem and replace it with existing code. This also allows
us to include the host in the JSON response, rather than in the page HTML.
2. Leave the line profiler parts as here-be-dragons: nicer would be a separate
endpoint for these, so we could use them on Ajax requests too.
3. The performance bar is too fiddly to rewrite right now, so apply the same
logic to that.
Then, add features! All requests made through Axios are able to be tracked. To
keep a lid on memory usage, only the first two requests for a given URL are
tracked, though. Each request that's tracked has the same data as the initial
page load, with the exception of the performance bar and the line profiler, as
explained above.
|
|
This is as important as SQL timings, and much more important most of the time
than GC, Redis, or Sidekiq.
|
|
|
|
doc page
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
|
|
|
|
|
The implementation now simply rely on the
`performance_bar_allowed_group_id` Application Setting.
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
|
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
|
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
|
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
|
[ci skip]
|
|
https://gitlab.com/gitlab-org/omnibus-gitlab/blob/master/files/gitlab-config-template/gitlab.rb.template#L1147
|
|
[ci skip]
|
|
[ci skip]
|
|
Replaces https://gitlab.com/gitlab-org/omnibus-gitlab/merge_requests/1237
Omnibus MR: https://gitlab.com/gitlab-org/omnibus-gitlab/merge_requests/1227
Omnibus issue: https://gitlab.com/gitlab-org/omnibus-gitlab/issues/1481
|
|
Took it from https://gitlab.com/gitlab-com/www-gitlab-com/merge_requests/3232
[ci skip]
|
|
Closes #23239
|
|
|