summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* added specide-commit-state-bugPhil Hughes2018-05-221-0/+19
|
* Fixed web IDE review & commit editors getting into incorrect statePhil Hughes2018-05-181-2/+7
| | | | Closes #46392
* Merge branch 'remove-css-imports' into 'master'Phil Hughes2018-05-172-4/+0
|\ | | | | | | | | Remove unneccessary imports See merge request gitlab-org/gitlab-ce!18993
| * Remove unneccessary importsremove-css-importsAnnabel Dunstone Gray2018-05-162-4/+0
| |
* | Merge branch 'jivl-add-dot-system-notes' into 'master'Phil Hughes2018-05-175-5/+18
|\ \ | | | | | | | | | | | | | | | | | | Add dot to separate system notes content Closes #45676 See merge request gitlab-org/gitlab-ce!18864
| * | Add dot to separate system notes contentjivl-add-dot-system-notesJose2018-05-145-5/+18
| | |
* | | Merge branch 'jprovazn-remote-upload-destroy' into 'master'Kamil Trzciński2018-05-1613-11/+109
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Delete remote uploads Closes #45425 See merge request gitlab-org/gitlab-ce!18698
| * | | Whitelisted query limits for group destroy APIjprovazn-remote-upload-destroyJan Provaznik2018-05-162-0/+2
| | | |
| * | | Fixed typoJan Provaznik2018-05-161-1/+1
| | | |
| * | | Changed order of includeJan Provaznik2018-05-164-4/+4
| | | |
| * | | Use find_in_batches instead of destroy_allJan Provaznik2018-05-161-1/+3
| | | | | | | | | | | | | | | | destroy_all loads all records at once
| * | | Delete remote uploadsJan Provaznik2018-05-1611-11/+105
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ObjectStore uploader requires presence of associated `uploads` record when deleting the upload file (through the carrierwave's after_commit hook) because we keep info whether file is LOCAL or REMOTE in `upload` object. For this reason we can not destroy uploads as "dependent: :destroy" hook because these would be deleted too soon. Instead we rely on carrierwave's hook to destroy `uploads` in after_commit hook. But in before_destroy hook we still have to delete not-mounted uploads (which don't use carrierwave's destroy hook). This has to be done in before_Destroy instead of after_commit because `FileUpload` requires existence of model's object on destroy action. This is not ideal state of things, in a next step we should investigate how to unify model dependencies so we can use same workflow for all uploads. Related to #45425
* | | | Merge branch '46010-allow-managing-group-runners-via-api' into 'master'Kamil Trzciński2018-05-1610-97/+120
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | | | | | | | | | API support + Improved policies for group runners Closes #45894 and #38979 See merge request gitlab-org/gitlab-ce!18851
| * | | Refactor out duplication in runner_policy.rb46010-allow-managing-group-runners-via-apiDylan Griffith2018-05-161-4/+8
| | | |
| * | | Remove unnecessary runner.is_shared? checks in api because they are handled ↵Dylan Griffith2018-05-161-2/+0
| | | | | | | | | | | | | | | | by policy
| * | | Allow admin to assign shared runner to project through APIDylan Griffith2018-05-162-2/+11
| | | |
| * | | Change policy list_runner_jobs -> read_runnerDylan Griffith2018-05-162-2/+1
| | | |
| * | | Rename User#ci_authorized_runners -> ci_owned_runnersDylan Griffith2018-05-166-18/+18
| | | |
| * | | Improve efficiency of authorized_runner policy queryDylan Griffith2018-05-161-1/+1
| | | |
| * | | Use can? policies for lib/api/runners.rbDylan Griffith2018-05-162-13/+9
| | | |
| * | | Allow group runners to be viewed/edited in APIDylan Griffith2018-05-167-70/+87
| | | |
* | | | Merge branch 'fix/contributors-graph-width' into 'master'Mike Greiling2018-05-162-18/+41
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix contributors graph width Closes #22647 See merge request gitlab-org/gitlab-ce!18639
| * | | | Use correct base widthPaul Vorbach2018-05-151-8/+22
| | | | |
| * | | | Refactor duplicate codePaul Vorbach2018-05-151-6/+10
| | | | |
| * | | | Add changelog entry for contrib graphs width fixPaul Vorbach2018-05-151-0/+5
| | | | |
| * | | | Use ES6 `const` more consistentlyPaul Vorbach2018-05-151-9/+6
| | | | |
| * | | | Fix width of contributors graphsPaul Vorbach2018-05-151-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The contributors graphs were sized with the full .content width in mind, but at some point GitLab changed to a more narrow design, so graphs would be too wide for their container divs. This change resizes contributors graphs relative to their correct container widths.
* | | | | Merge branch 'sh-fast-admin-counts' into 'master'Yorick Peterse2018-05-165-10/+127
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix Error 500 viewing admin page due to statement timeouts Closes #46255 See merge request gitlab-org/gitlab-ce!18982
| * | | | | Fix Error 500 viewing admin page due to statement timeoutsStan Hu2018-05-165-10/+127
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Uses PostgreSQL tuple estimates to provide a much faster yet approximate count. See https://wiki.postgresql.org/wiki/Slow_Counting for more details. We only use this fast method if the table has been analyzed or vacuumed within the last hour. Closes #46255
* | | | | | Merge branch '46303_copy_button_fix_embedded_snippets' into 'master'Clement Ho2018-05-163-2/+8
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixed copy to cliboard button in embedded snippets Closes #46303 See merge request gitlab-org/gitlab-ce!18923
| * | | | | | fixed copy to cliboard button in embedded snippetshaseeb2018-05-163-2/+8
|/ / / / / /
* | | | | | Merge branch 'osw-fix-boards-list-header-color' into 'master'Robert Speicher2018-05-163-2/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adjust board lists header text color Closes #36983 See merge request gitlab-org/gitlab-ce!18955
| * | | | | | Adjust board lists header text colorosw-fix-boards-list-header-colorOswaldo Ferreira2018-05-143-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Adjusts regression fixed by https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/18786/. We were not returning the label text color correctly on the backend.
* | | | | | | Merge branch 'dz-fix-cluster-error-message' into 'master'Annabel Gray2018-05-161-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove table-section from div holding cluster error messages See merge request gitlab-org/gitlab-ce!18971
| * | | | | | | Remove unnecessary section-100 css class from application_row.vueDmitriy Zaporozhets2018-05-161-1/+1
| | | | | | | |
| * | | | | | | Remote table-section from div holding cluster error messagesDmitriy Zaporozhets2018-05-151-1/+1
| | | | | | | |
* | | | | | | | Merge branch 'update-license' into 'master'Rémy Coutable2018-05-162-6/+3
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | edit GitLab license info in regards to CC licensing See merge request gitlab-org/gitlab-ce!18981
| * | | | | | | | edit GitLab license info in regards to CC licensingAndrea Kao2018-05-152-6/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | GitHub uses a library called Licensee to identify a project's license type. It shows this information in the status bar and via the API if it can unambiguously identify the license. This commit moves the reference to Creative Commons licensing from the LICENSE file to the README, which allows Licensee to successfully identify the license type of GitLab CE as MIT. Signed-off-by: Andrea Kao <eirinikos@gmail.com>
* | | | | | | | | Merge branch 'move-disussion-actions-to-the-right' into 'master'Tim Zallmann2018-05-165-30/+26
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move discussion and notes actions to the right Closes #33645 See merge request gitlab-org/gitlab-ce!18476
| * | | | | | | | | Reorder discussion timestamp information for issuesGeorge Tsiolis2018-05-043-7/+7
| | | | | | | | | |
| * | | | | | | | | Move note actions to the rightGeorge Tsiolis2018-05-011-4/+0
| | | | | | | | | |
| * | | | | | | | | Move discussion actions to the right for small viewportsGeorge Tsiolis2018-05-013-20/+20
| | | | | | | | | |
* | | | | | | | | | Merge branch 'fix-metrics-doorkeeper' into 'master'Nick Thomas2018-05-162-4/+8
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix GitLab Performance Monitoring content types for Doorkeeper Closes #46412 See merge request gitlab-org/gitlab-ce!18984
| * | | | | | | | | | Fix GPM content types for DoorkeeperYorick Peterse2018-05-162-4/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Certain controllers (e.g. Doorkeeper::TokensController) don't expose the method "request_format". This commit changes Gitlab::Metrics::WebTransaction so we don't rely on this method, instead using the underlying code this method uses. Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/46412
* | | | | | | | | | | Merge branch '43673-operations-tab-mvc' into 'master'Filipa Lacerda2018-05-165-72/+229
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Project Sidebar: Split CI/CD into CI/CD and Operations Closes #43673 See merge request gitlab-org/gitlab-ce!18941
| * | | | | | | | | | | Project Sidebar: Split CI/CD into CI/CD and OperationsLukas Eipert2018-05-165-72/+229
|/ / / / / / / / / / /
* | | | | | | | | | | Merge branch 'zj-add-branch-mandatory' into 'master'Douwe Maan2018-05-163-77/+22
|\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move Gitlab::Git::Repository#add_branch to mandatory Closes gitaly#540 See merge request gitlab-org/gitlab-ce!18939
| * | | | | | | | | | Move Gitlab::Git::Repository#add_branch to mandatoryZeger-Jan van de Weg2018-05-143-77/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prior to this change, a feature flag could be used to disable this feature. Now all requests go through Gitaly's OperationService. Closes https://gitlab.com/gitlab-org/gitaly/issues/540 When vendoring `Gitlab::Git` again in Gitaly, this implemenation will be gone, but this is readded there through: https://gitlab.com/gitlab-org/gitaly/merge_requests/717
* | | | | | | | | | | Merge branch 'registry-ux-improvements-remove-clipboard-prefix' into 'master'Filipa Lacerda2018-05-163-11/+7
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove docker pull prefix from registry clipboard feature Closes #46314 and #45968 See merge request gitlab-org/gitlab-ce!18933
| * | | | | | | | | | Remove docker pull prefix from registry clipboard featureLars Greiss2018-05-163-11/+7
|/ / / / / / / / / /