| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
| |
This removes the need for running a database query every time we want to
check the database version.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When displaying the pipelines of a project we now preload the following
data:
1. Authors of the commits that belong to these pipelines
2. The number of warnings per pipeline, which is used by
Ci::Pipeline#has_warnings?
== Commit Authors
Previously this data was queried for every Commit separately, leading to
20 SQL queries being executed in the worst case. With an average of 3 to
5 milliseconds per SQL query this could result in 100 milliseconds being
spent in _just_ getting Commit authors.
To preload this data Commit#author now uses BatchLoader (through
Commit#lazy_author), and a separate module
Gitlab::Ci::Pipeline::Preloader is used to ensure all authors are loaded
before they are used.
== Number of warnings
This changes Ci::Pipeline#has_warnings? so it supports preloading of the
number of warnings per pipeline. This removes the need for executing a
COUNT(*) query for every pipeline just to see if it has any warnings or
not.
|
| |\
| |
| |
| |
| |
| |
| | |
Workhorse to send raw diff and patch for commits
Closes gitaly#1196
See merge request gitlab-org/gitlab-ce!18974
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Prior to this change, this was done through unicorn. In theory this
could time out. Workhorse has been sending these raw patches and diffs
for a long time and is stable in doing so.
Added bonus is the fact that `Commit#to_patch` can be removed.
`Commit#to_diff` too, which closes
https://gitlab.com/gitlab-org/gitaly/issues/324
Closes https://gitlab.com/gitlab-org/gitaly/issues/1196
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Fix Error 500 viewing admin page due to statement timeouts
Closes #46255
See merge request gitlab-org/gitlab-ce!18982
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Uses PostgreSQL tuple estimates to provide a much faster yet approximate
count. See https://wiki.postgresql.org/wiki/Slow_Counting for more details.
We only use this fast method if the table has been analyzed or vacuumed
within the last hour.
Closes #46255
|
| |/ /
| |
| |
| |
| |
| |
| |
| |
| | |
Certain controllers (e.g. Doorkeeper::TokensController) don't expose the
method "request_format". This commit changes
Gitlab::Metrics::WebTransaction so we don't rely on this method, instead
using the underlying code this method uses.
Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/46412
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Ref contains oid check done by Gitaly
Closes gitaly#884
See merge request gitlab-org/gitlab-ce!18944
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
These two endpoints are in opt-out, and no one noticed. Has been running
on .com for 4 months now and considered stable.
Closes https://gitlab.com/gitlab-org/gitaly/issues/884
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Fix setting Gitlab metrics content types
Closes gitlab-com/infrastructure#3499
See merge request gitlab-org/gitlab-ce!18975
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The method "content_type" on a controller does not always return the
correct content type. On the other hand, the method "request_format"
does _and_ immediately returns a Symbol (e.g. :json) instead of a
mime-type name (e.g. application/json). With these changes metrics
should again report their action names correctly.
Fixes https://gitlab.com/gitlab-com/infrastructure/issues/3499
|
| |\ \ \ \
| |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | | |
Resolve "Add nip.io domain suggestion in the Kubernetes cluster details view"
Closes #45584
See merge request gitlab-org/gitlab-ce!18496
|
| | | |/
| |/| |
|
| |/ / |
|
| |\ \
| | |
| | |
| | |
| | | |
Backport EE-specific untrusted regexp implementation
See merge request gitlab-org/gitlab-ce!18938
|
| | | | |
|
| | |/
|/|
| |
| |
| |
| |
| |
| |
| | |
An LDAP sign-in request results in a different request parameter than
a standard GitLab sign-in. Since Warden doesn't pass us the user that
was blocked, we first search for a `username` in the request parameters
and then look for `user.login`.
Closes #46307
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Block access to API & git when terms are enforced
Closes #45849
See merge request gitlab-org/gitlab-ce!18816
|
| | | |
| | |
| | |
| | |
| | | |
Allow builds that have been triggered by a user before terms were
enforced access to git. That way the builds can complete as usual.
|
| | | |
| | |
| | |
| | |
| | |
| | | |
When terms are enforced, but the user has not accepted the terms
access to the API & git is rejected with a message directing the user
to the web app to accept the terms.
|
| | |/
|/|
| |
| |
| |
| | |
This used to be executed because of the post receive worker, but this
uses Gitlab::GlRepository.parse now. So this removes dead code and
reduces the number of calls on `#legacy_storage_path`.
|
| |\ \
| |/
|/|
| |
| | |
Remove method call to deprecated method
See merge request gitlab-org/gitlab-ce!18815
|
| | |
| |
| |
| |
| | |
Given the settings initializer creates Gitaly Storage Settings objects
already, the calls to path can be moved to that initializer.
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Support resetting of Prometheus metrics between test runs
Closes #39968
See merge request gitlab-org/gitlab-ce!18836
|
| | |/
| |
| |
| |
| |
| |
| | |
Adding the :prometheus tag to an rspec test will clear out
memory-mapped files and reset the registry.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/39968
|
| |/
|
|
|
| |
The lib/gitlab/git/repository.rb needs to have the same content between
gitlab-ce and gitlab-ee in order to have Gitaly working fine.
|
| |\
| |
| |
| |
| |
| |
| | |
'master'
Raise InvalidRepository error for non-valid git repositories
See merge request gitlab-org/gitlab-ce!18594
|
| | | |
|
| |\ \
| |/
|/|
| |
| | |
Backport 5480-epic-notifications from EE
See merge request gitlab-org/gitlab-ce!18724
|
| | | |
|
| |\ \
| | |
| | |
| | |
| | | |
Backport of 1481-changing-weight-values-should-trigger-system-notes
See merge request gitlab-org/gitlab-ce!18699
|
| | | | |
|
| |\ \ \
| |_|/
|/| |
| | |
| | |
| | |
| | | |
CE backport of Backports Push Mirrors to CE
Closes #42099
See merge request gitlab-org/gitlab-ce!18715
|
| | | | |
|
| | | |
| | |
| | |
| | | |
Part of https://gitlab.com/gitlab-org/gitaly/issues/1138
|
| | | |
| | |
| | |
| | |
| | |
| | | |
Hooks were run for wikis, but given the internal API wasn't responding
failures happended, as seen on:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/18693
|
| |\ \ \ |
|
| | |\ \ \
| | |/ / |
|
| | | |\ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
'44059-specify-variables-when-executing-a-manual-pipeline-from-the-ui' into 'master'
Resolve "Specify variables when executing a manual pipeline from the UI"
Closes #44059
See merge request gitlab-org/gitlab-ce!18440
|
| | | | | | |
|
| | | | |\ \
| | | | | |
| | | | | |
| | | | | | |
44059-specify-variables-when-executing-a-manual-pipeline-from-the-ui
|
| | | | |\ \ \
| | | | | | |
| | | | | | |
| | | | | | | |
44059-specify-variables-when-executing-a-manual-pipeline-from-the-ui
|
| | | | | | | | |
|
| | | | | | | | |
|
| | | |\ \ \ \ \
| | | | |_|/ /
| | | |/| | | |
|
| | | |\ \ \ \ \ |
|
| | | |\ \ \ \ \ \ |
|
| | | | |_|_|_|_|/
| | |/| | | | | |
|
| |\ \ \ \ \ \ \ \
| |/ / / / / / / |
|
| | | | | | | | | |
|
