summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
* Whitelist authorized_keys.lock in the gitlab:check rake taskNick Thomas2017-10-021-1/+6
|
* Merge branch 'gitaly-add-branch' into 'master'Sean McGivern2017-10-027-53/+181
|\ | | | | | | | | | | | | Implement OperationService.UserAddBranch Gitaly RPC Closes gitaly#544 See merge request gitlab-org/gitlab-ce!14522
| * Implement OperationService.UserAddBranch Gitaly RPCgitaly-add-branchAlejandro Rodríguez2017-09-302-25/+105
| |
| * Populate `Gitlay::Repository`'s `gl_repository` fieldAlejandro Rodríguez2017-09-302-1/+45
| |
| * Add a factory for `Gitaly::GitCommit`sAlejandro Rodríguez2017-09-303-27/+31
| |
* | Merge branch '38571-fix-exception-in-raven-report' into 'master'Sean McGivern2017-10-021-0/+21
|\ \ | | | | | | | | | | | | | | | | | | Ensure no exception is raised when Raven tries to get the current user in API context Closes #38571 See merge request gitlab-org/gitlab-ce!14580
| * | Ensure no exception is raised when Raven tries to get the current user in ↵38571-fix-exception-in-raven-reportRémy Coutable2017-09-291-0/+21
| | | | | | | | | | | | | | | | | | API context Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | Merge branch ↵Tim Zallmann2017-10-024-45/+135
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '34366-issue-sidebar-don-t-render-participants-in-collapsed-state' into 'master' Resolve "Issue Sidebar : Don't render participants in collapsed state" Closes #34366 See merge request gitlab-org/gitlab-ce!14270
| * | | use LazyImageTagHelper.placeholder_image for blank src base64 image34366-issue-sidebar-don-t-render-participants-in-collapsed-stateLuke "Jared" Bennett2017-10-011-1/+1
| | | |
| * | | Merge remote-tracking branch 'origin/master' into ↵Luke "Jared" Bennett2017-09-2834-206/+917
| |\ \ \ | | | | | | | | | | | | | | | 34366-issue-sidebar-don-t-render-participants-in-collapsed-state
| * | | | Only trigger loadCheck on open toggle, not close. Update specs and add ↵Luke "Jared" Bennett2017-09-282-43/+79
| | | | | | | | | | | | | | | | | | | | right_sidebar_spec test for additions
| * | | | issuable_context_spec review changesLuke "Jared" Bennett2017-09-281-5/+5
| | | | |
| * | | | Finish issuable_context_specLuke "Jared" Bennett2017-09-281-10/+7
| | | | |
| * | | | Add issauble_context_specLuke "Jared" Bennett2017-09-281-0/+31
| | | | |
| * | | | Finish _participants view specLuke "Jared" Bennett2017-09-281-22/+7
| | | | |
| * | | | Update projects_helper_specLuke "Jared" Bennett2017-09-281-2/+2
| | | | |
| * | | | Add participants specLuke "Jared" Bennett2017-09-281-0/+41
| | | | |
* | | | | Merge branch '38052-use-simple-api-for-projects' into 'master'Tim Zallmann2017-10-021-1/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use `simple=true` for projects API in Projects dropdown for better search performance Closes #38052 See merge request gitlab-org/gitlab-ce!14585
| * | | | | Use `simple=true` for projects API for better performancekushalpandya2017-09-291-1/+1
| | | | | |
* | | | | | Merge branch 'remote_user' into 'master'Douwe Maan2017-10-025-17/+31
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add GL_USERNAME environment variable for hooks See merge request gitlab-org/gitlab-ce!13264
| * | | | | | Add username as GL_USERNAME in hooks (http)David Turner2017-09-295-17/+31
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When calling pre-receive, post-receive, and update hooks, add the GitLab username as the GL_USERNAME environment variable. This patch only handles cases where pushes are over http, or via the web interface. Later, we will address the ssh case.
* | | | | | Fix pushes to an empty repository not invalidating has_visible_content? cacheStan Hu2017-10-011-1/+12
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `Repository#has_visible_content?` used to rely on the cached count of local branches, but since it is now an independently cached value it needs to be invalidated on its own. Closes #38646
* | | | | Create repositories via GitalyJacob Vosmaer2017-09-297-35/+50
| |_|_|/ |/| | |
* | | | Merge branch 'fork-btn-enabled-user-groups' into 'master'Filipa Lacerda2017-09-293-0/+100
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix fork button being disabled for users who can fork to group Closes #38462 See merge request gitlab-org/gitlab-ce!14551
| * | | | moved fork checks into policiesfork-btn-enabled-user-groupsPhil Hughes2017-09-292-0/+43
| | | | |
| * | | | Fix fork button being disabled for users who can fork to groupPhil Hughes2017-09-291-0/+57
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously the fork button was disabled for all users if they have exceeded their project limit. This fixes that by changing the check to see if the user can fork to a group instead of their own namespace. This behaviour is already possible by visiting the new fork page directly, so this just fixes the button being disabled. Closes #38462
* | | | | Merge branch 'hash-mr-scroll-load' into 'master'Filipa Lacerda2017-09-291-0/+56
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | Fixed anchored content not being scrolled into view Closes #38326 See merge request gitlab-org/gitlab-ce!14562
| * | | | spec fixhash-mr-scroll-loadPhil Hughes2017-09-291-7/+3
| | | | |
| * | | | Fixed anchored content not being scrolled into viewPhil Hughes2017-09-291-0/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Correctly scrolls anchored content into view when the user loads the page. This is most obvious when the user loads a link note in a merge request & the page does not scroll down to the note. Closes #38326
* | | | | Merge branch ↵Rémy Coutable2017-09-292-20/+161
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '36631-activerecord-statementinvalid-pg-querycanceled-error-canceling-statement-due-to-statement-timeout' into 'master' Insert at most 1,000 rows at once in MR diff background migration Closes #36631 et #37505 See merge request gitlab-org/gitlab-ce!13661
| * | | | | Reschedule merge request diff background migration36631-activerecord-statementinvalid-pg-querycanceled-error-canceling-statement-due-to-statement-timeoutSean McGivern2017-09-291-0/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The first attempt didn't migrate all rows on GitLab.com, due to a couple of issues: 1. Some rows in merge_request_diffs had truly huge numbers of commits and diffs serialised - one in particular had 26,000 commits! 2. The jobs were sometimes on Sidekiq hosts with frequent OOM errors, leading to the job being lost. The previous commit adds more logging, and a more robust insertion method. This commit reschedules the jobs, with a generous pause between each.
| * | | | | Make MR diff background migration less likely to time outSean McGivern2017-09-291-22/+76
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This version does not use transactions, but individual statements. As we have unique constraints on the target tables for the inserts, we can just ignore uniqueness violations there (as long as we always insert the same batch size, in the same order). This means the spec now must use truncation, not a transaction, as the uniqueness violation means that the whole transaction for that spec would be invalid, which isn't what we'd want. In real-world use, this isn't run in a transaction anyway. This commit also wraps unhandled exceptions, for easier finding in Sentry, and logs with a consistent format, for easier searching.
| * | | | | Insert at most 1,000 rows at once in MR diff background migrationSean McGivern2017-09-291-2/+30
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | We were hitting the statement timeout for very large MR diffs. Now we insert at most 1,000 rows to `merge_request_diff_commits` in a single statement, or 100 rows to `merge_request_diff_files`.
* | | | | Merge branch 'dm-json-page-title' into 'master'Jacob Schatz2017-09-291-0/+12
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use backend-provided page title in repo editor Closes #36029 See merge request gitlab-org/gitlab-ce!13763
| * | | | | Title has now its own testdm-json-page-titleTim Zallmann2017-09-291-4/+11
| | | | | |
| * | | | | Not the Test will also work with other tests beforeTim Zallmann2017-09-291-3/+3
| | | | | |
| * | | | | Added Test for PageTitle / Removed Gitlab Part from TItleTim Zallmann2017-09-291-0/+5
| | | | | |
* | | | | | Make Repository#has_visible_content more efficientJacob Vosmaer (GitLab)2017-09-293-14/+56
| | | | | |
* | | | | | Fix notes type created from importLin Jen-Shin (godfat)2017-09-294-0/+62
| |/ / / / |/| | | |
* | | | | Merge branch 'breadcrumb-item-links' into 'master'Tim Zallmann2017-09-293-3/+11
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixed breadcrumb title not linking Closes #38059 See merge request gitlab-org/gitlab-ce!14532
| * | | | | spec fixesbreadcrumb-item-linksPhil Hughes2017-09-282-3/+3
| | | | | |
| * | | | | Fixed breadcrumb title not linkingPhil Hughes2017-09-271-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes the breadcrumb titles not having links. This was a regression in the updated breadcrumbs introduced in 10.0. This introduces the link for all pages rather than selectively doing it. This will cover issues, merge requests, pipelines, jobs etc. Closes #38059
* | | | | | Merge branch 'remove-target-dd' into 'master'Filipa Lacerda2017-09-291-2/+0
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove target dd Closes #38485 See merge request gitlab-org/gitlab-ce!14558
| * | | | | Fix karma testJacob Schatz2017-09-281-2/+0
| | | | | |
* | | | | | Merge branch '38528-build-url' into 'master'Phil Hughes2017-09-291-0/+14
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes data parameter not being sent in ajax request for jobs log Closes #38528 See merge request gitlab-org/gitlab-ce!14564
| * | | | | | Fixes data parameter not being sent in ajax request for jobs log38528-build-urlFilipa Lacerda2017-09-281-0/+14
| | |_|_|_|/ | |/| | | |
* | | | | | Migrate Gitlab::Git::Repository#add_tag to Gitalyfeature/migrate-repository-add-tag-to-gitalyAhmad Sherif2017-09-292-22/+30
| | | | | | | | | | | | | | | | | | | | | | | | Closes gitaly#601
* | | | | | Merge branch 'feature/migrate-repository-rm-tag-to-gitaly' into 'master'Robert Speicher2017-09-283-11/+37
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Migrate Git::Repository#rm_tag to Gitaly Closes gitaly#562 See merge request gitlab-org/gitlab-ce!14388
| * | | | | | Migrate Git::Repository#rm_tag to GitalyAhmad Sherif2017-09-283-11/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes gitaly#562
* | | | | | | Merge branch 'acet-repo-editor-prevent-extra-request' into 'master'Jacob Schatz2017-09-281-0/+14
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | RepoEditor: Prevent extra network call for already opened files. Closes #38256 See merge request gitlab-org/gitlab-ce!14489
View Lorry Controller Status