summaryrefslogtreecommitdiff
path: root/app/controllers/clusters
Commit message (Collapse)AuthorAgeFilesLines
* Add gitlab-managed option to clusters formMayra Cabrera2019-05-031-0/+2
| | | | | | | | When this option is enabled, GitLab will create namespaces and service accounts as usual. When disabled, GitLab wont create any project specific kubernetes resources Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/56557
* DELETE clusters/:id/:application endpointThong Kuah2019-04-291-0/+13
| | | | Add endpoint to delete/uninstall a cluster application
* Make the monitoring bundle reusablejivl-add-feature-flag-gon-eeJose Vargas2019-04-161-0/+3
| | | | Also push the frontend flag in the clusters controller
* Renames Cluster#managed? to provided_by_user?Mayra Cabrera2019-03-291-5/+5
| | | | | | This will allow to user the term managed? on https://gitlab.com/gitlab-org/gitlab-ce/issues/56557. Managed? will be used to distinct clusters that are automatically managed by GitLab
* Creates Clusterss::ApplciationsController update endpointJoão Cunha2019-03-051-7/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Creates new route - Creates new controller action - Creates call stack: Clusterss::ApplciationsController calls --> Clusters::Applications::UpdateService calls --> Clusters::Applications::ScheduleUpdateService calls --> ClusterUpdateAppWorker calls --> Clusters::Applications::PatchService --> ClusterWaitForAppInstallationWorker DRY req params Adds gcp_cluster:cluster_update_app queue Schedule_update_service is uneeded Extract common logic to a parent class (UpdateService will need it) Introduce new UpdateService Fix rescue class namespace Fix RuboCop offenses Adds BaseService for create and update services Remove request_handler code duplication Fixes update command Move update_command to ApplicationCore so all apps can use it Adds tests for Knative update_command Adds specs for PatchService Raise error if update receives an unistalled app Adds update_service spec Fix RuboCop offense Use subject in favor of go Adds update endpoint specs for project namespace Adds update endpoint specs for group namespace
* Merge branch 'fix-misspellings-app-comments' into 'master'Rémy Coutable2019-02-261-1/+1
| | | | | Fix misspellings in app/spec comments See merge request gitlab-org/gitlab-ce!25517
* Revert "Merge branch 'fix-misspellings-app-comments' into 'master'"Michael Kozono2019-02-251-1/+1
| | | | | This reverts commit 9202bbd129537a698b986e6295d0c783b5a84815, reversing changes made to 4b282e9ce1ae246c4538b3ede18d1380ea778029.
* Fix misspellings in app/spec commentsTakuya Noguchi2019-02-251-1/+1
| | | | Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
* Addresses backend/db review commentsMayra Cabrera2019-02-041-2/+2
| | | | | | | | | - Fixes multiple typos on AutoDevops script - Add an alias to Clusters::Cluster#domain as base_domain, so it's more descriptive - Removes unnecessary memoization on qa specs - Changes migration to a post migration to deal better with traffic on big instances (like gitlab.com)
* Moves domain setting to Cluster settingMayra Cabrera2019-02-041-0/+2
| | | | | | | | | | | | Changes domain field to be on the Cluster page show, removing it from Auto DevOps setting. Also injects the new environment variable KUBE_INGRESS_BASE_DOMAIN into kubernetes#predefined_variables. Migration to move the information from ProjectAutoDevops#domain to Clusters::Cluster#domain. As well as necessary modifications to qa selectors Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/52363
* Use finder to decide to show note to userThong Kuah2018-12-171-1/+13
| | | | | | | | | | | Given the note is about how to interpret ancestor clusters, use the finder which actually knows if there are any ancestor clusters to find out if the note should be shown, rather than passing the same info via a view to a helper Added note about Kaminari.paginate_array Link to followup issue too
* Show clusters of ancestors in cluster list pageThong Kuah2018-12-171-2/+2
| | | | | Show both the cluster(s) of the clusterable, and the cluster(s) of ancestor groups.
* Extract Clusters::BuildServiceDylan Griffith2018-12-101-21/+6
|
* Refactor to re-use similar blockThong Kuah2018-12-101-3/+9
|
* Set cluster_type for a new clusterThong Kuah2018-12-101-0/+9
| | | | | This enables #allow_user_defined_namespace? to work so that the form for a group cluster will now longer show the "Project namespace" field.
* Allows user to override default issuer email for cert managerAmit Rathi2018-12-071-1/+1
|
* Only project clusters has Project Namespace fieldThong Kuah2018-11-081-2/+2
| | | | | Group clusters should not allow Project Namespace so don't show that field input too
* Resolve controller sharing concernThong Kuah2018-11-012-24/+226
| | | | | | | | | | | | | | Use ClustersController as base while having Projects::ClustersController to inform what `clusterable` is. Thanks @ayufan for the great suggestion ! - View changes to work with new approach - Fix javascript for new approach - Fix feature specs for new approach - Fix QA
* Restore 403 functionality for external auth (EE)Thong Kuah2018-11-011-1/+2
| | | | | | | | | | When we unhooked ClustersController from Project::ApplicationsController, we missed an EE override to handle_not_found_or_authorized. Rather than carry on with override RoutingActions, make a specific proc for Project that we override in EE instead. Use that proc in both Clusters::BaseController and Project::ApplicationsController.
* Move view and path concerns to presentersThong Kuah2018-11-011-22/+2
| | | | | | | - Move show path for cluster to ClusterPresenter - Create ClusterablePresenter to encapsulate logic. Consolidates scattered methods from BaseController and ClustersHelper into an object.
* Abstract out project out of ClustersControllerThong Kuah2018-11-012-2/+32
| | | | | | | | | | | | | | | | | | | To the extent possible swap out `project` with `clusterable` - Abstract paths for showing cluster or clusters. This will allow us to swap in alternative paths for group level cluster - Push :project_id and :namespace_id params from the URL to the POST body. - Create a nice helper for to generate links for the destroy action For some reason, spec :project_id and :namespace_id param are not going through `to_param` for a JSON format. Manually call `to_param` to fix specs. - Move :layout to BaseController
* Convert clusters to use a top-level controllerThong Kuah2018-11-012-0/+70
In preparation so that we can create both cluster attached to project and cluster attached to group. - Move ClustersController to top level - Move Clusters::ApplicationsController to top-level too - Creates a Clusters::BaseController to share common functions - Do not rely on @project ivar. Anything could set the ivar. - Fix Vue page components due to new data-page value Because of the controller change we have gone from `projects:clusters:new` to `clusters:new`, so we need to update the file location of the page components. There is somewhere a function that will convert data-page to a file location. On that note, projects/clusters/gcp/new/, translate to Projects::Clusters::Gcp#new doesn't exist so replace that with clusters/create_gcp/ and clusters/create_user/
View Lorry Controller Status