summaryrefslogtreecommitdiff
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
* Update grape gemdz-update-grapeDmitriy Zaporozhets2017-07-2021-0/+28
| | | | | | | | New version of the gem returns 200 status code on delete with content instead of 204 so we explicitly set status code to keep existing behavior Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Merge branch 'feature/gb/auto-retry-failed-ci-job' into 'master'Kamil Trzciński2017-07-202-4/+9
|\ | | | | | | | | | | | | Make it possible to auto retry a failed CI/CD job Closes #3442 See merge request !12909
| * Do not allow to auto-retry a job more than 2 timesGrzegorz Bizon2017-07-191-1/+1
| |
| * Add a new `retry` CI/CD configuration keywordGrzegorz Bizon2017-07-172-4/+9
| |
* | Merge branch 'backstage/gb/build-stage-id-ref-bg-migration-cleanup' into ↵Yorick Peterse2017-07-201-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | 'master' Implement build stage_id reference migration clean up Closes #34893 See merge request !12785
| * \ Merge branch 'master' into backstage/gb/build-stage-id-ref-bg-migration-cleanupbackstage/gb/build-stage-id-ref-bg-migration-cleanupGrzegorz Bizon2017-07-1916-63/+247
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (59 commits) Resolve "Clarify k8s service keys" Add Portuguese Brazil translations of Commits Page & Pipeline Charts Add Japanese Translation to i18n Update Prometheus gem to version that explicitly calls `munmap` Simplify width for dropdown-menu on mobile Update CHANGELOG.md for 9.3.7 Remove developer documentation about not describing symbols Incorporate Gitaly's Commits#between RPC Adapt to new Gitaly commit message format Remove transitions on nav link hover Provide option to trigger build only for official CE and EE repos in .com Fix queries duration sorting in Performance Bar Rename Project nav items Add structured logging for Rails processes Disable Rails logging in CI test environments Fix download artifacts button alignment Update avatar border to be opaque for better stacking Fixed typos Fix typos Fix external issue trackers redirect ... Conflicts: db/schema.rb
| * | | Remove obsolete argument from bg migrations codeGrzegorz Bizon2017-07-181-1/+1
| | | |
* | | | Merge branch '24570-use-re2-for-user-supplied-regexp-9-3' into 'security-9-3'Douwe Maan2017-07-193-4/+61
| | | | | | | | | | | | | | | | | | | | | | | | 24570 use re2 for user supplied regexp 9 3 See merge request !2129
* | | | Merge branch '34563-usage-ping-github' into 'master'Sean McGivern2017-07-191-0/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add github imported projects count to usage data Closes #34563 See merge request !12959
| * | | | Add github imported projects count to usage data34563-usage-ping-githubJarka Kadlecova2017-07-191-0/+1
| | | | |
* | | | | Merge branch '35034-fix-unstranslated-visibility-levels' into 'master'Robert Speicher2017-07-191-2/+2
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Always return the translated level name. Closes #35034 See merge request !12852
| * | | | | Always return the translated level name.35034-fix-unstranslated-visibility-levelsRuben Davila2017-07-131-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are many places that expect the `.level_name` method to return the translated level name.
* | | | | | Merge branch 'htve/gitlab-ce-34880_add_ukrainian_translations_to_i18n'Rémy Coutable2017-07-191-0/+1
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | See merge request !12744. Signed-off-by: Rémy Coutable <remy@rymai.me>
| * | | | | add ukrainian translations to i18n黄涛2017-07-101-0/+1
| | | | | |
* | | | | | Merge branch '34881_add_russian_translations_to_i18n' into 'master'Rémy Coutable2017-07-191-0/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add Russian Translations To I18N Closes #34881 See merge request !12743
| * | | | | | add russian translations to i18n黄涛2017-07-101-0/+1
| |/ / / / /
* | | | | | Add Japanese Translation to i18nHuang Tao2017-07-191-1/+2
| | | | | |
* | | | | | Merge branch 'bvl-free-system-namespace' into 'master'Sean McGivern2017-07-192-1/+26
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Free `system` namespace by moving uploads into `uploads/-` Closes #33550 See merge request !12917
| * | | | | | Allow groups with the name systembvl-free-system-namespaceBob Van Landuyt2017-07-182-30/+8
| | | | | | |
| * | | | | | Add a background migration to rename `uploads` in the uploads tableBob Van Landuyt2017-07-181-0/+47
| | | | | | |
* | | | | | | Update Prometheus gem to version that explicitly calls `munmap`Paweł Chojnacki2017-07-191-3/+5
| | | | | | |
* | | | | | | Merge branch 'rc/bump-peek-performance_bar' into 'master'Sean McGivern2017-07-191-22/+0
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bump peek-performance_bar to 1.3.0 and get rid of a monkey-patch See merge request !12939
| * | | | | | | Bump peek-performance_bar to 1.3.0 and get rid of a monkey-patchrc/bump-peek-performance_barRémy Coutable2017-07-181-22/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | | | | | Merge branch 'gitaly-commits-between' into 'master'Rémy Coutable2017-07-194-13/+54
|\ \ \ \ \ \ \ \ | |_|_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Gitaly commits between Closes gitaly#350 See merge request !12669
| * | | | | | | Incorporate Gitaly's Commits#between RPCAlejandro Rodríguez2017-07-183-12/+33
| | | | | | | |
| * | | | | | | Adapt to new Gitaly commit message formatAlejandro Rodríguez2017-07-182-4/+24
| | |/ / / / / | |/| | | | |
* | | | | | | 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
| | | | | |
View Lorry Controller Status