| 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
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
'refine-docs-trigger-script-and-use-a-really-long-branch-name-to-test-it' into 'master'
Refine docs trigger script and use a really long branch name to test it
For branch names more than 63 characters we were hitting the
filesystem limit and rsync created a truncated directory.
This change truncates the branch name beforehand so the Review
App URL is properly shown in the merge request widget.
See merge request gitlab-org/gitlab-ce!14354
|
|/ / |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
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
|
|\ \
| | |
| | |
| | |
| | | |
Detect n+1 issues involving Gitaly
See merge request gitlab-org/gitlab-ce!13996
|
|/ / |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
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
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Fix admin area selector in GitLab QA
Closes gitlab-qa#66
See merge request gitlab-org/gitlab-ce!14356
|
| | |/
| |/|
| | |
| | | |
Fixes gitlab-org/gitlab-qa#66
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Improve the Users API specs by asserting against the response schema
See merge request gitlab-org/gitlab-ce!14334
|
| | | |
| | | |
| | | |
| | | | |
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
|\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | |
| | | |
| | | | |
Port of EE translations
Closes #37914
See merge request gitlab-org/gitlab-ce!14340
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | | |
New Crowdin translations
See merge request gitlab-org/gitlab-ee!2929
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Allow DEVELOPER role to admin milestones
Closes #32215
See merge request gitlab-org/gitlab-ce!14257
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Change all instances of btn-sm to btn-small to keep things consistent
See merge request gitlab-org/gitlab-ce!14113
|
|/ / / / / |
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Allow updating of project auto devops settings
Closes #37893
See merge request gitlab-org/gitlab-ce!14291
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
When a user selected 'Instance default' as value for the auto devops
settings, this was interpreted as a true value. Now we post an empty
string in this case, meaning we want to set `NULL` in the database.
Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/37893#note_40541294
|
| | | | | | |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Updating didn't work, as the project_id was missing to be set for the
project_auto_devops model.
Fixes gitlab-org/gitlab-ce#37893
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Fixes project denial of service via gitmodules using Extended ASCII.
Closes #34259
See merge request gitlab-org/gitlab-ce!14301
|
| | | | | | | |
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Repo editor UI fixes part 2
Closes #37830, #36247, #37829, and #36244
See merge request gitlab-org/gitlab-ce!14261
|
| | | | | | | | |
|
| | | | | | | | |
|
| | | | | | | | |
|
| | | | | | | | |
|
|\ \ \ \ \ \ \ \
| |_|_|_|_|_|_|/
|/| | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Fix MR widget with external CI services/integrations
Closes #33287
See merge request gitlab-org/gitlab-ce!13977
|
|/ / / / / / /
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
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
|
|\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
fix #35290 Make read-only API for public merge requests available without authentication
Closes #35290
See merge request gitlab-org/gitlab-ce!13291
|
|/ / / / / / / /
| | | | | | | |
| | | | | | | |
| | | | | | | | |
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
|
| | |/ / / / / / / /
| |/| | | | | | | | |
|
|\ \ \ \ \ \ \ \ \ \
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Add missing import statements
See merge request gitlab-org/gitlab-ce!14209
|
| | |_|_|_|_|/ / / /
| |/| | | | | | | | |
|
|\ \ \ \ \ \ \ \ \ \
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Fixes milestone issuable assignee URL
Closes #33124
See merge request gitlab-org/gitlab-ce!14292
|