Age | Commit message (Collapse) | Author |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Group level JupyterHub
See merge request gitlab-org/gitlab-ce!32512
|
|
- We're also improving specs to avoid this could happen again
|
|
Removes limitations on cluster types that can install JupyterHub
|
|
Extract duplicated code from two similar classes into a parent one.
|
|
- Set as uninstallable app
- Update docs
- Adjust specs
|
|
Tweak cluster helper and refactor its specs.
|
|
|
|
|
|
|
|
|
|
Add endpoint to delete/uninstall a cluster application
|
|
Rescue and put into :uninstall_errored if something goes wrong while
destroying, which can happen. I think it is safe to expose the full
error message from the destroy error.
Remove the :uninstalled state as no longer used.
|
|
+ to monitor progress of uninstallation pod
|
|
Add frozen_string_literal to spec/services
See merge request gitlab-org/gitlab-ce!27267
|
|
As this are un-expected errors which we should hear about from Sentry.
Still rescue StandardError when operating a Helm action as we can get
non Kubeclient errors such as SSL certificate or network errors
|
|
Probably useful as we often move these files to "new" files.
|
|
DRY up tests for logging error.
|
|
Just the error message and error class alone makes it hard to determine
the full context of any errors, so we need to know where the error is
occuring as well.
|
|
A runner installed on a cluster will now use the
cluster's `cluster_type` as its `runner_type`.
|
|
|
|
- This is to avoid colision with EE ClusterUpdateAppWorker
|
|
- 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
|
|
- deletes schedule_installation_service.rb
- moves schedule_installation_service.rb logic to create_service.rb
- moves specs as well
Removes code duplication
Remove unecessary spec block
Abide review suggestions
Test installable applications which are not associated to a cluster
Fix a typo
Removes duplciated expectation
Reuse variable instead of redefining
Remove method in favor of a local scoped lambda
Improve 'failing service' shared examples
Test the increase of status count
Remove duplicated test
Enable fronzen literal
|
|
|
|
|
|
Json logging for k8s Integration
See merge request gitlab-org/gitlab-ce!23328
|
|
|
|
Don't remove failed install pods
Closes #51792
See merge request gitlab-org/gitlab-ce!23350
|
|
We want to keep failed install pods around so that it is easier to debug
why a failure occured. With this change we also need to ensure that we
remove a previous pod with the same name before installing so that
re-install does not fail.
Another change here is that we no longer need to catch errors from
delete_pod! in CheckInstallationProgressService as we now catch the
ResourceNotFoundError in Helm::Api. The catch statement in
CheckInstallationProgressService was also probably too broad before and
should have been narrowed down simply to ResourceNotFoundError.
|
|
|
|
|
|
These applications will need further work in future issues to enable
them to work with groups of projects.
|
|
|