| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| | |
Change HELM_HOST in Auto-DevOps to work behind proxy
See merge request gitlab-org/gitlab-ce!22596
|
| | |
|
|\ \
| | |
| | |
| | |
| | | |
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
|
|/ / / |
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
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
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | | |
And if it cannot find any routes, raise an error
|
| | | | |
|
| |\ \ \
| | |_|/
| |/| |
| | | |
| | | |
| | | |
| | | | |
Puma in GDK and rack server lifecycle event abstractions
Closes #52762
See merge request gitlab-org/gitlab-ce!22372
|
| | |/
| | |
| | |
| | |
| | |
| | | |
This allows us (and others) to test drive Puma without it affecting all
users. Puma can be enabled by setting the environment variable
"EXPERIMENTAL_PUMA" to a non empty value.
|
| | |
| | |
| | |
| | |
| | | |
[11.4] Fix Token lookup for Git over HTTP and registry authentication
See merge request gitlab/gitlabhq!2577
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
[master] Redact unsubscribe links in issuable texts
See merge request gitlab/gitlabhq!2528
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
It's possible that user pastes accidentally also unsubscribe link
which is included in footer of notification emails. This unsubscribe
link contains personal token which attacker then use to act as the
original user (e.g. for sending comments under his/her identity).
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
'master'
[master] JUnit test reports endpoint exposes full stack trace in production mode
See merge request gitlab/gitlabhq!2559
|
| |/ / / |
|
| | | | |
|