Age | Commit message (Collapse) | Author |
|
Move the methods to detect if we can access disk directly into
StorageSettings. This reduces complexity by no longer needing
synchronization, as well as being able to bypass the disk access check.
|
|
Disabling can_use_disk? temporarily
See merge request gitlab-org/gitlab-ce!30526
|
|
[CE port] Bypass push rules for merge to ref service
See merge request gitlab-org/gitlab-ce!30426
|
|
|
|
Turns out push rules to validate commit message does
not apply in the context of automatic merge to
the refs/merge-requests/:iid/merge. Mainly because
if it fails to merge to it, we currently can't give enough
preemptive feedback to the user and it'll turn the
merge request unmergeable (given we automatically
mark it as unmergeable if we can't merge to the ref).
In general, it's a systematic operation, which already
bypasses user authorization and git hooks.
Therefore, this commit makes it bypass the push rules
at EE as well.
|
|
Perform more redactions in Redis performance bar traces
Closes #64309
See merge request gitlab-org/gitlab-ce!30514
|
|
Task list updates should not mark users action as spam on akismet.
|
|
Fix dropdown styling issues
Closes #64250
See merge request gitlab-org/gitlab-ce!30477
|
|
HMSET and AUTH commands were not properly redacted. This commit
does that and adds a test.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/64309
|
|
Converted storage details into a small partial
Reworded the storage details summary as suggested
Updated pot file
|
|
- Remove `FeatureConstrainer` call wrapping api endpoint
- Remove `Feature.enabled?(:graphql)` conditionals in back and frontend
- Modify graphql test to be graphql flag agnostic
- Remove api routing spec
- Remove frontend feature flag via `gon`
|
|
Add additional condition to count median properly
See merge request gitlab-org/gitlab-ce!30229
|
|
This is a port MR for CE
Original MR:
https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/14141
|
|
Inside a subgroup scope, when the user clicks on the
search drop down in the header, some options are displayed.
Some of this links direct the user to the issues and merge
requests linked to the subgroup. Nevertheless, the url
was broken because they doesn't include the complete
group namespace
|
|
New API endpoint for merge request count
Updates all open tabs at the same time with one call
Restructured API response
API response changed to 401 if no current_user
Added API + JS specs
Fix for Static Check
Updated Count on Open/Close, Assign/Unassign of MR's
Checking if MR Count is refreshed
Added # frozen_string_literal: true to spec
Added Changelog
|
|
|
|
Fix typo on confidential MR dropdown
See merge request gitlab-org/gitlab-ce!30499
|
|
This interferes spec execution on local environment.
|
|
Change issues to issue's
|
|
- applied functional component patch
- fixed border radius for list
- fixed Karma test to take a correct li amount
|
|
Allow ReactiveCaching to support nil value
See merge request gitlab-org/gitlab-ce!30456
|
|
Schedules a Namespace::AggregationSchedule worker if some of the project
statistics are refreshed.
The worker is only executed if the feature flag is enabled.
|
|
When :calculate_reactive_caching returns a nil value
this caused ReactiveCaching to schedule a worker
every time the code using :with_reactive_cache was called.
This issue caused an increasing amount of Sidekiq jobs
being created continuously.
Implementing this fix behind feature flag
:reactive_caching_check_key_exists
|
|
CE Port: Fix race condition of `refs/merge` competing overwrite
See merge request gitlab-org/gitlab-ce!30308
|
|
Use Rugged if we detect storage is NFS and we can access the disk
See merge request gitlab-org/gitlab-ce!29725
|
|
Reduce N+1 queries in MergeRequestsController#ci_environment_status
See merge request gitlab-org/gitlab-ce!30224
|
|
Remove minimum required characters for fuzzy search if no trigram is used
See merge request gitlab-org/gitlab-ce!29810
|
|
As a workaround for https://gitlab.com/gitlab-org/gitlab-ce/issues/64116
|
|
Replace runtime isEE check with compile time one
Closes #60213
See merge request gitlab-org/gitlab-ce!30351
|
|
Today, Pipelines for merge train run on `refs/merge`,
however, this causes a race condition that it can be
overwritten by CheckMergeabilityService.
This patch fixes the problem by generating `refs/train`
for those pipelines.
|
|
Resolve "Removing an image via Web IDE shows binary data"
Closes #60856
See merge request gitlab-org/gitlab-ce!30314
|
|
- This enables us to use a scope to query all clusters in group
hierarchy order in one query, and also enables us to union to instance
clusters later.
- Handle case where clusters not present at level. In which case the
query should go ahead and return the next level's clusters.
- Swap with new CTE query behind Feature flag. This FF is default
disabled.
|
|
In 12.2 we will remove this fallback.
|
|
We stopped calling the fallback so we can remove this now
|
|
Split into followup issue
|
|
We can now share project so that we don't have to load project twice.
Also, this extracts non-relevant logic out of Deployment.
Update DeploymentsController accordingly
|
|
Otherwise, each EnvironmentStatus object instantiates its own project
when really they are the same. Improves query count performance
|
|
This improves query performance of
MergeRequestsController#ci_environments_status a lot.
However this means old deployments that deployed to kubernetes clusters
with prometheus installations will no longer show performance metrics as
we cannot backfill cluster_id from deployment_platform with certainty
(clusters may be edited/added/deleted, which changes the results of
deployment_platform).
|
|
We have an N+1 problem where N is environments.
|
|
When `ReactiveCachingWorker` hits an SSL or other exception that occurs
quickly and reliably, automatically rescheduling a new worker could lead
to excessive number of jobs being scheduled. This happens because not
only does the failed job get rescheduled in a minute, but each Sidekiq
retry will also add even more rescheduled jobs.
In busy instances, this can become an issue because large numbers of
`ReactiveCachingWorker` running can cause high rates of `ExclusiveLease`
reads to occur and possibly saturate the Redis server with queries.
We now disable this automatic retry and rely on Sidekiq to perform its 3
retries with a backoff period.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/64176
|
|
|
|
Some services can raise OpenSSL::X509::CertificateError due
to an invalid SSL certificates, with OpenSSL::OpenSSLError
we can handle these errors.
|
|
Add a module we use as a singleton to determine whether or not rails is
able to access the disk
|
|
This has the benefit that CE bundles will remove dead code in CE only
builds.
We have a follow-up issue to get rid of this completely:
https://gitlab.com/gitlab-org/gitlab-ce/issues/64173
|
|
Adding order by to list runner jobs api.
Closes #51794
See merge request gitlab-org/gitlab-ce!29629
|
|
Important to get file data for any file that has no content set yet: we
need this information in order to build a previe for changed/staged
files if they were not yet opened in the editor
|
|
CE Port: Extend `MergeToRefService` to create merge ref from an arbitrary ref
See merge request gitlab-org/gitlab-ce!30361
|
|
CE Port: Split AutoMergeService interfaces into two `cancel` and `abort`
See merge request gitlab-org/gitlab-ce!30249
|
|
Create private merge requests in forks
Closes #58583
See merge request gitlab-org/gitlab-ce!29984
|
|
Fix divergence graph loading error
Closes #64143
See merge request gitlab-org/gitlab-ce!30393
|