summaryrefslogtreecommitdiff
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
* Add github imported projects count to usage data34563-usage-ping-githubJarka Kadlecova2017-07-191-0/+1
|
* Fix queries duration sorting in Performance Bar35252-fix-performance-bar-queries-duration-sortingRémy Coutable2017-07-181-1/+1
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Merge branch 'backstage/gb/add-index-and-foreign-key-to-merge-requests' into ↵Kamil Trzciński2017-07-181-1/+3
|\ | | | | | | | | | | | | | | | | 'master' Add a foreign key to `merge_requests.head_pipeline_id` Closes #34065 See merge request !12837
| * Add a foreign key to `merge_requests.head_pipeline_id`Grzegorz Bizon2017-07-181-1/+3
| |
* | Merge branch 'request-store-wrap' into 'master'Rémy Coutable2017-07-182-4/+104
|\ \ | | | | | | | | | | | | Add RequestCache to cache via RequestStore See merge request !12920
| * | Rename the methods to make it fit with current nameLin Jen-Shin2017-07-182-14/+14
| | |
| * | Follow feedback on the merge requestLin Jen-Shin2017-07-182-15/+11
| | |
| * | Update the comments for the new functionalityLin Jen-Shin2017-07-181-3/+19
| | |
| * | Use RequestStoreWrap for Commit#authorLin Jen-Shin2017-07-181-12/+34
| | | | | | | | | | | | | | | | | | | | | We also try to use instance variable to cache the result if RequestStore is not available, so we could keep the same logic, using the same cache key. Also introduce a way to specify method specific cache key
| * | Add RequestStoreWrap to cache via RequestStoreLin Jen-Shin2017-07-172-4/+70
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I don't like the idea of `RequestStore` at all, because it's just a global state which shouldn't be used at all. But we have a number of places calling `ProtectedBranch.protected?` and `ProtectedTag.protected?` in a loop for the same user, project, and ref whenever we're checking against if the jobs for a given pipeline is accessible for a given user. This means we're effectively making N queries for the same thing over and over. To properly fix this, we need to change how we check the permission, and that could be a huge work. To solve this quickly, adding a cache layer for the given request would be quite simple to do. We're already doing this in Commit#author, and this is extending that idea and make it generalized.
* | | Merge branch 'perf/policy-class-cache' into 'master'Sean McGivern2017-07-183-17/+51
|\ \ \ | |_|/ |/| | | | | | | | Perf: Improve performance of basic ruby ops in DeclarativePolicy See merge request !12922
| * | cache the cache key...perf/policy-class-cachehttp://jneen.net/2017-07-171-7/+8
| | |
| * | avoid #respond_to? in Cache.id_forhttp://jneen.net/2017-07-171-5/+8
| | |
| * | cache DeclarativePolicy.class_for at the class levelhttp://jneen.net/2017-07-171-5/+35
| | |
* | | Renamed Gitaly servicesAndrew Newdigate2017-07-187-22/+22
| | |
* | | Merge branch '34831-remove-coffee-rails-gem' into 'master'Jacob Schatz2017-07-171-1/+1
|\ \ \ | |/ / |/| | | | | | | | | | | | | | Remove coffee-rails gem Closes #34831 See merge request !12715
| * | Remove .coffee ext files for i18nTakuya Noguchi2017-07-081-1/+1
| | |
* | | Don't track cached queries in Gitlab::PerformanceBar::PeekQueryTracker34940-peek-performance-bar-doesn-t-count-cached-queries-but-shows-themRémy Coutable2017-07-171-1/+8
| |/ |/| | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Merge branch 'fix/gb/process-scheduled-background-migrations' into 'master'Yorick Peterse2017-07-171-7/+28
|\ \ | | | | | | | | | | | | | | | | | | Process scheduled background migrations as well Closes #34951 See merge request !12787
| * | Simplify background migrations stealing codeGrzegorz Bizon2017-07-171-15/+3
| | | | | | | | | | | | | | | Simply re-raise an exception when it occurs, but guarantee that no background migration is lost in the process.
| * | Fix off-by-one error in background migration retriesfix/gb/process-scheduled-background-migrationsGrzegorz Bizon2017-07-171-5/+10
| | |
| * | Recover from all exceptions when stealing bg migrationGrzegorz Bizon2017-07-171-4/+22
| | | | | | | | | | | | | | | It also makes it possible to gracefully retry a migration in order to avoid problems like deadlocks.
| * | Catch exceptions when stealing background migrationsGrzegorz Bizon2017-07-141-1/+6
| | |
| * | Avoid race condition when stealing a background migrationGrzegorz Bizon2017-07-141-3/+1
| | | | | | | | | | | | | | | | | | We first pop a job from the Sidekiq queue / scheduled set and only if this has been successfully deleted we process the job. This makes it possible to minimize a possibility of a race condition happening.
| * | Fix mocks in background migrations specsGrzegorz Bizon2017-07-131-1/+0
| | |
| * | Implement draining scheduled sets of background migrationsGrzegorz Bizon2017-07-131-7/+12
| | |
| * | Extract background migratons queue class methodGrzegorz Bizon2017-07-111-2/+5
| | |
* | | Expire cached user IDs that can see the performance after 5 minutesRémy Coutable2017-07-171-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | If we don't expire the cached user IDs, the list of IDs would become outdated when a new member is added, or when a member ios removed from the allowed group. Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | Merge branch 'remove-git-search-files' into 'master'Sean McGivern2017-07-171-89/+0
|\ \ \ | | | | | | | | | | | | | | | | Remove Repository#search_files See merge request !12879
| * | | Remove Repository#search_filesremove-git-search-filesJacob Vosmaer2017-07-141-89/+0
| | | |
* | | | Merge branch 'gitaly-more-annotations' into 'master'Sean McGivern2017-07-175-61/+65
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Branch and tag refactors for Gitaly See merge request !12872
| * | | | Gitaly migration annotationsJacob Vosmaer2017-07-141-0/+4
| | | | |
| * | | | Use helper instead of ad-hoc regexJacob Vosmaer2017-07-141-2/+1
| | | | |
| * | | | Make commit lookups explicitJacob Vosmaer2017-07-145-42/+45
| | | | |
| * | | | Remove deprecated #heads methodJacob Vosmaer2017-07-141-7/+0
| | | | |
| * | | | Make branch filter support privateJacob Vosmaer2017-07-141-11/+16
| |/ / /
* | | | Merge branch 'bjk/metric_names' into 'master'Douwe Maan2017-07-144-49/+44
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update metric names to match Prometheus guidelines. Closes #35031 See merge request !12812
| * | | | Rename ConnectionRackMiddleware to RequestsRackMiddleware.bjk/metric_namesPawel Chojnacki2017-07-132-40/+40
| | | | | | | | | | | | | | | | | | | | + fix tests after metrics rename
| * | | | Add unit to latency metrics.Ben Kochie2017-07-122-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add `seconds` unit to latency metrics based on uptream naming convention[0]. [0]: https://prometheus.io/docs/practices/naming/#metric-names
| * | | | Update rack metric namesBen Kochie2017-07-121-9/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Follow Prometheus naming conventions[0]. * Simplify metrics by adding response lables to the histogram. * Use standard `http_request_duration_seconds_...` names for the histogram. [0]: https://prometheus.io/docs/practices/naming/#metric-names
* | | | | Merge branch 'pass-before-script-as-is' into 'master'Grzegorz Bizon2017-07-142-1/+4
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | Pass before_script and script as-is preserving arrays See merge request !12680
| * | | | Pass before_script and script as-is preserving arrayspass-before-script-as-isKamil Trzcinski2017-07-072-1/+4
| | | | |
* | | | | Merge branch 'gitlab-git-repository-log' into 'master'Sean McGivern2017-07-144-79/+102
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Gitlab::Git shuffling and Gitaly annotations See merge request !12849
| * | | | | Indirectly migrate count_commits_between to GitalyJacob Vosmaer2017-07-131-1/+1
| | | | | |
| * | | | | More Gitaly annotationsJacob Vosmaer2017-07-133-0/+19
| | | | | |
| * | | | | Change Git::Repository#log to return CommitsJacob Vosmaer2017-07-132-78/+82
| | | | | |
* | | | | | Merge branch 'fixes-for-internal-auth-disabled' into 'master'Sean McGivern2017-07-145-7/+28
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes needed when GitLab sign-in is not enabled See merge request !12491
| * | | | | | Fixes needed when GitLab sign-in is not enabledRobin Bobbitt2017-07-135-7/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When sign-in is disabled: - skip password expiration checks - prevent password reset requests - don’t show Password tab in User Settings - don’t allow login with username/password for Git over HTTP requests - render 404 on requests to Profiles::PasswordsController
* | | | | | | Handle case when BroadcastMessage.current is nilStan Hu2017-07-131-1/+1
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Somehow Rails.cache.fetch occasionally returns `nil` values, which causes this endpoint to crash. Closes #35094
* | | | | | Merge branch 'lib-git-gitaly' into 'master'Sean McGivern2017-07-1322-96/+70
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add Gitaly notes to Gitlab::Git, delete diff options See merge request !12814
View Lorry Controller Status