summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Skip canceling builds60666-kubernetes-applications-uninstall-runnerJoão Cunha2019-07-112-18/+3
|
* Remove transaction from make_uninstallingJoão Cunha2019-07-081-1/+1
|
* Apply suggestion to app/models/clusters/applications/runner.rbJoão Cunha2019-07-081-2/+3
|
* Apply suggestion to app/models/clusters/applications/runner.rbJoão Cunha2019-07-081-1/+1
|
* Implements post_uninstall in favor of dependent destroyJoão Cunha2019-07-085-4/+20
| | | | | - We want to avoid having future problem with a dependent destroy after hook.
* Pause ci_runner on application_runner prepare_uninstallJoão Cunha2019-07-042-1/+15
|
* Apply suggestion to app/models/clusters/applications/runner.rbJoão Cunha2019-07-041-1/+1
|
* Cancel builds before uninstalling with Helm and remove runnerJoão Cunha2019-07-034-7/+45
| | | | | | | | | | - Use the make_uninstalling transition to cancel builds related to the Clusters::Applications::Runner. If we uninstall with Helm before doing this, the pod that is running the build might get stuck and never be done. - Remove associated Ci::Runner after Clusters::Applications::Runner is destroyed.
* Rename variable to represent what it really isJoão Cunha2019-07-031-10/+10
|
* Enable GitLabb runner to be uninstalled from clusterJoão Cunha2019-07-024-9/+14
| | | | | | - Set as uninstallable app - Update docs - Adjust specs
* Merge branch '59177-squash-old-migrations' into 'master'harish-and-cindy-ssl-check-rake-taskharish-and-cindy-ldap-sync-rake-tasksDmitriy Zaporozhets2019-07-02876-22421/+1855
|\ | | | | | | | | | | | | Squash old migrations Closes #59177 See merge request gitlab-org/gitlab-ce!30226
| * Run the db:rollback-pg CI job from 2018Nick Thomas2019-07-021-1/+1
| |
| * Remove migrations from setup_postgresqlNick Thomas2019-07-021-14/+0
| |
| * Remove background migrations for old schemasNick Thomas2019-07-0213-1637/+0
| | | | | | | | | | | | | | | | | | | | On the assumption that a background migration whose specs need a schema older than 2018 is obsoleted by this migration squash, we can remove both specs and code for those that fail to run in CI as a result of the schema at that date no longer existing. This is true for all but the MigrateStageStatus background migration, which is also used from the MigrateBuildStage background migration.
| * Remove migration specs from 201[4567]Nick Thomas2019-07-0246-2948/+0
| | | | | | | | Since the migrations are gone, we don't need these specs either
| * Bring the InitSchema migration up to standardNick Thomas2019-07-021-1/+11
| |
| * Squash old migrationsNick Thomas2019-07-02815-17821/+1844
| | | | | | | | Use the `squasher` gem to squash migrations older than the start of 2018
* | Merge branch ↵Phil Hughes2019-07-0212-89/+219
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | '45104-special-characters-in-project-name-path-prevent-users-from-using-the-container-registry' into 'master' Resolve "Special characters in project name/path prevent users from using the container registry" Closes #62770 and #45104 See merge request gitlab-org/gitlab-ce!29616
| * | Registry component now includes error messagesNick Kipling2019-07-0212-89/+219
|/ / | | | | | | | | | | | | | | | | Includes changes to the Vue container registry component to support the display of an empty message and error message when invalid characters are found in the group, project or branch name. repositories/index.html.haml has been changed to remove the content and place into Vue component.
* | Merge branch 'feature/uninstall_cluster_ingress' into 'master'Dmitriy Zaporozhets2019-07-027-30/+42
|\ \ | | | | | | | | | | | | Feature/uninstall cluster ingress See merge request gitlab-org/gitlab-ce!29977
| * | Enables ingress for uninstallationJoão Cunha2019-07-027-30/+42
|/ / | | | | | | - JupyterHub must not be available (installed or updated)
* | Merge branch 'disable-junit-reports' into 'master'Marin Jankovski2019-07-022-6/+6
|\ \ | | | | | | | | | | | | Disable juint reports tmp See merge request gitlab-org/gitlab-ce!30254
| * | Disable juint reports tmpdisable-junit-reportsSteve Azzopardi2019-07-022-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The junit reports are creating 90mb files inside of redis for `gitlab-ce/ee` project. There is an ongoing production incident https://gitlab.com/gitlab-com/gl-infra/production/issues/928 that is because of high CPU uitlaization on the redis nodes. GitLab-ce/ee are the highest consumers of memory becuase of these junit reports. This is a bandage until we can scale redis properly.
* | | Merge branch '60859-upload-after-delete' into 'master'Phil Hughes2019-07-0212-15/+173
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Resolve "Cannot delete and upload file of the same name in the Web IDE" Closes #60859 See merge request gitlab-org/gitlab-ce!30239
| * | | Allow adding new entries to existing pathDenys Mishunov2019-07-0212-15/+173
|/ / / | | | | | | | | | | | | | | | | | | If an entry has been removed from the tree and later, during the same session, a user is trying to add a new item with the same name/path we should allow for that as long as the original entry is marked with `deleted`.
* | | Merge branch '12532-services-form' into 'master'Phil Hughes2019-07-021-1/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Removes EE differences for app/views/admin/services/_form.html.haml Closes gitlab-ee#12532 See merge request gitlab-org/gitlab-ce!30231
| * | | Removes EE differences12532-services-formFilipa Lacerda2019-07-011-1/+1
| | | |
* | | | Merge branch 'sh-add-thread-memory-cache' into 'master'Rémy Coutable2019-07-0213-53/+105
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | Add a memory cache local to the thread to reduce Redis load Closes #63977 See merge request gitlab-org/gitlab-ce!30233
| * | | Fix failing spec/models/user_spec.rbStan Hu2019-07-021-1/+1
| | | |
| * | | Use an uncached application setting for usage ping checksStan Hu2019-07-022-3/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The introduction of the in-memory cache for application settings had a side effect of making it harder to invalidate changes when the settings occur. We now bypass the cache because it's possible the admin enabled the usage ping, and we don't want to annoy the user again if they already set the value. To avoid causing significant load on the system, we add an extra check to ensure the user is an admin. and we don't want to annoy the user again if they already set the value. This is a bit of hack, but the alternative would be to put in a more complex cache invalidation step. Since this call only gets called in the uncommon situation where the user is an admin and the only user in the instance, this shouldn't cause too much load on the system.
| * | | Fix broken specs due to cached application settingsStan Hu2019-07-014-42/+49
| | | | | | | | | | | | | | | | | | | | | | | | The /admin panel will now always return an uncached application setting to ensure it always has the most current info.
| * | | Fix specs in spec/models/concerns/cacheable_attributes_spec.rbStan Hu2019-07-011-4/+5
| | | |
| * | | Add a memory cache local to the thread to reduce Redis loadStan Hu2019-07-016-3/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Loading `ApplicationSetting` from Redis was responsible for at least 50% of the CPU load of the Redis cluster on GitLab.com. Since these values generally don't change very much, we can load this from the database and cache it in memory, skipping Redis altogther. We use `ActiveSupport::Cache::MemoryStore` as a drop-in replacement for `RedisCacheStore` even though we probably don't need synchronized access within `Thread.current`. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/63977
* | | | Merge branch '58808-fix-image-diff-on-text' into 'master'James Lopez2019-07-023-1/+39
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Don't show image diff note on text file See merge request gitlab-org/gitlab-ce!30221
| * | | | Don't show image diff note on text file58808-fix-image-diff-on-textPatrick Bajao2019-07-013-1/+39
| | | | |
* | | | | Merge branch 'docs-clarify-10k-architecture-resources-per-node' into 'master'Achilleas Pipinellis2019-07-021-5/+5
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Clarify 10k architecture resource per node Closes #63424 See merge request gitlab-org/gitlab-ce!30237
| * | | | | Clarify 10k architecture resource per nodedocs-clarify-10k-architecture-resources-per-nodeabuerer2019-07-011-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | Adding a note to the 10k reference architecturre that the resource recommendations are per node
* | | | | | Merge branch 'fix-bare-urls-in-projects' into 'master'Achilleas Pipinellis2019-07-0230-109/+116
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix most bare URLs in project Closes gitlab-com/gitlab-docs#287 See merge request gitlab-org/gitlab-ce!30240
| * | | | | | Fix most bare URLs in projectEvan Read2019-07-0230-109/+116
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Linting rule not enabled for now because tooling produces false positives.
* | | | | | Merge branch 'docs-update-global-nav' into 'master'Achilleas Pipinellis2019-07-021-21/+13
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | | | | | | | | | | | | | Docs: update global nav document See merge request gitlab-org/gitlab-ce!30230
| * | | | | Docs: update global nav documentMarcia Ramos2019-07-021-21/+13
|/ / / / /
* | | | | Merge branch 'memoize-namespace-root-ancestor' into 'master'Thong Kuah2019-07-021-1/+3
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Memoizing root_ancestor in Namespace See merge request gitlab-org/gitlab-ce!30248
| * | | | | Memoizing root_ancestor in NamespaceAdam Hegyi2019-07-021-1/+3
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | The root_ancestor method is heavily used in EE for the group_saml feature. Having this memoization implemented in CE would eliminate the need of overriding the root_ancestor method in EE.
* | | | | Merge branch 'project-cluster-spec-improvements' into 'master'Jan Provaznik2019-07-022-44/+19
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Tidy spec for API::ProjectClusters See merge request gitlab-org/gitlab-ce!30154
| * | | | | Tidy spec for API::ProjectClustersproject-cluster-spec-improvementsDylan Griffith2019-07-022-44/+19
| | | | | |
* | | | | | Merge branch 'qa-transfer-project-spec' into 'master'Sanad Liaquat2019-07-0219-32/+229
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | QA: transfer project spec See merge request gitlab-org/gitlab-ce!28255
| * | | | | | QA: transfer project specJennifer Louie2019-07-0219-32/+229
|/ / / / / /
* | | | | | Merge branch '63984-rename-in-folder' into 'master'Kushal Pandya2019-07-022-1/+14
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Hid re-named entries in subfolders Closes #63984 See merge request gitlab-org/gitlab-ce!30242
| * | | | | | Hid re-named entries in subfolders63984-rename-in-folderDenys Mishunov2019-07-022-1/+14
| | | | | | |
* | | | | | | Merge branch 'hfy/add-missing-step-to-external-knative-docs' into 'master'Evan Read2019-07-021-1/+46
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | Add ClusterRole step to external Knative docs See merge request gitlab-org/gitlab-ce!30123
View Lorry Controller Status