- Nov 21, 2016
-
-
Z.J. van de Weg authored
Former-commit-id: 2d246acc
-
Z.J. van de Weg authored
The first take on this problem involved a worker requeueing 256 of these per 4 hours. Than it was suggested that this might not be needed, and post deployment migrations could be used. Given we monitor the number of projects which are pending deletion, we could give this strategy a try and take a look about a few hours later to see how this fares. Former-commit-id: f5febf60
-
Z.J. van de Weg authored
This reverts commit 5824a2b0e556631425ab709af2d79f58cd184395. Former-commit-id: 1266532f
-
Z.J. van de Weg authored
This reverts commit 614baa86ac6dbd82a2e89800052f177ef7c7f3c5. Former-commit-id: 7b8783b4
-
Z.J. van de Weg authored
Former-commit-id: c51834d2
-
Z.J. van de Weg authored
Also, this commit limits the amount of projects and groups we queue per call of the worker. At this time we have over 18k projects marked as pending_delete, so it wouldn't be wise to queue all these as removing a project for example is quite compute intensive for the disk and for PG. Former-commit-id: 1bb70e12
-
Zeger-Jan van de Weg authored
Former-commit-id: c297099b
-
Kamil Trzcińśki authored
Do not create TODO when build is allowed to fail ## What does this MR do? Do not create a TODO when build that failed is allowed to fail. ## Does this MR meet the acceptance criteria? - [x] [Changelog entry](https://docs.gitlab.com/ce/development/changelog.html) added - [x] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md) - Tests - [x] Added for this feature/bug - [ ] All builds are passing ## What are the relevant issue numbers? Closes #22280 See merge request !7618 Former-commit-id: 77a43827
-
Sean McGivern authored
Backport some changes done from Time Tracking feature in EE. See merge request !7604 Former-commit-id: 671c6d7d
-
Grzegorz Bizon authored
Former-commit-id: d840c462
-
Rémy Coutable authored
Grapify the users API Certain endpoints return different entities based on which user issues the request. Right now, I am not aware how to specify multiple entities as part of the description block of the endpoint. @rymai Do you know something about that. ## What are the relevant issue numbers? Related to #22928 See merge request !7147 Former-commit-id: 0be55733
-
Grzegorz Bizon authored
Former-commit-id: 0cb37c51
-
Grzegorz Bizon authored
Former-commit-id: f0d5ef96
-
Rémy Coutable authored
Remove bashism from scripts/prepare_build.sh Make `scripts/prepare_build.sh` an sh, rather than bash, script Write once, run everywhere. Even alpine. See merge request !7561 Former-commit-id: 5018bdcd
-
Rémy Coutable authored
Add constraints to group id in routing To prevent 404 when visit page like https://gitlab.com/groups/group.with.dot/activity Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/24622 See merge request !7614 Former-commit-id: 220c258c
-
Rémy Coutable authored
Add API endpoint for creating a pipeline Fixes #23468 See merge request !7209 Former-commit-id: c99522f2
-
Sean McGivern authored
Allow to block JIRA events for commits and merge requests implements #5541 See merge request !7469 Former-commit-id: b98193c5
-
Rémy Coutable authored
Remove unnecessary self from user model self keyword nod needed. Code is more clean and understandable. https://github.com/bbatsov/ruby-style-guide#no-self-unless-required See merge request !7551 Former-commit-id: 53714ddf
-
Rémy Coutable authored
Use build instead create in BroadcastMessage model spec See merge request !7592 Former-commit-id: 782a42ca
-
Dmitriy Zaporozhets authored
Signed-off-by:
Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> Former-commit-id: beb638ca
-
Robert Schilling authored
Former-commit-id: 4cb3c0b4
-
Rémy Coutable authored
Fixes regression causing an error message to appear when user first enters the Merge Request form. Closes #24627 See merge request !7599 Former-commit-id: 900999f8
-
- Nov 19, 2016
-
-
Fatih Acet authored
Revert signin tab order fix. The history: > 1. @sytses created this: https://gitlab.com/gitlab-org/gitlab-ce/issues/23279 > 2. The fix was submitted here: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6867 > 3. I [made](https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6867#note_16993324) the same observation you did after it was merged 4. A long discussion about accessibility and UX followed, and we decided to revert all changes (https://gitlab.com/gitlab-org/gitlab-ce/issues/23698). ## Screenshots (if relevant)  ## Does this MR meet the acceptance criteria? - [x] [Changelog entry](https://docs.gitlab.com/ce/development/changelog.html) added - [x] All builds are passing - [x] Conform by the [merge request performance guides](http://docs.gitlab.com/ce/development/merge_request_performance_guidelines.html) - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if it does - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) ## What are the relevant issue numbers? Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/23698 See merge request !7538 Former-commit-id: 9aded5c8
-
Kamil Trzcińśki authored
Remove duplicate sidekiq throttling parameters Improves `db/schema` after merging https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7015. cc @grzesiek @filipa See merge request !7603 Former-commit-id: e15ea8b8
-
Kamil Trzcińśki authored
Former-commit-id: b52dd684
-
Achilleas Pipinellis authored
Fix a typo in gitlab_flow.md ('munch'->'much') See merge request !7595 Former-commit-id: cb744959
-
Achilleas Pipinellis authored
Fix wrong link Not a big one, just fix a wrong link to internal page. See merge request !7597 Former-commit-id: ecef3da4
-
Jacob Schatz authored
Fix broken merge request widget spec ## What does this MR do? Converts the invalid es6 syntax of the merge_request_widget_spec which is actually causing the spec not to run through teaspoon ## Are there points in the code the reviewer needs to double check? None ## Why was this MR needed? Get specs working again ## Screenshots (if relevant) None ## Does this MR meet the acceptance criteria? - [ ] [Changelog entry](https://docs.gitlab.com/ce/development/changelog.html) added - [ ] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md) - [ ] API support added - Tests - [ ] Added for this feature/bug - [ ] All builds are passing - [ ] Conform by the [merge request performance guides](http://docs.gitlab.com/ce/development/merge_request_performance_guidelines.html) - [ ] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [ ] Branch has no merge conflicts with `master` (if it does - rebase it please) - [ ] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) ## What are the relevant issue numbers? Closes #24673 See merge request !7594 Former-commit-id: 18a646c3
-
Fatih Acet authored
Resolve "Display "folders" for environments" ## What does this MR do? Adds the ability to show the grouped environments inside "folders". Adds several reusable vue components in order to accomplish the recursive tree data structure presented. For the individual components, Jasmine tests were added. For the ones that depend of an API response, rspec tests are used. ## Screenshots (if relevant)     ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG.md) entry added - [ ] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md) - [ ] API support added - Tests - [x] Added for this feature/bug - [ ] All builds are passing - [x] Conform by the [merge request performance guides](http://docs.gitlab.com/ce/development/merge_request_performance_guidelines.html) - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if it does - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) ## What are the relevant issue numbers? Closes #22539 See merge request !7015 Former-commit-id: b9c2fb88
-
Alex Sanford authored
Merge request !7163 caused an error message to appear on the Merge Request form when it was first loaded by the user. This commit fixes that regression bug. Former-commit-id: 16a8d644
-
Filipa Lacerda authored
Former-commit-id: 531d4e56
-
Filipa Lacerda authored
Former-commit-id: 6e5a1ea0
-
Filipa Lacerda authored
Former-commit-id: f9a662c3
-
Kamil Trzcińśki authored
Make job script a required configuration entry ## What does this MR do? This MR makes a job script a required configuration entry. ## Does this MR meet the acceptance criteria? - [x] [Changelog entry](https://docs.gitlab.com/ce/development/changelog.html) added - Tests - [x] Added for this feature/bug - [x] All builds are passing ## What are the relevant issue numbers? Closes #24575 See merge request !7566 Former-commit-id: cf0283c8
-
Kamil Trzcińśki authored
Fix pipeline author for Slack and use pipeline id for pipeline link [Context](https://gitlab.slack.com/archives/runner-builds/p1479238493000018) Pipeline Slack message: > gitlab-org/gitlab-ci-multi-runner: Pipeline 8bed2a3b of fix/handle-failed-state-while-patching-trace branch by failed in 1389 seconds Quotes: > @godfat Take a look at these notifications, they seems to be missing author and should probably use Pipeline ID instead of SHA See merge request !7506 Former-commit-id: 7ef7135a
-
Kamil Trzcińśki authored
Fix faulty deployment refs In the 8.13 RC cycle, so before the release, there was a time in which references in git where stored by id instead of iid. This could be fixed by time, if the iid catches up with the id, it overwrites it. But in the mean time we have wrong refs in the folder. This commit fixes that. For all projects we have deployments we'll find the ones where the ref has a higher number than the iid is now and calls `#create_ref` on the corresponding deployment. Fixes gitlab-com/infrastructure#683 cc @ayufan One worry: I tested this locally, couldn't find a way to do automate this. See merge request !7352 Former-commit-id: e4c86bc8
-
Douwe Maan authored
Add logging for rack attack events Patch for issue: https://gitlab.com/gitlab-org/gitlab-ce/issues/15612 and partial patch for https://gitlab.com/gitlab-com/support/issues/119. Also related: https://gitlab.com/gitlab-org/gitlab-ce/issues/22527 Adds logging for throttle and blacklist rack attack events to the production log. @stanhu @MrChrisW See merge request !7585 Former-commit-id: 8d7cb865
-
Clement Ho authored
Former-commit-id: 34eec88c
-