summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Cache Flipper persisted names directly to local memory storagesh-cache-flipper-names-memory-cacheStan Hu2019-07-023-2/+9
| | | | | | | Now that application settings are no longer dominating network traffic, we see that the Feature#persisted_names is using a significant amount of CPU and network bandwidth for Redis. Move this cache into the thread-local memory storage to reduce Redis overhead.
* 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
| * | | | | | Revert "Elaborate instructions and add a reference"Hordur Freyr Yngvason2019-07-021-1/+46
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 092542d2fa5c036e4fc01ba3185f1dc0e9ddbebb. Included a stray file
* | | | | | Merge branch 'vue-i18n-js-clusters-directory' into 'master'Mike Greiling2019-07-024-9/+20
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Vue-i18n: autofix for app/assets/javascripts/clusters directory See merge request gitlab-org/gitlab-ce!29962
| * | | | | | Vue-i18n: app/assets/javascripts/clusters/ directoryEzekiel Kigbo2019-07-024-9/+20
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | i18n linting for .vue files under the app/assets/javascripts/clusters/ directory
* | | | | | Merge branch 'private-registry-docs' into 'master'Evan Read2019-07-011-30/+75
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Expand docs on configuring jobs with private registry access Closes gitlab-runner#3963 See merge request gitlab-org/gitlab-ce!27231
| * | | | | | Expand docs on configuring jobs with private registry accessRoss Vandegrift2019-07-011-30/+75
|/ / / / / / | | | | | | | | | | | | | | | | | | Fixes: https://gitlab.com/gitlab-org/gitlab-runner/issues/3963
* | | | | | Merge branch '82-add-new-onboarding-to-flipper-a-b-testing' into 'master'Nick Thomas2019-07-013-0/+46
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | Add new onboarding to flipper A/B testing See merge request gitlab-org/gitlab-ce!30125
| * | | | | Add A/B testing support to onboarding82-add-new-onboarding-to-flipper-a-b-testingAsh McKenzie2019-06-281-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Check if we're gitlab.com and the :user_onboarding feature flag is enabled.
| * | | | | Add new OnboardingExperimentHelper modulesAsh McKenzie2019-06-282-0/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | OnboardingExperimentHelpers take care of determining if the current_user should or should not see the new onboarding feature.
* | | | | | Merge branch 'renovate/gitlab-svgs-1.x' into 'master'Clement Ho2019-07-012-5/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update dependency @gitlab/svgs to ^1.66.0 See merge request gitlab-org/gitlab-ce!30193
View Lorry Controller Status