| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| | |
|
| |\
| |
| |
| |
| | |
Eliminate cached N+1 queries for projects in Issue API
See merge request gitlab-org/gitlab-ce!19269
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
In CE, every `Issue` entity is also a `ProjectEntity`, which calls
`entity&.project.try(:id)` to show the project ID. In an API
request with 100 issues, this would hit the Rails statement cache 100 times
for the same project and cause unnecessary overhead as related models would
also be loaded.
In EE, we call `Issue#supports_weight?` for each issue, which then calls
`project&.feature_available?(:issue_weights)`. If the project is not
preloaded, this incurs additional overhead, as each individual Project
object has to be queried. This can lead to a significant performance hit.
In loading the CE project with 100 issues, this contributed to at least 22%
of the load time.
See https://gitlab.com/gitlab-org/gitlab-ce/issues/47031 for why testing this
is a bit tricky.
|
| |\ \
| |/
|/|
| |
| |
| |
| | |
Removal of API v3 from the codebase
Closes #36819
See merge request gitlab-org/gitlab-ce!18970
|
| | | |
|
| |/
|
|
| |
Identified while resolving conflicts in https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/5906
|
| |\
| |
| |
| |
| | |
Only preload member records for the relevant projects/groups/user in projects API
See merge request gitlab-org/gitlab-ce!18945
|
| | |
| |
| |
| | |
projects API
|
| |\ \
| |/
|/|
| |
| |
| |
| | |
Replace grape-route-helpers with our own grape-path-helpers
Closes #45718
See merge request gitlab-org/gitlab-ce!19240
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This gem (https://gitlab.com/gitlab-org/grape-path-helpers) makes a number of changes:
1. Brings in @mdelaossa's changes in https://github.com/reprah/grape-route-helpers/pull/21
2. Fixes some broken specs and code for Grape 1.0+
3. Optimizes the generation of paths by bringing in @dblessing's
HashWithIndifferentAccess changes in https://gitlab.com/gitlab-org/gitlab-ce/issues/45718#note_70123793
Closes #45718
|
| |\ \
| |/
|/|
| |
| |
| |
| | |
Resolve "Squash and merge in GitLab Core (CE)"
Closes #34591
See merge request gitlab-org/gitlab-ce!18956
|
| | | |
|
| |\ \ |
|
| | |\ \
| | | |
| | | |
| | | |
| | | | |
[master] Fix API to remove deploy key from project instead of deleting it entirely
See merge request gitlab/gitlabhq!2379
|
| | | | | |
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Ensure ApplicationSetting#performance_bar_allowed_group_id is properly set when retrieved from cache
Closes #46758
See merge request gitlab-org/gitlab-ce!19144
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
when retrieved from cache
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
| |\ \ \ \ \
| |/ / / /
|/| | | |
| | | | |
| | | | | |
Resolve "Clarify/Update docs/text about restrictions."
See merge request gitlab-org/gitlab-ce!17970
|
| | | |_|/
| |/| | |
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Expose artifacts_expire_at field for job entity in api
See merge request gitlab-org/gitlab-ce!18872
|
| | | | | |
| | | | |
| | | | |
| | | | | |
https://gitlab.com/gitlab-org/gitlab-ce/issues/45798
|
| | |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This mimics the same thing the RailsQueueDuration does for Rails controller
requests and will help diagnose issues with API delays.
Closes #46601
|
| |\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | |
| | | |
| | | | |
Disallow updating job status if the job is not running
Closes #46383
See merge request gitlab-org/gitlab-ce!19101
|
| | | | | |
|
| | | | | |
|
| |\ \ \ \
| |/ / /
|/| | |
| | | |
| | | | |
Don't pass the repository path to Gitlab Shell
See merge request gitlab-org/gitlab-ce!19011
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
This isn't a requirement, and removes deprecated method calls
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
'46740-projectmilestones-setting-start_date-via-the-api-requires-a-second-parameter' into 'master'
Resolve "ProjectMilestones: Setting start_date via the API requires a second parameter"
Closes #46740
See merge request gitlab-org/gitlab-ce!19133
|
| | | |/ /
| |/| | |
|
| |/ / /
| | |
| | |
| | |
| | |
| | | |
without a .git directory
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
| | | |
| | |
| | |
| | |
| | |
| | | |
ApplicationSetting.current where applicable
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
API: Keyset pagination support
Closes #45756
See merge request gitlab-org/gitlab-ce!18584
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
Closes #45756.
|
| | | | | |
|
| | | | | |
|
| |/ / / |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Append trace chunk only if job is running
Closes #46383
See merge request gitlab-org/gitlab-ce!18969
|
| | | | | |
|
| | | | | |
|
| |\ \ \ \
| |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | | |
Resolve "API naming for issue scope"
Closes #44799
See merge request gitlab-org/gitlab-ce!18935
|
| | | | |
| | | |
| | | |
| | | | |
Deprecate corresponding dash versions created-by-me and assigned-to-me
|
| | | | | |
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Fix api_json.log not always reporting the right HTTP status code
See merge request gitlab-org/gitlab-ce!19028
|
| | | |/ /
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
As described in https://github.com/aserafin/grape_logging/issues/45, if
a Grape error is caught by the handlers and a different return code
is returned, then the api_json.log would have a 500 error code
instead of the right value. Inserting the GrapeLogging middleware
after the Grape middleware fixes this problem.
Seen in https://gitlab.com/gitlab-com/infrastructure/issues/4249
|
| |/ / / |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Delete remote uploads
Closes #45425
See merge request gitlab-org/gitlab-ce!18698
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
by policy
|
