| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |\
| |
| |
| |
| |
| |
| | |
Resolve "Follow-up from "Visual Review Toolbar Docs""
Closes #63362
See merge request gitlab-org/gitlab-ce!30040
|
| |/
|
|
| |
Reorganize and better explain what Visual Reviews are
|
| |\
| |
| |
| |
| | |
CE Port: Fix race condition of `refs/merge` competing overwrite
See merge request gitlab-org/gitlab-ce!30308
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| |\ \
| | |
| | |
| | |
| | | |
Use Rugged if we detect storage is NFS and we can access the disk
See merge request gitlab-org/gitlab-ce!29725
|
| | | |
| | |
| | |
| | |
| | | |
Add a module we use as a singleton to determine whether or not rails is
able to access the disk
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Reduce N+1 queries in MergeRequestsController#ci_environment_status
See merge request gitlab-org/gitlab-ce!30224
|
| | | | |
| | | |
| | | |
| | | | |
As a workaround for https://gitlab.com/gitlab-org/gitlab-ce/issues/64116
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Also memoize has_metrics? as well, that might be expensive, and it
should not change for the lifetime of EnvironmentStatus
|
| | | | |
| | | |
| | | |
| | | | |
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.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Upgrade prometheus-client-mmap to 0.9.7
See merge request gitlab-org/gitlab-ce!30430
|
| | | | | |
| | | | |
| | | | |
| | | | | |
This removes a warning message when prometheus_multiproc_dir is not set.
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Remove a high cardinality Prometheus metric
See merge request gitlab-org/gitlab-ce!30122
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
The metric was used to correlate Gitaly requests to the Rails controller
and action combination. However, Kibana provides better observability in
this specific metric, and can handle hig cardinality much better.
There's no dashboard in Grafana that currently depends on this metric
being exposed.
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Add lint rule that headings must increment one level at a time
See merge request gitlab-org/gitlab-ce!30376
|
| |/ / / / / /
| | | | | |
| | | | | |
| | | | | | |
Also fixes instances in Markdown that broke the rule.
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Starting a new discussion only on line without Draft note created on it
See merge request gitlab-org/gitlab-ce!30413
|
| |/ / / / / /
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
- fixed attribute to be not hyphenated
- applied parallel diffs view on CE
- separated left and right drafts
- added fallback for hasDrafts
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Remove minimum required characters for fuzzy search if no trigram is used
See merge request gitlab-org/gitlab-ce!29810
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
If we're not using a trigram index, then ignore the
min_chars_for_partial_matching setting
|
| |\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Docs: Changing badges to use parentheses not brackets
See merge request gitlab-org/gitlab-ce!30439
|
| |/ / / / / / /
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Previously, we used brackets to denote the tier badges,
but this made Kramdown, the docs site Markdown renderer,
show many warnings when building the site. This is now
fixed by using parentheses instead of square brackets.
This was caused by [PREMIUM] looking like a link to
Kramdown, which couldn't find a URL there.
See:
- https://gitlab.com/gitlab-com/gitlab-docs/merge_requests/484
- https://gitlab.com/gitlab-org/gitlab-ce/issues/63800
|
| |\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Replace runtime isEE check with compile time one
Closes #60213
See merge request gitlab-org/gitlab-ce!30351
|
| | | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
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
|
| |\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Vue-i18n: autofix for app/assets/javascripts/vue_shared directory
See merge request gitlab-org/gitlab-ce!30287
|
| |/ / / / / / / /
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
i18n linting for .vue files under the
app/assets/javascripts/vue_shared directory
|
| |\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Vue-i18n: autofix for app/assets/javascripts/vue_merge_request_widget directory
See merge request gitlab-org/gitlab-ce!30035
|
| |/ / / / / / / /
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
i18n linting for .vue files under the
app/assets/javascripts/vue_merge_request_widget directory
|
| |\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Update dependency @gitlab/svgs to ^1.67.0
See merge request gitlab-org/gitlab-ce!30402
|
| | | | | | | | | | |
|
| |\ \ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Removes EE differences for app/views/admin/users/show.html.haml
Closes gitlab-ee#12533
See merge request gitlab-org/gitlab-ce!30407
|
| | | | | | | | | | | |
|
| |\ \ \ \ \ \ \ \ \ \
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Resolve "Removing an image via Web IDE shows binary data"
Closes #60856
See merge request gitlab-org/gitlab-ce!30314
|
| | | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
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
|
| | | |_|/ / / / / / /
| |/| | | | | | | | |
|
| |\ \ \ \ \ \ \ \ \ \
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Clusters hierarchy CTE
See merge request gitlab-org/gitlab-ce!30063
|
| |/ / / / / / / / / /
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
- 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.
|
| |\ \ \ \ \ \ \ \ \ \
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Description of the optimized path wildcard
See merge request gitlab-org/gitlab-ce!30082
|
| |/ / / / / / / / / / |
|
| |\ \ \ \ \ \ \ \ \ \
| |_|_|_|_|_|_|_|_|/
|/| | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Docs: Update Jira integration
See merge request gitlab-org/gitlab-ce!29997
|
| |/ / / / / / / / /
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Current screenshots are from an older version of Jira and GitLab.
|
| |\ \ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
ADD '.git' ending to git clone with CI_JOB_TOKEN for clarity
See merge request gitlab-org/gitlab-ce!30200
|
