| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|\
| |
| |
| |
| | |
Reserve more RAM for master process in Puma
See merge request gitlab-org/gitlab-ce!22741
|
| |
| |
| |
| |
| |
| | |
The Puma Worker Killer checks the total RAM used by both the master
and worker processes. Bump the limits to N+1 instead of N workers
to account for this.
|
| | |
|
|\ \
| | |
| | |
| | |
| | | |
Fix typos in comments and specs
See merge request gitlab-org/gitlab-ce!22683
|
| | | |
|
|\ \ \
| |_|/
|/| |
| | |
| | | |
Reduce SQL queries needed to load open merge requests
See merge request gitlab-org/gitlab-ce!22709
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The SQL queries and memory allocation in MergeRequests::RefreshService
is dominated by queries for Project and Route loads. On staging, the
absence of an inverse relationship caused Rails to make over 1100
extraneous SQL queries for the www-gitlab-com repository.
Relates to https://gitlab.com/gitlab-org/gitlab-ce/issues/49703
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Whitelist none method from ActiveRecord::Querying
See merge request gitlab-org/gitlab-ce!22726
|
| | | | |
|
|/ / / |
|
|\ \ \
| |/ /
|/| |
| | |
| | | |
Split CE and EE params in issues API
See merge request gitlab-org/gitlab-ce!22689
|
| | | |
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Add language param to highlight
See merge request gitlab-org/gitlab-ce!21584
|
| | | |
| | | |
| | | |
| | | | |
This gives flexiblity to avoid duplicated query of Project.
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Extract language_from_git_attributes as a concern so it can
ben included in two blob classes.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This is to DRY the repeated file size check.
Move spec and constants to Highlight
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This replaces the repository param.
This allows more flexiblity as sometimes we have highlight content
not related to repository. Sometimes we know ahead of time the language
of the content. Lastly language determination seems better fit as a
logic in the Blob class.
`repository` param is only used to determine the language, which seems
to be the responsiblity of Blob.
|
| | | |
| | | |
| | | |
| | | | |
Force FoundBlob to use BlobPresenter
|
| | |/
| |/|
| | |
| | |
| | | |
Ported from Highlight class since it as a concept is more related to
blob, and this allows more flexibility.
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Move Repository#wrapped_gitaly_errors into concern
See merge request gitlab-org/gitlab-ce!22691
|
| |/ /
| | |
| | |
| | |
| | |
| | | |
Having this in a concern allows us to reuse it for different single
purpose classes that call out to git without going through the
repository every time.
|
|\ \ \
| |/ /
|/| |
| | |
| | | |
Enable some frozen string for lib/gitlab/ci
See merge request gitlab-org/gitlab-ce!22636
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Enables frozen string for the following:
* lib/gitlab/ci/*.rb
* lib/gitlab/ci/build/**/*.rb
* lib/gitlab/ci/config/**/*.rb
* lib/gitlab/ci/pipeline/**/*.rb
* lib/gitlab/ci/reports/**/*.rb
Partially addresses #47424.
|
|\ \ \ |
|
| |\ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Replace deprecated uniq on a Relation with distinct
See merge request gitlab-org/gitlab-ce!22625
|
| | | | | |
|
| | | | | |
|
| | |_|/
| |/| | |
|
| |\ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
'52780-stale-pipeline-status-cache-for-_project-after-disabling-pipelines' into 'master'
Resolve "Stale pipeline status cache for `_project` after disabling pipelines"
Closes #52780
See merge request gitlab-org/gitlab-ce!22589
|
| | | | | |
|
| | | | | |
|
| | |_|/
| |/| | |
|
| |\ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
User not defined in PostReceive#process_project_changes
Closes #51335
See merge request gitlab-org/gitlab-ce!22519
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Before we would need to identify a user when pushing
through the GitLab UI. Since this is no longer the case
we can remove the identification by commit and instead,
use the identify_using_user
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
When Gitlab::GitPostReceive#changes_refs is empty
user would not get defined and nil would be passed
to PostReceive#after_project_changes_hooks which would
then throw an error.
|
| |\ \ \ \
| | |_|_|/
| |/| | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
'42790-improve-feedback-for-internal-git-access-checks-timeouts' into 'master'
Adds trace of each access check when git push times out
Closes #42790
See merge request gitlab-org/gitlab-ce!22265
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Validating each commit on ChangeAccess times out if
it already took too long to complete.
Improves the TimedLogger specs to not make use of a stubbed
class anymore
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Adds a #full_message method so that external classes
do not have access to the state of the logger.
Adds a #append_message to always append to the array in-place
|
| | |/ /
| | | |
| | | |
| | | |
| | | |
| | | | |
Whenever a git push takes more than 50 seconds the user
will receive a trace from each check performed along with
their timings
|
| | | | |
|
| |\ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
CE: Put EE routes in EE files under EE directories
See merge request gitlab-org/gitlab-ce!22376
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | | |
And if it cannot find any routes, raise an error
|
| | | | | |
|
| |\ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Resolve "Filter by `None`/`Any` for assignee_id in issues/mrs API"
Closes #52384
See merge request gitlab-org/gitlab-ce!22598
|
| | | | | | |
|