summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Make Admin -> License selector to fix GitLab QA mastersh-fix-gitlab-qa-adminStan Hu2017-09-291-9/+4
| | | | Closes gitlab-qa#68
* Merge branch 'jramsay-38624-right-align' into 'master'Jacob Schatz2017-09-292-2/+2
|\ | | | | | | | | | | | | Fix incorrectly aligned last updated time Closes #38624 See merge request gitlab-org/gitlab-ce!14592
| * Fix incorrectly aligned last updated timeJames Ramsay2017-09-292-2/+2
|/
* Merge branch 'gitaly-create-repository' into 'master'Douwe Maan2017-09-2915-48/+77
|\ | | | | | | | | | | | | Create repositories via Gitaly Closes gitaly#595 See merge request gitlab-org/gitlab-ce!14566
| * Create repositories via GitalyJacob Vosmaer2017-09-2915-48/+77
| |
* | Merge branch 'lint-changelog-yaml' into 'master'Robert Speicher2017-09-2914-34/+33
|\ \ | |/ |/| | | | | Add static analysis job to find invalid YAML in changelogs See merge request gitlab-org/gitlab-ce!14518
| * Remove changelogs with invalid extensionsSean McGivern2017-09-298-31/+0
| | | | | | | | | | All of these were in past releases, so missed their chance to be included in the main changelog :-(
| * Extend changelog checker to test file extensionsSean McGivern2017-09-291-2/+5
| | | | | | | | | | Changelogs without a .yml extension won't be picked up, and will be ignored completely, so fail the pipeline when one of those is found.
| * Fix invalid changelog entrieslint-changelog-yamlSean McGivern2017-09-272-2/+2
| |
| * Manually add CHANGELOG entry for !13325Sean McGivern2017-09-271-0/+2
| | | | | | | | This was discarded from the initial compilation because it had invalid YAML.
| * Add static analysis job to find invalid YAML in changelogsSean McGivern2017-09-273-1/+26
| | | | | | | | | | | | | | When a changelog has invalid YAML (typically, there is an unquoted @ at the start of the author field), then the entry will be discarded. This script checks all unreleased changelogs for validity, and runs as part of the static-analysis step, so the pipeline will fail if this happens in future.
* | Merge branch 'fork-btn-enabled-user-groups' into 'master'Filipa Lacerda2017-09-2912-24/+146
|\ \ | | | | | | | | | | | | | | | | | | 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-296-9/+61
| | |
| * | Fix fork button being disabled for users who can fork to groupPhil Hughes2017-09-298-24/+94
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 '38582-popover-badge' into 'master'Phil Hughes2017-09-292-2/+7
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Improves UX of autodevops popover to match gpg one Closes #38582 See merge request gitlab-org/gitlab-ce!14582
| * | | Improves UX of autodevops popover to match gpg oneFilipa Lacerda2017-09-292-2/+7
| |/ /
* | | Merge branch 'hash-mr-scroll-load' into 'master'Filipa Lacerda2017-09-293-16/+72
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 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
| | | |
| * | | CHANGELOG itemPhil Hughes2017-09-291-0/+5
| | | |
| * | | Fixed anchored content not being scrolled into viewPhil Hughes2017-09-292-16/+71
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 'jramsay-4012-externalize-wiki-strings' into 'master'38609-generalise-pop-overs-to-work-on-hover-and-implement-in-existing-placesDouwe Maan2017-09-2912-64/+186
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Add translatable strings to wiki pages See merge request gitlab-org/gitlab-ce!14534
| * | | | Add translatable strings to wiki pagesjramsay-4012-externalize-wiki-stringsJames Ramsay2017-09-2812-64/+186
| | | | |
* | | | | Merge branch ↵Rémy Coutable2017-09-295-25/+226
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '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-293-0/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-292-31/+101
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-292-5/+39
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | 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 'icons-update-nav-bars' into 'master'Phil Hughes2017-09-2921-3366/+67
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Updated All Navigation Icons to Sprite_Icons See merge request gitlab-org/gitlab-ce!14559
| * | | | | FIxed the clearing icon + jumping of fluid nav baricons-update-nav-barsTim Zallmann2017-09-292-1/+2
| | | | | |
| * | | | | Fixed another linting error this time HAMLTim Zallmann2017-09-281-1/+1
| | | | | |
| * | | | | Fixed CSS Lint ErrorTim Zallmann2017-09-281-1/+1
| | | | | |
| * | | | | Updated All Navigation Icons to Sprite_IconsTim Zallmann2017-09-2820-3365/+65
| | | | | |
* | | | | | Merge branch 'dm-json-page-title' into 'master'Jacob Schatz2017-09-297-6/+31
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-292-4/+4
| | | | | | |
| * | | | | | Added Test for PageTitle / Removed Gitlab Part from TItleTim Zallmann2017-09-292-1/+6
| | | | | | |
| * | | | | | Encode Page-Title header as ISO-8859-1Douwe Maan2017-09-261-1/+2
| | | | | | |
| * | | | | | Added Page Title setting for files + directoriesTim Zallmann2017-09-262-4/+6
| | | | | | |
| * | | | | | Add Page-Title header to tree and blob JSON endpointsDouwe Maan2017-09-264-1/+11
| | | | | | |
* | | | | | | Merge branch 'accepting-merge-requests-ux' into 'master'Victor Wu2017-09-291-4/+10
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tell community members to ping PM when working on an issue See merge request gitlab-org/gitlab-ce!14571
| * | | | | | | Update CONTRIBUTING.mdaccepting-merge-requests-uxVictor Wu2017-09-281-4/+10
| | | | | | | |
* | | | | | | | Merge branch 'make-has-visible-commit-more-efficient' into 'master'Rémy Coutable2017-09-296-22/+91
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make Repository#has_visible_content more efficient See merge request gitlab-org/gitlab-ce!14554
| * | | | | | | | Make Repository#has_visible_content more efficientJacob Vosmaer (GitLab)2017-09-296-22/+91
|/ / / / / / / /
* | | | | | | | Merge branch '38432-fix-notes-type-for-import' into 'master'Rémy Coutable2017-09-2912-37/+155
|\ \ \ \ \ \ \ \ | |_|_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix notes type created from import Closes #38432 See merge request gitlab-org/gitlab-ce!14524
| * | | | | | | Fix notes type created from importLin Jen-Shin (godfat)2017-09-2912-37/+155
|/ / / / / / /
* | | | | | | Merge branch 'add-composite-index-on-merge-requests-merge-commit-sha' into ↵Sean McGivern2017-09-293-1/+40
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'master' Add index for merge_requests.merge_commit_sha Closes #38507 See merge request gitlab-org/gitlab-ce!14549
| * | | | | | | Add index for merge_requests.merge_commit_shaadd-composite-index-on-merge-requests-merge-commit-shaYorick Peterse2017-09-283-1/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a composite index on (target_project_id, merge_commit_sha, id) that allows queries such as the following to use a full backwards index scan: SELECT "merge_requests".* FROM "merge_requests" WHERE "merge_requests"."deleted_at" IS NULL AND "merge_requests"."target_project_id" = 13083 AND "merge_requests"."merge_commit_sha" = 'e80a893ff0ea8466099f6478183631af55933db2' ORDER BY "merge_requests"."id" DESC LIMIT 1; Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/38507
* | | | | | | | Merge branch 'first-time-contributor-tooltip-text' into 'master'Filipa Lacerda2017-09-292-2/+2
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tweaked the first time contributor tooltip text See merge request gitlab-org/gitlab-ce!14565
| * | | | | | | | Tweaked the first time contributor tooltip textfirst-time-contributor-tooltip-textPhil Hughes2017-09-282-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removes the text `Handle with care` as it sounds condescending.
* | | | | | | | | Merge branch 'docs-fix-auto-devops-banner-feature-flag' into 'master'Grzegorz Bizon2017-09-291-1/+1
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix Auto DevOps banner feature flag Closes #38576 See merge request gitlab-org/gitlab-ce!14576
| * | | | | | | | | Fix Auto DevOps banner feature flagFabio Busatto2017-09-291-1/+1
| | | | | | | | | |