From 66d04fde2a1c3b8133f04f5395e69836ebc60231 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Maximilian=20Pa=C3=9F?= <22845248+mpass99@users.noreply.github.com> Date: Wed, 30 Jun 2021 09:54:45 +0200 Subject: [PATCH] Remove unused function ScaleAllEnvironments --- runner/manager.go | 13 ------------- runner/manager_mock.go | 14 -------------- 2 files changed, 27 deletions(-) diff --git a/runner/manager.go b/runner/manager.go index 7e37096..51890e7 100644 --- a/runner/manager.go +++ b/runner/manager.go @@ -56,9 +56,6 @@ type Manager interface { // The runner is deleted or cleaned up for reuse depending on the used executor. Return(r Runner) error - // ScaleAllEnvironments checks for all environments if enough runners are created. - ScaleAllEnvironments() error - // Load fetches all already created runners from the executor and registers them. // It should be called during the startup process (e.g. on creation of the Manager). Load() @@ -213,16 +210,6 @@ func (m *NomadRunnerManager) Return(r Runner) (err error) { return } -func (m *NomadRunnerManager) ScaleAllEnvironments() error { - for _, environment := range m.environments.List() { - err := m.scaleEnvironment(environment.ID()) - if err != nil { - return fmt.Errorf("can not scale up: %w", err) - } - } - return nil -} - func (m *NomadRunnerManager) Load() { for _, environment := range m.environments.List() { environmentLogger := log.WithField("environmentID", environment.ID()) diff --git a/runner/manager_mock.go b/runner/manager_mock.go index 6483ed1..142243c 100644 --- a/runner/manager_mock.go +++ b/runner/manager_mock.go @@ -97,17 +97,3 @@ func (_m *ManagerMock) Return(r Runner) error { return r0 } - -// ScaleAllEnvironments provides a mock function with given fields: -func (_m *ManagerMock) ScaleAllEnvironments() error { - ret := _m.Called() - - var r0 error - if rf, ok := ret.Get(0).(func() error); ok { - r0 = rf() - } else { - r0 = ret.Error(0) - } - - return r0 -}