airship/airshipctl
Code Issues Proposed changes

Remove unused fields from ExecutorConfig

Browse Source 
AirshipConfig and BundleName fields are not used within Executors,
so they should be removed.

Change-Id: I23c54b854938ae554e888102a41c7ec215b85c27
Signed-off-by: Ruslan Aliev <raliev@mirantis.com>
This commit is contained in:
Ruslan Aliev 2021-04-03 16:52:15 -05:00
parent 346196e6c1
commit 675f718317
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
pkg/phase/ifc/executor.go
View File

@ -19,7 +19,6 @@ import (
"time"
"opendev.org/airship/airshipctl/pkg/cluster/clustermap"
"opendev.org/airship/airshipctl/pkg/config"
"opendev.org/airship/airshipctl/pkg/document"
"opendev.org/airship/airshipctl/pkg/events"
"opendev.org/airship/airshipctl/pkg/k8s/kubeconfig"
@ -61,11 +60,9 @@ type ExecutorFactory func(config ExecutorConfig) (Executor, error)
type ExecutorConfig struct {
PhaseName string
ClusterName string
BundleName string
ClusterMap clustermap.ClusterMap
ExecutorDocument document.Document
AirshipConfig *config.Config
Helper Helper
KubeConfig kubeconfig.Interface
BundleFactory document.BundleFactoryFunc