| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
|\
| |
| |
| |
| |
| |
| | |
Fix the filesystem shard health check to check all configured shards
Closes #37999
See merge request gitlab-org/gitlab-ce!14341
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Remove unnecessary loading of discussions in `IssuesController#show`
Closes #38034
See merge request gitlab-org/gitlab-ce!14351
|
| |/
| |
| |
| |
| |
| |
| |
| | |
Discussions are now done asynchronously via the `IssuesController#discussions`
endpoint, so this should no longer be needed. This was taking 32% of the load
time for GitLab CE issue 1.
Closes #38034
|
|\ \
| |/
|/|
| |
| |
| |
| | |
Eliminate N+1 queries referencing issues
Closes #38033
See merge request gitlab-org/gitlab-ce!14349
|
| |
| |
| |
| |
| |
| |
| |
| | |
To load issue 1, we see that in #38033 that about 835 ms of the
SQL queries were due to loading ProjectFeature. We should be
able to cut this down by eagerly loading this information.
Closes #38033
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Allow DEVELOPER role to admin milestones
Closes #32215
See merge request gitlab-org/gitlab-ce!14257
|
| | | |
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Fixes project denial of service via gitmodules using Extended ASCII.
Closes #34259
See merge request gitlab-org/gitlab-ce!14301
|
| | | | |
|
| |_|/
|/| |
| | |
| | |
| | |
| | |
| | |
| | | |
Fix https://gitlab.com/gitlab-org/gitlab-ce/issues/33287
The MR widget was trying to render the pipelines section when
there are no GitLab CI pipelines which was throwing some NPE
errors.
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Fix MR widget with ready to merge buttons/controls at mobile breakpoint
Closes #37259
See merge request gitlab-org/gitlab-ce!14242
|
| | |/
| |/|
| | |
| | | |
Fix https://gitlab.com/gitlab-org/gitlab-ce/issues/37259
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Update "x/x discussions resolved" checkmark icon to be green when all discussions resolved
Closes #37465
See merge request gitlab-org/gitlab-ce!14255
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Fix https://gitlab.com/gitlab-org/gitlab-ce/issues/37465
Related MRs
- https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/13821
- https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/14042
|
| | | |
| | | |
| | | |
| | | | |
authentication
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Add 'closed_at' attribute to Issues API
Closes #5935
See merge request gitlab-org/gitlab-ce!14316
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Eliminate N+1 queries in loading discussions.json endpoint
Closes #37955
See merge request gitlab-org/gitlab-ce!14327
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
In #37955,we see that the profile had a number of N+1 queries from repeated
access to `cross_reference_not_visible_for?`. This was optimized in previous
versions of GitLab by rendering all notes at once, counting the number of
visible references, and then using that number to check whether a system note
should be fully redacted.
There was also another N+1 query calling `ProjectTeam#member?`, which did not
take advantage of an optimization in prepare_notes_for_rendering that would
preload the maximum access level per project.
Closes #37955
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Fix mini graph pipeline breaking in merge request view
Closes #37590
See merge request gitlab-org/gitlab-ce!14306
|
| | |/ / / /
| |/| | | | |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Fixes milestone issuable assignee URL
Closes #33124
See merge request gitlab-org/gitlab-ce!14292
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Closes #33124
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
'master'
Fix docs for lightweight tag creation via API
Closes #17499
See merge request gitlab-org/gitlab-ce!14328
|
| | | | | | | | |
|
|\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Replace the 'project/snippets.feature' spinach test with an rspec analog
See merge request gitlab-org/gitlab-ce!14326
|
| | | | | | | | | |
|
|\ \ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Extract AutocompleteController#users into finder
Closes #24121
See merge request gitlab-org/gitlab-ce!13778
|
| | | | | | | | | | |
|
| | | | | | | | | | |
|
|\ \ \ \ \ \ \ \ \ \
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Replace the 'project/commits/revert.feature' spinach test with an rspec analog
See merge request gitlab-org/gitlab-ce!14325
|
| | |/ / / / / / / /
| |/| | | | | | | | |
|
|\ \ \ \ \ \ \ \ \ \
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Replace the 'project/archived.feature' spinach test with an rspec analog
See merge request gitlab-org/gitlab-ce!14322
|
| |/ / / / / / / / / |
|
|\ \ \ \ \ \ \ \ \ \
| |/ / / / / / / / /
|/| | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Replace the 'search.feature' spinach test with an rspec analog
See merge request gitlab-org/gitlab-ce!14248
|
| | |_|_|/ / / / /
| |/| | | | | | | |
|
| |_|_|_|/ / / /
|/| | | | | | | |
|
|\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
# Conflicts:
# db/schema.rb
|
| | | | | | | | | |
|
| |\ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Make the labels in the Compare form less confusing
Closes #21331
See merge request gitlab-org/gitlab-ce!14225
|
| | |/ / / / / / /
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
* Improve the descriptive text
* Rename "from" to "Target" and "to" to "Source"
* Swap "Target" and "Source" to have the same order as in MRs
* Reworded "Switch base of comparison" to "Swap versions"
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
| |\ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
SQL performance improvements for ProjectsController#show
See merge request gitlab-org/gitlab-ce!14226
|
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
This ensures that if a pipeline is present for the last commit on a
project's homepage we only run 1 query to get the builds, instead of
running 2 queries.
See https://gitlab.com/gitlab-org/gitlab-ce/issues/36878#note_40073339
for more information.
|
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
This adds Project#latest_successful_pipeline_for and
Project#latest_successful_pipeline_for_default_branch. The 2nd method
memoizes the result (taking nil values into account) to ensure the
underlying query isn't executed multiple times when viewing a project's
homepage.
See https://gitlab.com/gitlab-org/gitlab-ce/issues/36878#note_40073607
for more information.
|
| | |/ / / / / / /
| |/| | | | | | | |
|
| |\ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Resolve "Scope users by group inside dropdowns if I'm on group page (Issue, MRs)"
Closes #20824
See merge request gitlab-org/gitlab-ce!14249
|
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Signed-off-by: Rémy Coutable <remy@rymai.me>
|