| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| | |
Implement suggestion diff lines parser
See merge request gitlab-org/gitlab-ce!26069
|
| | |
|
| |
| |
| |
| | |
Replaced Settings with Gitlab.config for uniformity
|
|\ \
| | |
| | |
| | |
| | | |
Move EE specific code out of lib/gitlab
See merge request gitlab-org/gitlab-ce!25741
|
| | |
| | |
| | |
| | |
| | | |
This backports the changes EE made to Gitlab::UserExtractor, removing
the need for an EE specific module.
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | | |
This backports support for finding EE workers in ee/app/workers from EE
to CE, since we already do this for the configuration paths constant.
This removes the need for EE specific modules.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
EE adds this method to Gitlab::Utils, which is also required by our
SimpleCov helper. This prevents us from injecting EE modules into
Gitlab::Utils, because the necessary bits for this are not yet in place.
To work around this we just backport try_megabytes_to_bytes, as there's
no particular reason to keep this in EE only.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This backports the allow_mirror_update update field for Chain::Command
from EE to CE. There's no particular reason to keep this in EE. Doing so
would require refactoring of CE as there currently is no easy way to
inject new Struct members into the Chain::Command class.
|
| | |
| | |
| | |
| | |
| | |
| | | |
EE added the ability to set timeouts when handling blobs. Since there's
no particular reason for this to be EE specific we can just backport
this to CE.
|
| | |
| | |
| | |
| | |
| | | |
This moves the development favicon name in Gitlab::Favicon to a separate
method, allowing EE to redefine it to return a custom favicon.
|
| | |
| | |
| | |
| | |
| | | |
This backports the EE changes to Gitlab::Ci::Model back into CE,
ensuring the code is the same for both versions.
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Fix storage migration check for non-empty queues
Closes #58739
See merge request gitlab-org/gitlab-ce!26109
|
| | | |
| | | |
| | | |
| | | |
| | | | |
The previous code works fine when running specs, but the API
does not exist outside that environment.
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
'ce-10421-implement-the-retrieval-reading-of-the-insights-file-to-be-used-in-the-insights-controller' into 'master'
New Repository#insights_config method
Closes gitlab-ee#10421
See merge request gitlab-org/gitlab-ce!26183
|
| | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
|/ / / /
| | | |
| | | |
| | | | |
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Update clair-local-scan to 2.0.6
Closes #58482
See merge request gitlab-org/gitlab-ce!25743
|
| | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
|
|\ \ \ \ \
| |_|/ / /
|/| | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Remove timeout for GitalyClient remove_remote
Closes #56015
See merge request gitlab-org/gitlab-ce!26213
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This can be slow because it must potentially remove a lot of remote
refs (if the remote being removed has a lot of refs)
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
From pg_stat_wal_receiver_supported? to
postgresql_minimum_supported_version?
Also add test coverage
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Explicitly set master_auth for new GKE clusters
Closes #58208
See merge request gitlab-org/gitlab-ce!26018
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
From 1.12, GKE will disable basic auth and client certificate by
default. Explicitly enable those now (currently we use 1.10/1.11
clusters) so that GKE cluster configuration will continue to work.
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Update rubocop target ruby version
See merge request gitlab-org/gitlab-ce!24804
|
| |/ / / / / |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Use project name and ID in Review App environment URLs instead of the project path slug
Closes #52990
See merge request gitlab-org/gitlab-ce!25990
|
| |/ / / / / |
|
|\ \ \ \ \ \
| |_|_|/ / /
|/| | | | |
| | | | | |
| | | | | | |
Update recaptcha gem from 3.x to 4.11
See merge request gitlab-org/gitlab-ce!25921
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
In recaptcha 4.0.0 there was an API change:
- `public_key` -> `site_key`
- `private_key` -> secret_key
See:
https://github.com/ambethia/recaptcha/blob/master/CHANGELOG.md
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Allow filtering labels by a single character
Closes #58795
See merge request gitlab-org/gitlab-ce!26012
|
| | |_|_|/ /
| |/| | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
When we use Gitlab::SQL::Pattern, this typically relates to a trigram
index. As the 'tri' indicates, we need at least three characters to be
able to use that index.
Labels don't have a trigram index, because we never allow you to search
for them globally: it's always in the context of a project or a
group. In that context, it's just as fast to search for a single
character (in general) because there is already a pretty specific index
being used.
|
| | | | | | |
|
| |_|/ / /
|/| | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Including a CODEOWNERS file with lines without any matching username or
e-mail regular expressions would cause an Error 500. Don't attempt a
database query if there is nothing to query.
Closes https://gitlab.com/gitlab-org/gitlab-ee/issues/10282
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Fix health checks not working behind load balancers
Closes #58573
See merge request gitlab-org/gitlab-ce!26055
|
| | |_|_|/
| |/| | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The change in
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/24199 caused
requests coming from a load balancer to arrive as 127.0.0.1 instead of
the actual IP.
`Rack::Request#ip` behaves slightly differently different than
`ActionDispatch::Request#remote_ip`: the former will return the first
X-Forwarded-For IP if all of the IPs are trusted proxies, while the
second one filters out all proxies and falls back to REMOTE_ADDR, which
is 127.0.0.1.
For now, we can revert back to using `Rack::Request` because these
middlewares don't manipulate parameters. The actual fix problem involves
fixing Rails: https://github.com/rails/rails/issues/28436.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/58573
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This produces stable approximate counts among invocations unless the
table changed.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/58791.
|
|\ \ \ \ \
| |/ / / /
|/| | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
'58739-hashed-storage-prevent-a-migration-and-rollback-running-at-the-same-time' into 'master'
Hashed Storage: Prevent a migration and rollback running at the same time
Closes #58739
See merge request gitlab-org/gitlab-ce!25976
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This is a small polishing on the storage migration and storage rollback
rake tasks. By aborting a migration while a rollback is already
scheduled we want to prevent unexpected consequences.
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Fix counting of groups in admin dashboard
Closes gitlab-ee#7435
See merge request gitlab-org/gitlab-ce!26009
|
| | |/ / /
| |/| | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
1. Ignore tables that use STI in reltuples count strategy.
Models that use Rails' single-type inheritance, such as `Group` and
`CiService`, need an additional WHERE clause to count the total
properly, which isn't supported by the reltuples strategy. For now,
we just omit these from the statistics sampling and rely on the other
strategies to get this data.
2. Fix tablesample count strategy not counting groups properly.
Models such as `Group` needs a WHERE clause to distinguish it from
namespaces. We now add in the WHERE clause if STI is in use.
Closes https://gitlab.com/gitlab-org/gitlab-ee/issues/7435
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Resolve CE to EE differences in the lib/api directory
See merge request gitlab-org/gitlab-ce!25430
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This allows EE to customize the parameters used in two places, without
having to modify the source code directly.
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This backports a line from EE so that both the CE and EE code are
identical.
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This refactors API::Settings so that EE can more easily extend its
behaviour, without having to modify the CE source code directly.
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This moves various constants previously defined in API::Services to the
API::Helpers::ServicesHelpers module, using class methods instead of
constants. This allows EE to extend the returned values, without having
to modify the CE source code directly.
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This refactors API::Search so that Enterprise Edition can more easily
extend its functionality, without having to modify the file directly.
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This moves API::ResourceLabelEventsHelpers::EVENTABLE_TYPES to a class
method in a separate helper module, allowing EE to redefine it without
having to directly modify the source code.
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This adds a method to API::Internal that returns the URL to use for LFS
authentication tokens. This makes it possible for EE to redefine this
behaviour, without having to directly modify the source code.
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Similar to creating groups in the groups API, EE injects custom code
just before we update an existing group. Moving the update logic to a
separate method in CE allows EE to more easily extend it.
|