summaryrefslogtreecommitdiff
path: root/app/models
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch ↵Robert Speicher2019-07-291-1/+1
|\ | | | | | | | | | | | | | | | | '63547-add-system-notes-for-when-a-zoom-call-was-added-removed-from-an-issue' into 'master' Resolve "Add system notes for when a zoom call was added/removed from an issue" Closes #63547 See merge request gitlab-org/gitlab-ce!30857
| * Add system notes for when a zoom call was added/removed from an issue63547-add-system-notes-for-when-a-zoom-call-was-added-removed-from-an-issueJacopo2019-07-291-1/+1
| | | | | | | | | | Add a zoom link added / removed system note when a zoom link is being added / removed to the issue description.
* | Make quick action "commands applied" banner more useful🙈 jacopo beschi 🙉2019-07-291-0/+4
| | | | | | | | | | Extends the quick actions "commands applied" banner to show the quick action preview text, but with everything in past tense.
* | Resolve "MembersFinder contains slow database query with OR conditions"🙈 jacopo beschi 🙉2019-07-291-0/+2
| |
* | Further remove code branches by database typeAndreas Brandl2019-07-293-36/+6
| | | | | | | | | | | | | | | | We dropped MySQL support and a lot of mysql specific code has been removed in https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/29608. This comes in from the other direction and removes any `if postgresql?` branches.
* | Add Job specific variablesMatija Čupić2019-07-293-2/+33
| | | | | | | | | | Adds Job specific variables to facilitate specifying variables when running manual jobs.
* | Merge branch '32495-improve-slack-notification-on-pipeline-status' into 'master'Robert Speicher2019-07-261-14/+211
|\ \ | | | | | | | | | | | | | | | | | | Make pipeline failure Slack notifications prettier and more informative Closes #32495 See merge request gitlab-org/gitlab-ce!27683
| * | Improve pipeline status Slack notifications32495-improve-slack-notification-on-pipeline-statusNathan Friend2019-07-251-14/+211
| | | | | | | | | | | | | | | | | | This commit adds some formatting to the Slack notifications for pipeline statuses, as well as adds information about the stage and jobs that failed in the case of pipeline failure.
* | | Merge branch 'remove-nested-groups-checks' into 'master'Lin Jen-Shin2019-07-263-14/+0
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Remove code related to object hierarchy and MySQL Closes #65056 and #65055 See merge request gitlab-org/gitlab-ce!31095
| * | | Remove code related to object hierarchy in MySQLremove-nested-groups-checksHeinrich Lee Yu2019-07-253-14/+0
| | | | | | | | | | | | | | | | | | | | These are not required because MySQL is not supported anymore
* | | | Merge branch 'mc/feature/find-all-artifacts-for-sha' into 'master'Robert Speicher2019-07-262-9/+26
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Find build by sha from ref Closes #64534 and #45697 See merge request gitlab-org/gitlab-ce!30843
| * | | | Rename latest_successful to be more explicitMatija Čupić2019-07-262-8/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Reword Project#latest_successful_build_for to Project#latest_successful_build_for_ref * Reword Ci::Pipeline#latest_successful_for to Ci::Pipeline#latest_successful_build_for_ref
| * | | | Find build by sha from refMatija Čupić2019-07-232-2/+18
| | | | | | | | | | | | | | | | | | | | Adds ability to find builds by sha when only specifying a ref.
* | | | | Merge branch 'remove_deployment_metrics_deployment_platform_fallback' into ↵Kamil Trzciński2019-07-261-12/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'master' Remove fallback to deployment_platform_cluster in `DeploymentMetrics` See merge request gitlab-org/gitlab-ce!31126
| * | | | | Removes potentially incorrect, and slow fallbackremove_deployment_metrics_deployment_platform_fallbackThong Kuah2019-07-251-12/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Deployment_platform is relatively expensive and calling this after the fact means that this may not be the cluster that was deployed to. Correspondingly reduce the leeway given in the related N+1 spec
* | | | | | Prefer `flat_map` over `map` + `flatten`Peter Leitzen2019-07-262-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Convert several occurrences of `map` + `flatten` to `flat_map` where applicable.
* | | | | | Merge branch '9643-jira-api-serializer-ce' into 'master'Sean McGivern2019-07-251-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | [CE] Create serializer for Jira API JSON payload See merge request gitlab-org/gitlab-ce!31139
| * | | | | | Backport Jira issue key regexHeinrich Lee Yu2019-07-251-1/+1
| | |_|/ / / | |/| | | |
* | | | | | Merge branch 'add-caching-to-archive-endpoint' into 'master'Grzegorz Bizon2019-07-251-0/+3
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | Return an ETag header for the archive endpoint See merge request gitlab-org/gitlab-ce!30581
| * | | | | Return an ETag headers for the archive endpointadd-caching-to-archive-endpointAhmad Sherif2019-07-231-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | We use the relative path of the archive to check for archive staleness.
* | | | | | Merge branch 'make-relative-positioning-module-reusable' into 'master'Bob Van Landuyt2019-07-252-19/+39
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make RelativePositioning reusable See merge request gitlab-org/gitlab-ce!30993
| * | | | | | Make RelativePositioning reusablemake-relative-positioning-module-reusableAdam Hegyi2019-07-242-19/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | RelativePositioning module was heavily dependent on the Issue model. This changes makes it easier to reuse the functionality provided by RelativePositioning in other models. Needed by: https://gitlab.com/gitlab-org/gitlab-ee/issues/12196
* | | | | | | Enable Rubocop Performance/ReverseEachsh-rubocop-reverse-eachStan Hu2019-07-241-1/+1
| |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `Array.reverse_each` is faster than `Array.reverse.each` because: * reverse.each creates a new array then loops each element * reverse_each loops in reverse order (no intermediate array created)
* | | | | | Merge branch 'sh-enable-rubocop-hash-search' into 'master'Mayra Cabrera2019-07-242-2/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enable Rubocop Performance/InefficientHashSearch Closes #64975 See merge request gitlab-org/gitlab-ce!31117
| * | | | | | Enable Rubocop Performance/InefficientHashSearchsh-enable-rubocop-hash-searchStan Hu2019-07-242-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When used with a Hash, `.keys.include?` is bad because: 1. It performs a O(n) search instead of the efficient `.has_key?` 2. It clones all keys into separate array. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/64975
* | | | | | | Merge branch 'ce-docker_image_replication-event-generation' into 'master'Douglas Barbosa Alexandre2019-07-241-0/+5
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | Backport of https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/14597 See merge request gitlab-org/gitlab-ce!30520
| * | | | | | Backport of https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/14597ce-docker_image_replication-event-generationValery Sizov2019-07-241-0/+5
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | This is the first part of Docker Registry replication for secondary Geo node.
* | | | | | Add RateLimiter to RawControllerMayra Cabrera2019-07-241-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Limits raw requests to 300 per minute and per raw path. * Add a new attribute to ApplicationSettings so user can change this value on their instance. * Uses Gitlab::ActionRateLimiter to limit the raw requests. * Add a new method into ActionRateLimiter to log the event into auth.log Related to https://gitlab.com/gitlab-org/gitlab-ce/issues/48717
* | | | | | Merge branch '63730-fix-500-status-labels-pd' into 'master'Robert Speicher2019-07-241-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add where condition to filter out labels with type here Closes #63730 See merge request gitlab-org/gitlab-ce!30885
| * | | | | | Add where condition to filter out invalid labels with nil type63730-fix-500-status-labels-pdPatrick Derichs2019-07-241-1/+1
| |/ / / / /
* | | | | | Merge branch '62128-remove-ignore-rule-for-project-auto-devops-domain' into ↵Douwe Maan2019-07-241-4/+0
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'master' Remove ignore rule for ProjectAutoDevops#domain See merge request gitlab-org/gitlab-ce!30989
| * | | | | Remove ignore rule for ProjectAutoDevops#domain62128-remove-ignore-rule-for-project-auto-devops-domainTiger2019-07-221-4/+0
| | | | | |
* | | | | | Merge branch 'optimise-import-performance' into 'master'Stan Hu2019-07-241-18/+14
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Optimise import performance Closes #64924 See merge request gitlab-org/gitlab-ce!31045
| * | | | | | Optimise import performanceoptimise-import-performanceKamil Trzciński2019-07-241-18/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Fix `O(n)` complexity of `append_or_update_attribute`, we append objects to an array and re-save project - Remove the usage of `keys.include?` as it performs `O(n)` search, instead use `.has_key?` - Remove the usage of `.keys.first` as it performs a copy of all keys, instead use `.first.first`
* | | | | | | [ADD] outbound requests whitelistReuben Pereira2019-07-242-11/+50
| |_|/ / / / |/| | | | | | | | | | | | | | | | | Signed-off-by: Istvan szalai <istvan.szalai@savoirfairelinux.com>
* | | | | | Merge branch '52442-initial-mysql-support-removal' into 'master'Robert Speicher2019-07-234-53/+7
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove dead mysql code Closes #63191 See merge request gitlab-org/gitlab-ce!29608
| * | | | | | Remove dead MySQL codeNick Thomas2019-07-234-53/+7
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | None of this code can be reached any more, so it can all be removed
* | | | | | Merge branch 'adjust-group-level-analytics-to-accept-multiple-project-ids' ↵Stan Hu2019-07-231-3/+1
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | into 'master' Adjust group level analytics to accept multiple projects ids See merge request gitlab-org/gitlab-ce!30744
| * | | | | Add code review remarksadjust-group-level-analytics-to-accept-multiple-project-idsMałgorzata Ksionek2019-07-231-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add cr remarks Improve specs according to the review Fix schema Add cr remarks Fix naming Add cr remarks
| * | | | | Add basic project extractionMałgorzata Ksionek2019-07-231-1/+2
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To allow project filtering Prepare summary for accepting multiple groups Modify deploys group summary class Add filtering by project name in issues summary Fix rubocop offences Add changelog entry Change name to id in project filtering Fix rebase problem Add project extraction
* | | | | Merge branch 'bvl-mark-remote-mirrors-as-failed-sooner' into 'master'Robert Speicher2019-07-231-2/+2
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Mark mirrors as failed 1 hour after they started See merge request gitlab-org/gitlab-ce!30999
| * | | | | Mark mirrors as failed 1 hour after they startedBob Van Landuyt2019-07-231-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We call `Project#mark_stuck_remote_mirrors_as_failed!` from the `Git::BaseHooksService`. So that gets called every time we push tags or branches. Before this would only mark started mirrors as failed if they had been started 24 hours ago. A push would never take 24 hours, especially not when we run it so often. Lowering that threshold 1 hour should at least allow us to retry broken mirrors more often on pushes. The timeout for the initial push is set somewhat longer to accommodate for pushing large repos. Both numbers are currently picked arbitrarily.
* | | | | | Merge branch 'safe-archiving-for-traces' into 'master'Kamil Trzciński2019-07-232-0/+12
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Extra logging for new live trace architecture See merge request gitlab-org/gitlab-ce!30892
| * | | | | | Validate the existence of archived traces before removing live tracesafe-archiving-for-tracesShinya Maeda2019-07-232-0/+12
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | Often live traces are removed even though the archived trace doesn't exist. This commit checkes the existence strictly.
* | | | | | Merge branch 'sh-use-shared-state-cluster-pubsub' into 'master'Kamil Trzciński2019-07-231-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use persistent Redis cluster for Workhorse pub/sub notifications See merge request gitlab-org/gitlab-ce!30990
| * | | | | | Use persistent Redis cluster for Workhorse pub/sub notificationssh-use-shared-state-cluster-pubsubStan Hu2019-07-221-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, in Omnibus, Workhorse expected to listen via the Redis shared state cluster for the `workhorse:notifications` publish/subscribe channel, but the Rails code was using the Sidekiq queue cluster for this. To fix this inconsistency, we make the Rails code use the persistent cluster, since we don't want Workhorse to be looking at anything Sidekiq-related.
* | | | | | | Merge branch '63485-fix-pipeline-emails-to-use-group-setting' into 'master'Nick Thomas2019-07-232-0/+11
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make pipeline emails respect group email setting Closes #63485 See merge request gitlab-org/gitlab-ce!30907
| * | | | | | Make pipeline emails respect group email setting63485-fix-pipeline-emails-to-use-group-settingHeinrich Lee Yu2019-07-232-0/+11
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | When a user's notification email is set for a group, we should use that for pipeline emails
* | | | | | Merge branch 'bw-add-index-for-relative-position' into 'master'Nick Thomas2019-07-231-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add index for issues on relative position, project, and state for manual sorting See merge request gitlab-org/gitlab-ce!30542
| * | | | | | Add index for issues on relative position, projectbw-add-index-for-relative-positionBrett Walker2019-07-221-1/+1
| |/ / / / / | | | | | | | | | | | | | | | | | | and state